Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Gitea to v1.21.4 #5929

Merged
merged 16 commits into from
Jan 22, 2024
Merged

Update Gitea to v1.21.4 #5929

merged 16 commits into from
Jan 22, 2024

Conversation

wkobiela
Copy link
Contributor

@wkobiela wkobiela commented Nov 18, 2023

Description

This PR includes the following changes:

  1. Update Gitea to v1.21.4
  2. Use redesigned shared folder structure
  3. Amend upgrade and install wizards

Fixes #5481

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Package update

@wkobiela wkobiela mentioned this pull request Nov 18, 2023
10 tasks
@wkobiela
Copy link
Contributor Author

wkobiela commented Nov 18, 2023

Gitea requires Go 1.21 or greater to build. You can get it at https://go.dev/dl/

#5930

@efussi
Copy link

efussi commented Nov 18, 2023

@wkobiela what about updating Gitea to v1.20.5 which doesn't require a Go version update (as an intermediate step), so we don't have to wait for cloudflare/cloudflared#1054?

@wkobiela
Copy link
Contributor Author

Trying to do that here #5931

@hgy59 hgy59 mentioned this pull request Jan 15, 2024
3 tasks
@wkobiela wkobiela changed the title Update Gitea to v1.21.0 Update Gitea to v1.21.4 Jan 17, 2024
@wkobiela
Copy link
Contributor Author

As go was updated, trying to bump gitea further over 1.21.0 release

older PR with newest bump < 1.21.0 #5931

@wkobiela
Copy link
Contributor Author

Tested on DSM7.1 x64
fresh install with mysql database working OK
update from 1.17.3 with mysql database -> to 1.21.4 working OK

@wkobiela
Copy link
Contributor Author

Is there anything more I can do to merge this package?

@hgy59
Copy link
Contributor

hgy59 commented Jan 21, 2024

@wkobiela can you please update the check list in the description? So far no item is checked.

@wkobiela
Copy link
Contributor Author

Updated. Tested on vm dsm7, don't have an option to test on dsm6 right now

@mreid-tt
Copy link
Contributor

@wkobiela I've tested under DSM 6 and it seems to work okay. I see your note for upgrades to earlier versions and I had to manually edit my config using:

sudo vi /var/packages/gitea/var/conf.ini

This was to treat with the following warnings in the admin:

Deprecated config option `[server]` `LFS_CONTENT_PATH` present. Use `[lfs]` `PATH` instead. This fallback will be/has been removed in v1.19.0
Deprecated config option `[log]` `ROUTER` present. Use `[log]` `logger.router.MODE` instead. This fallback will be/has been removed in 1.21

Ideally, it would be nice to use sed to fix the config files for users but that's more of a nice to have. I've also added some suggestions above to make the installation wizard screen a bit more compact and cleaner looking.

All in all, the update seems to work well for clean installs as well as updates.

@wkobiela
Copy link
Contributor Author

Ideally, it would be nice to use sed to fix the config files for users but that's more of a nice to have. I've also added some suggestions above to make the installation wizard screen a bit more compact and cleaner looking.

I suppose it would be nice, but hard to guess what custom changes in configuration file particular user made. (I guess that these two deprecated options are just initial setup options - all differences between clear setup of 1.17 vs upgrade to 1.21).

If it is okay, I would stick with information to update config.ini manually

@mreid-tt
Copy link
Contributor

mreid-tt commented Jan 21, 2024

LGTM

@hgy59 any objections to me merging and publishing?

@hgy59 hgy59 merged commit 8873f21 into SynoCommunity:master Jan 22, 2024
17 checks passed
@hgy59 hgy59 mentioned this pull request Jan 22, 2024
10 tasks
@hgy59
Copy link
Contributor

hgy59 commented Jan 22, 2024

@mreid-tt can you please publish this?

in my local env, building gitea fails with:

error obtaining VCS status: exit status 128
        Use -buildvcs=false to disable VCS stamping.

@mreid-tt mreid-tt added the status/published Published and activated (may take up to 48h until visible in DSM package manager) label Jan 22, 2024
@mreid-tt
Copy link
Contributor

@hgy59, the version has been published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/published Published and activated (may take up to 48h until visible in DSM package manager)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gitea installer creates shared folder on wrong volume
4 participants