Skip to content

Conversation

@heiglandreas
Copy link
Contributor

Due to the lower-case name of the repo-folder and the mixed-case language name the brazilian translation was not updated.

This commit fixes this

Due to the lower-case name of the repo-folder and the mixed-case language name the brazilian translation was not updated. 

This commit fixes this
for i in $langs; do
cd /local/src/phpdoc-git/$i
lang=`echo "$i" | tr '[:upper:]' '[:lower:]'`
cd /local/src/phpdoc-git/$lang
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is a similar fix needed for the production build?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As the production build doesn't updatre the docs-sources by itself I doubt it. I'll recheck though!

Copy link
Contributor Author

@heiglandreas heiglandreas Feb 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And the cronjob on rsync.php.net runs a file /home/docbuilder/build-docs.sh which I can't find in this repo. So perhaps someone with access to that box (@derickr ?) needs to have a look for that...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That file got dropped in 6643326. As far as I can see, build-docs-rsync is the production build.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the same issue in build-docs-rsync.

This should fix issues with the brazilian translation.

Worst case is that the brazilian translation is checked out again and that we now have a `pt_br` and a `pt_BR` folder on the rsync machine.
@derickr
Copy link
Member

derickr commented Jul 30, 2024

Is this still relevant?

@derickr
Copy link
Member

derickr commented Aug 13, 2024

pt-br builds and pulls, so this no longer seems needed.

@derickr derickr closed this Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants