[Libosinfo] [PATCH] configure: Use LINGUAS file

[Zeeshan Ali (Khattak)]

On Wed, Jan 2, 2013 at 6:53 PM, Daniel P. Berrange <berrange at redhat.com> wrote:
> On Wed, Jan 02, 2013 at 06:45:31PM +0200, Zeeshan Ali (Khattak) wrote:
>> From: "Zeeshan Ali (Khattak)" <zeeshanak at gnome.org>
>>
>> Setting ALL_LINGUAS in configure.in is obsolete according to po.m4 in
>> favor of LINGUAS file. This is the reason why all (as far as I can see)
>> GNOME modules use LINGUAS file.
>>
>> Also the rule in configure.ac creates broken Makefile if there is more
>> than one translation files:
>>
>> [zeenix at z-laptop libosinfo]$ make
>> Makefile:166: *** missing separator.  Stop.
>> [zeenix at z-laptop libosinfo]$ head -n 166 Makefile|tail -n 2
>> ALL_LINGUAS = pl
>> uk
>> ---
>>  configure.ac | 2 --
>>  po/LINGUAS   | 0
>>  2 files changed, 2 deletions(-)
>>  create mode 100644 po/LINGUAS
>>
>> diff --git a/configure.ac b/configure.ac
>> index ade0ee7..f4b1044 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -21,8 +21,6 @@ AC_DEFINE_UNQUOTED([GETTEXT_PACKAGE],["$GETTEXT_PACKAGE"],[The name of the gette
>>  AM_GNU_GETTEXT([external])
>>  AM_GNU_GETTEXT_VERSION([0.17])
>>
>> -ALL_LINGUAS=`cd "$srcdir/po" > /dev/null && ls *.po | sed 's+\.po$++'`
>> -
>>  # Autoconf 2.61a.99 and earlier don't support linking a file only
>>  # in VPATH builds.  But since GNUmakefile is for maintainer use
>>  # only, it does not matter if we skip the link with older autoconf.
>> diff --git a/po/LINGUAS b/po/LINGUAS
>> new file mode 100644
>> index 0000000..e69de29
>
> ACK
>
> Don't forget to actually populate LINGUAS when you pull in new xlations.

Yeah, already done for the 2 i have merged in local branch. :)

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124