Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-06-20 Thread Philip Hands
Jmkr writes: > Sorry for the delay, I had to do some work and then forgot that I > planned to reply to this bug. No problem, these things happen -- anyway, your timing was great as I was just about to do something about this anyway. > > Philip Hands writes: > > ... > > > > Actually,

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-06-17 Thread Jmkr
Sorry for the delay, I had to do some work and then forgot that I planned to reply to this bug. > Philip Hands writes: > ... > > Actually, given that we don't care what's in the first field at that > point, and it's already matched the start of the country code, I think > this ought to work as

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-22 Thread Philip Hands
Jmkr writes: > I tested Phil's "sed" command in terminal like this (all one line - > email splits it): > > L=cs_CZ.UTF-8; sed -n > "/^${L%%_*}/"'{s/^[[:alpha:]]*;\([[:digit:]]\);.*/\1/p;q}' languagelist > > and it seemed to work - with "cs_CZ.UTF-8" locale it returned 2. But > some locales

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-21 Thread Jmkr
> Philip Hands writes: > ... > > BTW I would suggest using: > > ${LOCALE%%_*} in place of the $(echo ... | cut ...) Yes, I ended using that too in my version of the script, but I sent the bug report earlier:). Philip Hands writes: ... This is what it should have been:

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-19 Thread Philip Hands
Philip Hands writes: > > https://salsa.debian.org/philh/localechooser/-/commit/24206bbaccb9619302bab5ca603fb214841ebf58#63952ea2967253e426478ffea08ddc67c3bff9d8_98_92 I just noticed: that commit's line order was mangled while rebasing. This is what it should have been:

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-19 Thread Michael Tokarev
19.05.2022 17:23, Philip Hands wrote: Michael Tokarev writes: 19.05.2022 15:59, Philip Hands wrote: ... Actually, I'm unable to resist the urge to remove the redundant use of cat (that was already there), so how about doing it in a single sed: LEVEL=$(sed "/^${LOCALE%%_*}/"'{ print $4 ;

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-19 Thread Philip Hands
Michael Tokarev writes: > 19.05.2022 15:59, Philip Hands wrote: > ... >> Actually, I'm unable to resist the urge to remove the redundant use of >> cat (that was already there), so how about doing it in a single sed: >> >>LEVEL=$(sed "/^${LOCALE%%_*}/"'{ print $4 ; exit }' >>

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-19 Thread Michael Tokarev
19.05.2022 15:59, Philip Hands wrote: ... Actually, I'm unable to resist the urge to remove the redundant use of cat (that was already there), so how about doing it in a single sed: LEVEL=$(sed "/^${LOCALE%%_*}/"'{ print $4 ; exit }' /usr/share/localechooser/languagelist) That smells like

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-19 Thread Philip Hands
Jmkr writes: > Package: localechooser > Version: 2.94 > > Dear Maintainers, > > While working on my customized Debian installer (DI) I noticed that the > "./usr/lib/post-base-installer.d/05localechooser" file from the > "localechooser_2.93_*.udeb" package (at least for AMD64 and ARM64 >

Bug#1011254: The UDEB package "localechooser" contains probably some buggy source code in "05localechooser" file

2022-05-18 Thread Jmkr
Package: localechooser Version: 2.94 Dear Maintainers, While working on my customized Debian installer (DI) I noticed that the "./usr/lib/post-base-installer.d/05localechooser" file from the "localechooser_2.93_*.udeb" package (at least for AMD64 and ARM64 architectures) contains probably