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.

Sign up for GitHub

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

Jump to bottom

Update guide_forgejo.rst #1851

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Update guide_forgejo.rst #1851

wants to merge 6 commits into from

Conversation

rdwz
Copy link
Contributor

@rdwz rdwz commented Dec 27, 2024 •
edited
Loading

  • fetch latest version from feed
  • use vars $FORGEJO_HOME and ${latest}
    • remove hardcoded version
  • download compressed binary
  • create db with utf8mb4_bin collation
    • remove warning on migration

rdwz added 5 commits December 27, 2024 18:03
@rdwz
Update guide_forgejo.rst
e1a6e3f 
- update to forgejo v9.0.3
- fetch latest version from feed
@rdwz
Update guide_forgejo.rst
4e32dea 
- create db with `utf8mb4_bin` collation
- removes warning on db migration
@rdwz
Update guide_forgejo.rst
1602f3a 
- download compressed binary
- use vars `$FORGEJO_HOME` and `${latest}`
@rdwz
Update guide_forgejo.rst
946c148 
- use $HOME instead of `tilde`
@rdwz
Update guide_forgejo.rst
a23eee4
@nichtmax
Copy link
Member

Hey thank you for your PR!

  • create db with utf8mb4_bin collation

May I ask why this is needed?

@nichtmax nichtmax marked this pull request as draft January 22, 2025 12:54
@toddy15
Copy link
Contributor

toddy15 commented Feb 28, 2025

  • create db with utf8mb4_bin collation

May I ask why this is needed?

Forgejo emits a warning during the DB installation otherwise, this change prevents that. See line 169 in the original file:

[W] Current database has been altered to use collation "utf8mb4_bin"

@rdwz
Copy link
Contributor Author

rdwz commented Mar 17, 2025

  • create db with utf8mb4_bin collation

May I ask why this is needed?

@nichtmax

  • As mentioned by @toddy15 Forgejo emits a warning during db install
  • This ensures proper functionality by enabling accent- and case-sensitive comparisons, which Forgejo relies on

You can find more details in the Forgejo docs

@rdwz rdwz marked this pull request as ready for review March 17, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rdwz @nichtmax @toddy15