On Sat, Oct 20, 2018 at 8:22 PM Bernhard Voelker <[email protected]> wrote: > On 10/20/18 11:32 AM, Jim Meyering wrote: > > If you adjust your function to be: > > > > num_sort() { tr -s ' ' '\n' |sort -n; } > > > > then you can make less invasive changes like this: > > > > test "$(chroot --userspec=$NON_ROOT_USERNAME / id -G|num_sort)" = \ > > "$(id -G $NON_ROOT_USERNAME|num_sort)" || fail=1 > > Fair enough. My intention was to catch more errors (while they are > swallowed by the pipe) vs. yours is the smaller change, indeed. > New version attached, pushing soon.
Thank you. That looks fine.
