You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Animesh Chaturvedi <an...@citrix.com> on 2013/02/12 07:49:01 UTC

RE: Review Request: Utility for generating the .tx/config file

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 <ru...@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" <da...@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
> >>>
> >>>
> >>
> >

Re: Review Request: Utility for generating the .tx/config file

Posted by Sebastien Goasguen <ru...@gmail.com>.
I added a review.

This can be discarded and closed.

-sebastien

On Feb 12, 2013, at 7:49 AM, Animesh Chaturvedi <an...@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 <ru...@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" <da...@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
>>>>> 
>>>>> 
>>>> 
>>>