Re: [platform-testers] new snapshot available: sed-4.5.48-58eb

[Assaf Gordon]

On 2018-12-13 2:11 p.m., Jim Meyering wrote:
> On Thu, Dec 13, 2018 at 12:18 PM Assaf Gordon <address@hidden> wrote:
> > On 2018-12-12 11:24 p.m., Jim Meyering wrote:
> > > sed snapshot:
> > >     https://meyering.net/sed/sed-4.5.48-58eb.tar.xz

> > I noticed one bug which I made in the latest 'obinary' test -
> >
> > The attached patch fixes it.
>
>
> What do you think of using tr -d '[:space:]' in place of sed there, in
> case some other implementation emits a TAB rather than spaces?
>
> Feel free to push the fix either way.

Thanks,
pushed here with your 'tr' suggestion:
https://git.savannah.gnu.org/cgit/sed.git/commit/?id=5155e56775469518d97ee119f5dc7f9db88f22b1


The original snapshot (4.5.48-58eb) passes all tests on:
  Debian 8.11 (i686)
  Debian 8.11 (mips64)
  Debian unstable (sparc64)
  Linux 3.10.0-693.11.6.el7.ppc64 (ppc64)
  Linux 3.10.0-693.11.6.el7.ppc64le (ppc64le)
  Oracle SunOS 5.11 (i86pc)
  Oracle SunOS 5.11 (sun4u)
  Raspbian 9.6 (armv7l)
  Ubuntu 14.04 (aarch64)

With the 'obinary' fix (4.5.49-5155e), it also passes on:
  FreeBSD 11.1-RELEASE-p7 (amd64)
  NetBSD 7.1 (amd64)
  OpenBSD 6.2 (amd64)
  Darwin 17.4.0 (x86_64)
  AIX 7.2


On Cygwin, the following fails:
   FAIL: testsuite/panic-tests
Due to being able to read a file from directory without reading
permissions (so the panic error isn't triggered).
I don't think that's a show stopper.


There were some minor (and already known) gnulib errors,
I'm ignoring them for now.


regards,
 - assaf