Skip to content

Conversation

sgk
Copy link
Contributor

@sgk sgk commented Jan 18, 2013

The "app/src/processing/app" directory became mess because there are
a lot of "Resources__.po" and "Resources__.properties" files for the
internationalization.

Moved the resource related files into a directory "i18n" under the
"app/src/processing/app" directory.

This is for developers only. No feature changed.

Shigeru KANEMOTO added 5 commits February 28, 2013 22:31
The "app/src/processing/app" directory became mess because there are
a lot of "Resources_*.po" and "Resources_*.properties" files for the
internationalization.

Moved the resource related files into a directory "i18n" under the
"app/src/processing/app" directory.

This is for developers only. No feature changed.
 * The shell should be "bash" not "sh".
 * Added an ability to update specific files only.
Without sorting the key, it is difficult to maintain the language
resource files on Git.
 * Translated missing texts.
 * Small fixes for translation.
@sgk
Copy link
Contributor Author

sgk commented Mar 3, 2013

@cmaglie @ffissore @dcuartielles Sorry to ask to you directly.

I will appreciate if you can merge this pull request a little bit earlier. This will make it easier to manage the result of Transifex on Git. The last commit a6c6a71 is not required, but please include if it is not a problem (this can be done on Transifex if the POT file is up-to-date).

cmaglie added a commit that referenced this pull request Mar 8, 2013
Rearrange i18n Resource files into a directory
@cmaglie cmaglie merged commit bc90e3d into arduino:ide-1.5.x Mar 8, 2013
@cmaglie
Copy link
Member

cmaglie commented Mar 8, 2013

@sgk
do you have any suggestion to make transifex integration smoother?

@sgk
Copy link
Contributor Author

sgk commented Mar 9, 2013

@cmaglie

Thank you!

Do you mean Transifex integration like my recent commit 3dd601f ? This commit is a little bit large because it adds the Python "reqeusts" library. The core of the commit is "i18n/python/pull.py". This retrieves the translation from Transifex.
https://github.com/sgk/Arduino/blob/transifex/app/src/processing/app/i18n/python/pull.py

Please read the instruction:
https://github.com/sgk/Arduino/blob/transifex/app/src/processing/app/i18n/README.md

Currently, the script can only retrieve the translation from Transifex. I will try to write another script to "push" local file to Transifex.

@cmaglie
Copy link
Member

cmaglie commented Mar 9, 2013

Cool, I didn't noticed the pull script, that answer part of my question.

My main concern is: how can I push new strings into transifex? The 1.5.x series has a bunch of new strings to be translated.

@sgk
Copy link
Contributor Author

sgk commented Mar 9, 2013

I think it can be done by first incorporate the new strings into "*.po" files by using "./update.sh", run "./pull.sh", and then "push" the files to Transifex. I will try to write a script to "push".

@sgk
Copy link
Contributor Author

sgk commented Mar 10, 2013

I added a "./push.sh" script in the commit 4d94310.

https://github.com/sgk/Arduino/tree/transifex/app/src/processing/app/i18n

@sgk
Copy link
Contributor Author

sgk commented Mar 10, 2013

I have submitted a pull request for this. See #1312. Please merge if you like it.

You should be able to push the new strings into Transifex as follows.

% ./update.sh -a
% ./pull.sh -a
% (check the result before push)
% ./push.sh -a

This may require a project administrator permission for Transifex.

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.

2 participants