Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-12-06 Thread intrigeri
winterfa...@riseup.net wrote (02 Dec 2013 19:50:44 GMT) : Do you think it is enough to resolve the ticket (#6207)? Replied on #6207. Cheers, -- intrigeri | GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc | OTR fingerprint @

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-12-02 Thread winterfairy
intrigeri wrote: winterfairy wrote: For the other repositories: The bug says something about Makefile targets, but it is okay that it is a script, right? And that the user must manually invoke the script to pull translations, just like in the tails repo right now? I think it's a very good

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-12-01 Thread winterfairy
intrigeri wrote: winterfairy at riseup.net wrote (30 Nov 2013 19:00:56 GMT) : There was a purpose of the code in import-translations that was removed by commit: e76059cd74bec591d7104b65e3f188e43a36ef7f Commits b2a3952 and d62bf4c (directly pushed to devel, sorry) should restore the

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-12-01 Thread intrigeri
winterfa...@riseup.net wrote (01 Dec 2013 17:57:12 GMT) : If you wait with reviewing the greeter repository, I can rebase it with an import-translations script that more closely resembles the one now in the tails repo. Would this be good? Yes, please :) For the other repositories: The bug

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-11-30 Thread intrigeri
Hi, winterfa...@riseup.net wrote (29 Nov 2013 14:14:46 GMT) : Please review and merge: - repo winterfairy/tails, branch import-translations (based on devel) Merged and pushed some refactoring commits on top. bertagaz should be reviewing my own changes soonish. Works fine, thanks! -

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-11-30 Thread bertagaz
On Sat, Nov 30, 2013 at 04:08:35PM +0100, intrigeri wrote: Hi, winterfa...@riseup.net wrote (29 Nov 2013 14:14:46 GMT) : Please review and merge: - repo winterfairy/tails, branch import-translations (based on devel) Merged and pushed some refactoring commits on top. bertagaz

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-11-30 Thread winterfairy
intrigeri wrote: winterfairy at riseup.net wrote (29 Nov 2013 14:14:46 GMT) : Please review and merge: - repo winterfairy/tails, branch import-translations (based on devel) Merged and pushed some refactoring commits on top. There was a purpose of the code in import-translations that was

Re: [Tails-dev] Scripts for importing Transifex translations, please review

2013-11-30 Thread intrigeri
winterfa...@riseup.net wrote (30 Nov 2013 19:00:56 GMT) : There was a purpose of the code in import-translations that was removed by commit: e76059cd74bec591d7104b65e3f188e43a36ef7f OK, got it. I'm not excessively convinced by the if someone commits without running refresh-translations

[Tails-dev] Scripts for importing Transifex translations, please review

2013-11-29 Thread winterfairy
I have started working on bug #6288, automating the Update PO files step of the release process. This is done with a new import-translations script that automatically import completed translations from Transifex (through torproject.org:translation.git), except for French and Portuguese (Brazil).