On Mon, 11 Oct 2010 14:36:49 -0400
John Dennis <jden...@redhat.com> wrote:

> On 10/11/2010 02:18 PM, John Dennis wrote:
> > On 10/11/2010 01:01 PM, Simo Sorce wrote:
> >> On Fri, 08 Oct 2010 10:26:18 -0400
> >> Rob Crittenden<rcrit...@redhat.com>   wrote:
> >>
> >>> Simo Sorce wrote:
> >>>>
> >>>>
> >>>> This is some very basic initial localization work for the C
> >>>> tools. I do not have any translation yet, and creation and
> >>>> merging of .po and binary files is not yet done. But the
> >>>> clients.pot file is regularly updated when make is run in the
> >>>> main dir (or make gettext in the ipa-clients dir).
> >>>>
> >>>> Fixes trac#186
> >>>>
> >>>> Simo.
> >>>
> >>> Nack. As discussed in IRC we are going to use a single po file for
> >>> all translations.
> >>
> >> Ok, here a revised patch that uses the existing intall/po
> >> infrastructure and generates a single .po file
> >>
> >> I took the liberty of converting the Makefile in there to
> >> automatically source .py, c and .h files, and also removed
> >> install/po/Makefile as the Makefile.in is all we need in git I
> >> think.
> >
> > NAK
> >
> > There are several things I'd like to see you address:
> >
> > 1) Please keep the independent list of python and c files potfiles
> > as make variables. This is preferred because:
> >
> > a) It allows the use of these as explicit make target and
> > dependencies.
> >
> > b) It permits knowing exactly what these files are and could be
> > dumped out via a trivial make target for debugging and information
> > purposes.
> >
> > c) It avoids bizarre magic. By letting xgettext find some files and
> > have others explicitly listed you've buried inside some cryptic
> > shell commands both implicit and explicit file lists, the implicit
> > file list being invisible. That's just very hard to maintain and
> > understand.
> 
> Oh and I forgot to add:
> 
> By demanding some files be manually added to the Makefile.in and
> having others be automatically picked up you've created developer
> confusion. When do I have to manually add a file? Why does this work
> sometimes and not others? But it worked when I added foo.py but not
> when I added the command do_foo, I don't get it.

It is explained in the NOTE I added to the README file ...

> It should be one way or the other for sanity and maintenance sake. 
> Either it's manual or it's automatic, not a mix of the two.

A mix look ok, the rule is simple:
- for files the end in .py do nothing
- any other python file add to list

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to