Re: [PATCH] po: fix POTFILES.in file ordering

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

 



On 10/04/16 14:03, Pavel Hrdina wrote:
> On Sat, Apr 09, 2016 at 08:10:55PM +0200, Erik Skultety wrote:
>> When it comes to a situation that a new translatable file needs to be added
>> into the list of files, an automatically generated patch is proposed during
>> syntax-check. However, the diff was made against a sorted list of files and
>> the same sorted list of files + the new file that actually needs to be added
>> into the list. Since we do not keep POTFILES sorted, the proposed patch thus
>> can't be applied...most of the time - depending on the context.
>>
>> Signed-off-by: Erik Skultety <eskultet@xxxxxxxxxx>
>> ---
>>  po/POTFILES.in | 68 +++++++++++++++++++++++++++++-----------------------------
>>  1 file changed, 34 insertions(+), 34 deletions(-)
> 
> This is sorted for different locales (my guess is en_US).  Libvirt uses LC_ALL=C
> (see maint.mk:145) and you should use this command:
> 
>     LC_ALL=C sort po/POTFILES.in -o po/POTFILES.in
> 
> Fix this please and send a new patch :)
> 
> Pavel
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list
> 

Oops..well, I pushed it before you replied. Anyway, I proposed a new
patch, this time, a correct one hopefully.

Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]