Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Use transifex for i18n #2831

Merged
merged 10 commits into from
Oct 28, 2014
Merged

Use transifex for i18n #2831

merged 10 commits into from
Oct 28, 2014

Conversation

Changaco
Copy link
Contributor

For #2642.

Translations are synced between git and transifex in release.sh. The Transifex password is automatically pulled from the Heroku environment variables when necessary.

I've temporarily removed the Chinese translation because it's very incomplete.

@@ -1,1042 +1,773 @@

# Translations template for PROJECT.
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this is no big deal, but is PROJECT a variable that should be updated at transifex?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IIRC this comment is generated by Babel, not Transifex.

chadwhitacre added a commit that referenced this pull request Oct 28, 2014
@chadwhitacre chadwhitacre merged commit c7e3c17 into master Oct 28, 2014
@chadwhitacre chadwhitacre deleted the i18n-transifex branch October 28, 2014 15:25
@chadwhitacre
Copy link
Contributor

Tried running the new release.sh and hit this:

writing PO template file to i18n/core.pot
env/bin/tx push -s
/Users/whit537/.transifexrc not found.
Pushing translations for resource gratipay-com.core:
Pushing source file (i18n/core.pot)
Done.
rm i18n/*.pot
env/bin/tx pull -a -f --mode=reviewed --minimum-perc=50
Pulling translations for resource gratipay-com.core (source: i18n/core.pot)
 -> eo: i18n/core/eo.po
 -> ko: i18n/core/ko.po
 -> fr: i18n/core/fr.po
Done.
sed -r -e '/^"POT?-[^-]+-Date: /d' -i i18n/*/*.po
sed: illegal option -- r
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]
make: *** [i18n_download] Error 1
[gratipay] $

@Changaco
Copy link
Contributor Author

Fixed in 31b5ea7.

chadwhitacre added a commit that referenced this pull request Oct 28, 2014
In 31b5ea7 we fixed one sed portability
issue introduced in #2831. Here's another.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants