Re: [PATCH] busybox: tr: enable options required by POSIX

2020-07-14 Thread Jordan Geoghegan
On 2020-07-14 02:54, Bjørn Mork wrote: Jordan Geoghegan writes: 'tr' is a standard system utility, I wasn't expecting such pushback against making it behave as every other modern implementation does. OpenWrt exists because it is different from every other distribution. https://openwrt.org

Re: [PATCH] busybox: tr: enable options required by POSIX

2020-07-14 Thread Jordan Geoghegan
On 2020-07-14 02:08, Petr Štetiar wrote: Jordan Geoghegan [2020-07-13 23:06:30]: Also, the use of "tr 'a-z'..." is unsafe for exotic locales Buildsystem sets LC_ALL=C explicitly, so whats the issue here? I've reported that 'tr' behaves abnormally, that's about all I can do. I

Re: [PATCH] busybox: tr: enable options required by POSIX

2020-07-14 Thread Jordan Geoghegan
On 2020-07-13 22:56, Rosen Penev wrote: On Mon, Jul 13, 2020 at 10:44 PM Jordan Geoghegan wrote: On 2020-07-13 22:17, Rosen Penev wrote: On Mon, Jul 13, 2020 at 12:14 PM Jordan Geoghegan wrote: On 2020-07-13 08:36, Petr Štetiar wrote: Magnus Kroken [2020-07-13 15:49:30]: Hi

Re: [PATCH] busybox: tr: enable options required by POSIX

2020-07-13 Thread Jordan Geoghegan
On 2020-07-13 22:17, Rosen Penev wrote: On Mon, Jul 13, 2020 at 12:14 PM Jordan Geoghegan wrote: On 2020-07-13 08:36, Petr Štetiar wrote: Magnus Kroken [2020-07-13 15:49:30]: Hi, Support for character classes (e.g. [:upper:] and [:lower:]) and equivalence classes (e.g

Re: [PATCH] busybox: tr: enable options required by POSIX

2020-07-13 Thread Jordan Geoghegan
On 2020-07-13 08:36, Petr Štetiar wrote: Magnus Kroken [2020-07-13 15:49:30]: Hi, Support for character classes (e.g. [:upper:] and [:lower:]) and equivalence classes (e.g. [=a=]) in the tr utility are required by POSIX. This change increases package size by approx. 500 bytes. where does

Re: AW: 'tr' character class support?

2020-07-11 Thread Jordan Geoghegan
scripts, but I'd be happy to go through and start cleaning stuff up if there is interest. Attached please find a patch for "package/network/services/hostapd/files/hostapd.sh" Regards, Jordan Geoghegan --- hostapd.sh.orig Sat Jul 11 11:16:06 2020 +++ hostapd.sh Sat Jul 11 11:

Re: 'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
Woops, accidentally mangled whitespace in my last diff. Fix below On 2020-07-10 21:13, Jordan Geoghegan wrote: Please find patch to enable character classes in 'tr' below. On 2020-07-10 20:33, Rosen Penev wrote: On Fri, Jul 10, 2020 at 5:15 PM Jordan Geoghegan wrote: On 2020-07-10 16:59

Re: 'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
Please find patch to enable character classes in 'tr' below. On 2020-07-10 20:33, Rosen Penev wrote: On Fri, Jul 10, 2020 at 5:15 PM Jordan Geoghegan wrote: On 2020-07-10 16:59, Rosen Penev wrote: On Fri, Jul 10, 2020 at 4:17 PM Jordan Geoghegan wrote: On 2020-07-10 14:54, Rosen Penev

Re: 'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
On 2020-07-10 16:59, Rosen Penev wrote: On Fri, Jul 10, 2020 at 4:17 PM Jordan Geoghegan wrote: On 2020-07-10 14:54, Rosen Penev wrote: On Fri, Jul 10, 2020 at 2:29 PM Jordan Geoghegan wrote: On 2020-07-10 14:15, Magnus Kroken wrote: Hi Jordan On 10.07.2020 22:45, Jordan Geoghegan

Re: 'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
On 2020-07-10 14:54, Rosen Penev wrote: On Fri, Jul 10, 2020 at 2:29 PM Jordan Geoghegan wrote: On 2020-07-10 14:15, Magnus Kroken wrote: Hi Jordan On 10.07.2020 22:45, Jordan Geoghegan wrote: Hey folks, Does the 'tr' utility support character classes in OpenWRT? I was playing around

Re: 'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
On 2020-07-10 14:15, Magnus Kroken wrote: Hi Jordan On 10.07.2020 22:45, Jordan Geoghegan wrote: Hey folks, Does the 'tr' utility support character classes in OpenWRT? I was playing around with an OpenWRT x86_64 VM and I noticed that 'tr' doesn't seem to support character classes

'tr' character class support?

2020-07-10 Thread Jordan Geoghegan
Hey folks, Does the 'tr' utility support character classes in OpenWRT? I was playing around with an OpenWRT x86_64 VM and I noticed that 'tr' doesn't seem to support character classes. The command " echo HELLO | tr '[:upper:]' '[:lower:]' "  does not convert to the text to lowercase as it