tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: xgettext with UI files support



Le 15/06/15 17:08, Jonathan Perkin a écrit :
> * On 2015-06-15 at 16:00 BST, Ryo ONODERA wrote:
> 
>> My commit to x11/gtk3 was wrong.
>> How about the following patches?
>> If REQUIRE_XGETTEXT_WITH_UI_FILES_SUPPORT=yes, xgettext>=0.18.3 from
>> devel/gettext-tools is used.
> 
> I really don't like that variable name, it's way too long.
> 
> I'd also prefer to see a more general way of requiring specific
> versions of USE_TOOLS (e.g. USE_TOOLS+= xgettext>=0.18.3 or something)
> rather than adding special cases like this all over the place,
> especially during a freeze.
> 
> Can we go with a simpler, non-infrastructure fix for now and discuss a
> general purpose solution post-freeze?
> 
> Thanks,
> 

I believe (as I indicated offline to Ryo) that the following syntax is already supported:
> TOOLS_DEPENDS.xgettext=	gettext-tools>=0.18.3:../../devel/gettext-tools

This seems a reasonably usable [and existing] syntax.

The default is currently:
> richard@omnis:/home/richard/src/pkgsrc/x11/gtk3$ bmake show-var VARNAME=TOOLS_DEPENDS.xgettext
> gettext-tools>=0.15:../../devel/gettext-tools

because the following is set in mk/tools/gettext.mk:
> _TOOLS_DEP.gettext-tools=	gettext-tools>=0.15

As Patrick indicated, the problem does seem less the version involved than the build options for gettext-tools.

I believe Ryo's problem remains how to force using pkgsrc version... perhaps the best thing is to simply
set "TOOLS_PLATFORM.xgettext=  #" in this case in mk.conf (not to mention msgfmt & msgmerge)...

-- 
Richard PALO



Home | Main Index | Thread Index | Old Index