Jim Meyering j...@meyering.net writes:
However, I'm a little reluctant to change back.
Let's wait a day or two, in case Paul Eggert has an objection.
Here are some objections to the change, under the assumption that
we're in a poorly-configured environment (as the behavior is
unaffected in
Pádraig Brady wrote:
Jim Meyering wrote:
The only advantage is that my patch uses the existing framework,
rather than adding special case code in ls.c proper.
Whether that is worth the apparent complexity...
If you prefer his patch and want to adjust it and
handle the rest, I have no
Pádraig Brady wrote:
Pádraig Brady wrote:
Jim Meyering wrote:
The only advantage is that my patch uses the existing framework,
rather than adding special case code in ls.c proper.
Whether that is worth the apparent complexity...
If you prefer his patch and want to adjust it and
handle the
Pádraig Brady wrote:
The full patch is attached.
...
That looks fine. Thanks.
However, I'm a little reluctant to change back.
Let's wait a day or two, in case Paul Eggert has an objection.
Would you please add a test to exercise this,
perhaps based on the one from Ondřej?
From
Jim Meyering wrote:
Pádraig Brady wrote:
Paul Eggert wrote:
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and
Pádraig Brady wrote:
...
So that will apply generate an en.po with the
traditional unix format to apply to en_* for e.g.
Right.
$ locale -a | sed -n 's/\(en_..\).*/\1/p' | sort -u |
while read LANG; do echo $LANG $(locale territory); done
en_AG Antigua and Barbuda
en_AU Australia
en_BW
Jim Meyering wrote:
The only advantage is that my patch uses the existing framework,
rather than adding special case code in ls.c proper.
Whether that is worth the apparent complexity...
If you prefer his patch and want to adjust it and
handle the rest, I have no objection.
Yes it's
Pádraig Brady wrote:
Paul Eggert wrote:
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and the
patch do not
Paul Eggert wrote:
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and the
patch do not seem correct. The
Jim Meyering wrote:
Here's an incomplete patch.
It needs a test and a NEWS entry.
Ondřej, can you adjust your test to work (or skip)
if there is no en* locale?
Maybe something like that (attachment)?
It's checking for existence of locale binary and en_US locale and
performing the test only if
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and the
patch do not seem correct. The diagnosis says:
In ls.c
Paul Eggert wrote:
Ondřej Vašík ova...@redhat.com writes:
as reported in https://bugzilla.redhat.com/show_bug.cgi?id=525134 by
Daniel Qarras, ls -l shows iso long format for en_* locales.
I just now read that Bugzilla report, and the diagnosis and the
patch do not seem correct. The
12 matches
Mail list logo
