incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sebastien Goasguen <run...@gmail.com>
Subject Re: Review Request: Utility for generating the .tx/config file
Date Tue, 12 Feb 2013 08:58:11 GMT
I added a review.

This can be discarded and closed.

-sebastien

On Feb 12, 2013, at 7:49 AM, Animesh Chaturvedi <animesh.chaturvedi@citrix.com> wrote:

> Kelcey, Sebastien
> 
> This review board item has not seen any activity since Dec. Can you both bring this to
closure?
> 
> Animesh
> 
>> -----Original Message-----
>> From: Kelceydamage@bbits [mailto:kelcey@bbits.ca]
>> Sent: Monday, December 03, 2012 12:48 AM
>> To: cloudstack-dev@incubator.apache.org
>> Subject: Re: Review Request: Utility for generating the .tx/config file
>> 
>> I was unaware of the transifex tool.
>> 
>> I can switch to .slice, can you provide me an example .ent file?
>> 
>> Sent from my iPhone
>> 
>> On Dec 3, 2012, at 12:29 AM, Sebastien Goasguen <runseb@gmail.com> wrote:
>> 
>>> Doesn't transifex client create the tx config file automatically based on the
>> op/pot files created by publican ?
>>> 
>>> something like this:
>>> tx set --auto-local -r project.$resource "<lang>/$resource.po"
>>> --source-language=en --source-file "pot/$resource.pot" --execute
>>> 
>>> http://help.transifex.com/features/client/
>>> 
>>> 
>>> On Dec 3, 2012, at 9:24 AM, "David Nalley" <david@gnsa.us> wrote:
>>> 
>>>> 
>>>> -----------------------------------------------------------
>>>> This is an automatically generated e-mail. To reply, visit:
>>>> https://reviews.apache.org/r/8307/#review13963
>>>> -----------------------------------------------------------
>>>> 
>>>> 
>>>> I tested this and see at least one problem.
>>>> It's looking for a '.' as a delimiter - which means files like
>>>> 
>>>> docs/en-US/added-API-commands-4.0.xml generate the below - which is
>> problematic.
>>>> Perhaps instead of using .split on line 37 you can slice instead
>>>> (which assumes only .xml files (which need to be tested for as there
>>>> are also .ent files in that directory that should not be included))
>>>> 
>>>> [ACS_DOCS.added-API-commands-4]
>>>> file_filter = <lang>/added-API-commands-4.po source_file =
>>>> pot/added-API-commands-4.pot source_lang = en type = PO
>>>> 
>>>> 
>>>> - David Nalley
>>>> 
>>>> 
>>>> On Dec. 2, 2012, 1:29 a.m., Kelcey Jamison-Damage wrote:
>>>>> 
>>>>> -----------------------------------------------------------
>>>>> This is an automatically generated e-mail. To reply, visit:
>>>>> https://reviews.apache.org/r/8307/
>>>>> -----------------------------------------------------------
>>>>> 
>>>>> (Updated Dec. 2, 2012, 1:29 a.m.)
>>>>> 
>>>>> 
>>>>> Review request for cloudstack.
>>>>> 
>>>>> 
>>>>> Description
>>>>> -------
>>>>> 
>>>>> Utility for generating the .tx/config file
>>>>> 
>>>>> Long names are truncated and suffixed with -long, and duplicates are
>> numerically tagged.
>>>>> 
>>>>> 
>>>>> Diffs
>>>>> -----
>>>>> 
>>>>> docs/doc-config-gen.py PRE-CREATION
>>>>> 
>>>>> Diff: https://reviews.apache.org/r/8307/diff/
>>>>> 
>>>>> 
>>>>> Testing
>>>>> -------
>>>>> 
>>>>> tested against dupes and long names in en-US. uses relative paths so
>>>>> must be executed from /docs
>>>>> 
>>>>> 
>>>>> Thanks,
>>>>> 
>>>>> Kelcey Jamison-Damage
>>>>> 
>>>>> 
>>>> 
>>> 


Mime
View raw message