Skip to content

/ pgjdbc

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

Jump to bottom

fix: restore clobbered translations (cs, de, fr, it) #409

Merged
merged 2 commits into from Oct 26, 2015
Merged

fix: restore clobbered translations (cs, de, fr, it) #409

merged 2 commits into from Oct 26, 2015

Conversation

@jcflack
Copy link
Contributor

@jcflack jcflack commented Oct 25, 2015

This should close #408.

I notice there is an open #348 to make update-translations.sh be automatically run (contrary to the maintainer instructions on the translations page). I wonder if the project has escaped embarrassment in international quarters exactly because that didn't happen in 2013 when these files got clobbered....
jcflack added 2 commits Oct 25, 2015
@jcflack
fix: recover cs, de, fr, it translations
1793454 
These four translations were clobbered in commit 86c0db2 when the
contributor apparently opened these iso-8859-1 and -2 files in a
utf-8 editor to make three small changes, the editor changed all
the non-ascii characters to REPLACEMENT CHARACTER everywhere, and
the files were saved and committed that way. This change just reverts
86c0db2, leaving the three small spelling changes to be reapplied.
@jcflack
fix: redo spelling changes in unclobbered .po
Loading status checks…
5bfb26d 
Re-apply the original spelling changes after recovering the cs, de, fr,
and it translations from the commit that clobbered them.

Closes #408.
@jcflack
Copy link
Contributor Author

@jcflack jcflack commented Oct 26, 2015

Yup ... the .class files all still have their old unclobbered contents (and the old uncorrected spelling occured in three places, so they won't find the translations for those 3 messages).

So somebody ought to run update-translations.sh at some point and commit the updated .class files.

According to msgfmt, though, there's one fatal error in the Japanese translation: a string that has a format element {0} in the original doesn't have it in the translation. I don't read Japanese so I can't tell where the {0} should go.
@davecramer
Copy link
Member

@davecramer davecramer commented Oct 26, 2015

The spelling error wasn't a huge deal... ;) But I will run it.

Dave Cramer

On 25 October 2015 at 20:13, Chapman Flack notifications@github.com wrote:

Yup ... the .class files all still have their old unclobbered contents
(and the old uncorrected spelling occured in three places, so they won't
find the translations for those 3 messages).

So somebody ought to run update-translations.sh at some point and commit
the updated .class files.

According to msgfmt, though, there's one fatal error in the Japanese
translation: a string that has a format element {0} in the original
doesn't have it in the translation. I don't read Japanese so I can't tell
where the {0} should go.

—
Reply to this email directly or view it on GitHub
#409 (comment).
@jcflack
Copy link
Contributor Author

@jcflack jcflack commented Oct 26, 2015

The spelling itself could not be a big deal, but with the spelling fixed in the driver code but the messages_*.class files not updated, those three messages won't match when looked up, so I think they'll be output in English instead of in translation. (Affecting all 15 translations, not just the 4 that got zonked.)

Which might also not be such a big deal, but might surprise somebody....
davecramer added a commit that referenced this pull request Oct 26, 2015
@davecramer
Merge pull request #409 from jcflack/master
Loading status checks…
e66b085 
fix: restore clobbered translations (cs, de, fr, it)
@davecramer davecramer merged commit e66b085 into pgjdbc:master Oct 26, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
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
Linked issues

Successfully merging this pull request may close these issues.

Translations (cs, de, fr, and it) got clobbered in 2013
2 participants
@jcflack @davecramer