Fujiwara - we are going to RC1 next Monday and I would really like to 
bring this to a resolution or I will not get l10n support in for PM and 
UM for 2008.11, nor will I get the doc support landed.

Are you saying that we must use the GNU gettext support to get the 
translation support needed for globals via N_ and the intl tool?
So if we are using it in this instance you are saying we should use it 
consistently across all of the code base. The other issue being that if 
we do need to swap domains we can with this api and not the other class 
based one.

Its one argument, but I would favor simplicity every time over a 
possible future requirement.

When you say "install() has a problem to disable the compatibility of C 
gettext()." Why is this a problem for us? When would we want to disable 
it? Can you give me a concrete example in the code please, I do not 
understand what you are saying.

I would vote along with Danek for using gettext.install and keeping 
things simple in the code. If at some stage down the line we do need to 
access the domain we can use the GNU gettext API, but until we need to 
we should stick with gettext.install().

Please take the webrev I created with your various mods and modify it so 
it only uses gettext.install(). If this is not possible please explain 
in detail why.

http://cr.opensolaris.org/~jmr/pm_4126_v4_Oct30/

If you have a specific instance such as the use of globals that needs 
the GNU gettext API, please add in support for this only with a comment 
explaining why this is needed in this instance. If the code can be 
reworked to remove this dependency on the globals all the better.

If this approach is not possible as when using gettext.install "Other 
things break", please give us a detailed breakdown of what will break 
and why, so we can better understand what you are trying to achieve and 
work around.

Thanks,

JR


Danek Duvall wrote:
> On Fri, Oct 31, 2008 at 03:54:52AM +0900, Takao Fujiwara - Tokyo S/W Center 
> wrote:
>
>   
>> I'm not convinced you have a proper reason to use gettext.install. Other 
>> things break - they would be just easy bugs.
>>     
>
> What are those things?
>
>   
>> gettext.install is not a class based too.
>>     
>
> Yes it is. 
>
> Danek
>   

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to