GNU bug report logs - #24121
new snapshot available: diffutils-3.3.52-e974

Previous Next

Package: diffutils;

Reported by: Jim Meyering <jim <at> meyering.net>

Date: Mon, 1 Aug 2016 01:17:01 UTC

Severity: normal

Tags: notabug

Done: Jim Meyering <jim <at> meyering.net>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 24121 in the body.
You can then email your comments to 24121 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 01:17:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Jim Meyering <jim <at> meyering.net>:
New bug report received and forwarded. Copy sent to bug-diffutils <at> gnu.org. (Mon, 01 Aug 2016 01:17:01 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: bug-diffutils <at> gnu.org
Cc: TP coordinator <coordinator <at> translationproject.org>,
 platform-testers <at> gnu.org
Subject: new snapshot available: diffutils-3.3.52-e974
Date: Sun, 31 Jul 2016 18:15:36 -0700
Here's a new snapshot:
  - fix a portability bug in the "colors" tests
  - in that same test, work around an infelicity in shells like dash
  - update to more recent gnulib
---------------------
diffutils snapshot:
  http://meyering.net/diff/diffutils-ss.tar.xz      1.3 MB
  http://meyering.net/diff/diffutils-ss.tar.xz.sig
  http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz

Changes in diffutils since 3.3.50-0353:

Jim Meyering (2):
      tests: colors: fix a portability problem and work around a shell bug
      maint: remove gl/lib/reg*.c.diff; no longer needed

Changes in gnulib since 3.3.50-0353:

* gnulib 9c780eb...348402f (114):
  > autoupdate
  > update from texinfo
  > autoupdate
  > update from texinfo
  > obstack: pacify GCC 6 with -Wnull-dereference
  > update from texinfo
  > doc: mention glibc, OS X, Cygwin [S]SIZE_MAX buglet
  > doc: mention glibc SSIZE_MAX buglet
  > printf-posix: Fix mingw build
  > mktime: call tzset as per POSIX
  > fts: handle readdir() errors
  > intprops: port better to GCC 7
  > intprops-test: port to GCC 6
  > autoupdate
  > update from texinfo
  > autoupdate
  > xalloc-oversized: port to GCC 7; fewer warnings
  > autoupdate (gettext-0.19.8.1)
  > update from texinfo
  > regex: port to Sun C
  > update from texinfo
  > stdbool: Restore __bool_true_false_are_defined check
  > stdbool: Port AC_CHECK_HEADER_STDBOOL to C++11
  > Use GCC_LINT, not lint
  > secure_getenv: Port to many more platforms.
  > update from texinfo
  > canonicalize: Fix broken probe for realpath.
  > unsetenv: relax to LGPLv2+
  > update from texinfo
  > gendocs.sh: Set default TOP_NODE_UP_URL in HTML output.
  > manywarnings: update for GCC 6.1
  > autoupdate
  > intdiv0, memmem, nocrash, strcasestr, strstr: no exit
  > glob: size_t overflow checks
  > glob: don't assume INT_MAX < SIZE_MAX
  > Fix undefined behaviour in gettext.h.
  > autoupdate
  > git-version-gen: avoid undefined shift
  > update from texinfo
  > glob: merge glibc changes into lib/glob.c
  > mktime: spelling fix in comment
  > mktime: port to stricter signed overflow checking
  > mktime: speed up DEBUG_MKTIME benchmarks
  > mktime: resurrect DEBUG_MKTIME testing
  > mktime: simplify DEBUG_MKTIME
  > Port mktime_internal offset to unsigned time_t
  > xstrtol: prohibit monstrosities like "1bB"
  > xstrtod: reinstate setting of *result upon ERANGE
  > xstrtod: modify *result only if no errors
  > btowc: document problems in C locale
  > autoupdate
  > update from texinfo
  > mktime: improve integer overflow checking
  > intprops: check two's complement assumption
  > intprops, mktime, strtol: assume two's complement
  > stdint: port to strict C11 left shift
  > mbrtowc: work around glibc bug#19932
  > update from texinfo
  > autoupdate
  > stdint: detect good enough pre-C++11 stdint.h in C++ mode
  > argp: merge changes from glibc
  > Prefer American spelling for "initialize"
  > autoupdate
  > stddef: support configuring with g++
  > autoupdate
  > autoupdate
  > update from texinfo
  > test-framework-sh: minor cleanups
  > test-framework-sh: revert port to NetBSD 7.0
  > autoupdate
  > Port better to Alpine Linux
  > test-framework-sh: port to NetBSD 7.0
  > update from texinfo
  > gitlog-to-changelog: suppress ignored chatter
  > update from texinfo
  > update from texinfo
  > setlocale: add "sv" to Windows language table
  > update from texinfo
  > sys_select: port to new Cygwin
  > test-userspec.c: do not trigger gcc's new -Wmisleading-indentation
  > time_rz: port to clang -Wunused-const-variable
  > std-gnu11: improve clang support
  > select: port more to Intel 2016.1.150 compiler
  > select: try to port to 2016.1.150 compiler
  > localename-tests: memory allocation fixes
  > intprops: make .h file license match module
  > acl: fix missing return on Cygwin
  > update from texinfo
  > update from texinfo
  > extern-inline: port to PGI CC
  > update from texinfo
  > update from texinfo
  > signbit: port back to pre-C++11 GCC
  > mountlist: recognize autofs-mounted remote file systems, too
  > signbit: port to C++ with GCC 6
  > regex: make it closer to libc
  > regex: merge patches from libc
  > update from texinfo
  > update from texinfo
  > autoupdate
  > autoupdate
  > stdalign: port to older HP and IBM cc
  > stdalign: port to clang 3.7.0
  > update from texinfo
  > readdir_r: now obsolescent
  > Use modern texinfo when syncing install.texi output from autoconf
  > update from texinfo
  > sync install.texi from autoconf
  > misc: port better to gcc -fsanitize=address
  > update from texinfo
  > autoupdate
  > verify-tests: also remove stray test-verify.Tpo
  > std-gnu11: new module
  > update from texinfo




Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 03:40:02 GMT) Full text and rfc822 format available.

Message #8 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Assaf Gordon <assafgordon <at> gmail.com>
To: Jim Meyering <jim <at> meyering.net>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Sun, 31 Jul 2016 23:38:51 -0400
[Message part 1 (text/plain, inline)]
Hello,

> On Jul 31, 2016, at 21:15, Jim Meyering <jim <at> meyering.net> wrote:
> 
> Here's a new snapshot:
> [...]
> http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
> 
> Changes in diffutils since 3.3.50-0353:
> 
> Jim Meyering (2):
>      tests: colors: fix a portability problem and work around a shell bug
>      maint: remove gl/lib/reg*.c.diff; no longer needed

With this update, no more failures on the tested gnu/linux system, but few failures on others (BSDs, Mac OS X, AIX, OpenSolaris).

For BSD/Mac OS X/AIX, it seems the 'color' test uses 'touch --date' which is a gnu extension and is not available.
Perhaps that leads to other failures.

For OpenSolaris 5.10 (both i86pc and sparc) 'make check' does not complete (perhaps an old shell syntax issue).

For GNU Hurd 0.7, the test 'brief-vs-stat-zero-kernel-lies' fails.

logs attached.

regards,
 - assaf


[make-check.sunos510.log.xz (application/octet-stream, attachment)]
[test-suite.AIX.log.xz (application/octet-stream, attachment)]
[test-suite.freebsd103.log.xz (application/octet-stream, attachment)]
[test-suite.gnu-hurd07.log.xz (application/octet-stream, attachment)]
[test-suite.macosx.log.xz (application/octet-stream, attachment)]
[test-suite.openbsd58.log.xz (application/octet-stream, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 04:36:02 GMT) Full text and rfc822 format available.

Message #11 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Sun, 31 Jul 2016 21:35:12 -0700
[Message part 1 (text/plain, inline)]
On Sun, Jul 31, 2016 at 8:38 PM, Assaf Gordon <assafgordon <at> gmail.com> wrote:
> Hello,
>
>> On Jul 31, 2016, at 21:15, Jim Meyering <jim <at> meyering.net> wrote:
>>
>> Here's a new snapshot:
>> [...]
>> http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
>>
>> Changes in diffutils since 3.3.50-0353:
>>
>> Jim Meyering (2):
>>      tests: colors: fix a portability problem and work around a shell bug
>>      maint: remove gl/lib/reg*.c.diff; no longer needed
>
> With this update, no more failures on the tested gnu/linux system, but few failures on others (BSDs, Mac OS X, AIX, OpenSolaris).
>
> For BSD/Mac OS X/AIX, it seems the 'color' test uses 'touch --date' which is a gnu extension and is not available.
> Perhaps that leads to other failures.
>
> For OpenSolaris 5.10 (both i86pc and sparc) 'make check' does not complete (perhaps an old shell syntax issue).
>
> For GNU Hurd 0.7, the test 'brief-vs-stat-zero-kernel-lies' fails.

Great! Thanks again for all of that quick feedback.
So far I've only looked at the solaris 5.10 build failure, and wrote
the attached that I hope will fix it. It upgrades to the
infrastructure (forgot I hadn't yet done it for diffutils) that is
used already to good effect in at least grep, sed, coreutils. Would
you please let me know if this fixes it?
[test-improve-test-infrastructure.diff (text/plain, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 09:18:01 GMT) Full text and rfc822 format available.

Message #14 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Dagobert Michelsen <dam <at> opencsw.org>
To: bug-diffutils <at> gnu.org
Cc: platform-testers <at> gnu.org
Subject: Re: [platform-testers] new snapshot available: diffutils-3.3.52-e974
Date: Mon, 1 Aug 2016 11:16:59 +0200
[Message part 1 (text/plain, inline)]
Hi Jim,

> Am 01.08.2016 um 03:15 schrieb Jim Meyering <jim <at> meyering.net>:
> 
> Here's a new snapshot:
>  - fix a portability bug in the "colors" tests
>  - in that same test, work around an infelicity in shells like dash
>  - update to more recent gnulib
> ---------------------
> diffutils snapshot:
>  http://meyering.net/diff/diffutils-ss.tar.xz      1.3 MB
>  http://meyering.net/diff/diffutils-ss.tar.xz.sig
>  http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz

Just tested on Solaris 10 Sparc, there is a missing „;" in Makefile.am:

> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 5457d3a..4538c7b 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -46,7 +46,7 @@ TESTS_ENVIRONMENT =                                                   \
>    PREFERABLY_POSIX_SHELL='$(PREFERABLY_POSIX_SHELL)';                  \
>      export PREFERABLY_POSIX_SHELL;                                     \
>    REPLACE_GETCWD=$(REPLACE_GETCWD); export REPLACE_GETCWD;             \
> -  PATH='$(abs_top_builddir)/src$(PATH_SEPARATOR)'"$$PATH"; export PATH \
> +  PATH='$(abs_top_builddir)/src$(PATH_SEPARATOR)'"$$PATH"; export PATH; \
>    stderr_fileno_=9; export stderr_fileno                               \
>    ; 9>&2

On Solaris 9 Sparc the substitute for „isblank“ is missing:

>   CCLD     test-c-ctype
> Undefined                       first referenced
>  symbol                             in file
> isblank                             test-c-ctype.o

It should be pulled in from Gnulib:
  https://www.gnu.org/software/gnulib/manual/html_node/isblank.html

On Solaris 11 Sparc I get this compilation error:

>   CC       rawmemchr.o
> rawmemchr.c: In function 'rawmemchr':
> rawmemchr.c:49:18: error: cast increases required alignment of target type [-Werror=cast-align]
>    longword_ptr = (const longword *) char_ptr;
>                   ^
> cc1: all warnings being treated as errors


Apart from that all tests pass on Solaris 10/11, all Sparc and x86, all 32 and 64 bit.


Best regards

  — Dago

--
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896

[signature.asc (application/pgp-signature, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 15:19:01 GMT) Full text and rfc822 format available.

Message #17 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: platform-testers <at> gnu.org, 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: [platform-testers] new snapshot
 available: diffutils-3.3.52-e974
Date: Mon, 1 Aug 2016 08:18:33 -0700
[Message part 1 (text/plain, inline)]
On Mon, Aug 1, 2016 at 2:16 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
> Hi Jim,

Hi Dagobert,

Thank you for the quick testing!

>> Am 01.08.2016 um 03:15 schrieb Jim Meyering <jim <at> meyering.net>:
>>
>> Here's a new snapshot:
...
>>  http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
>
> Just tested on Solaris 10 Sparc, there is a missing „;" in Makefile.am:
>
>> diff --git a/tests/Makefile.am b/tests/Makefile.am
>> index 5457d3a..4538c7b 100644
>> --- a/tests/Makefile.am
>> +++ b/tests/Makefile.am
>> @@ -46,7 +46,7 @@ TESTS_ENVIRONMENT =                                                   \
>>    PREFERABLY_POSIX_SHELL='$(PREFERABLY_POSIX_SHELL)';                  \
>>      export PREFERABLY_POSIX_SHELL;                                     \
>>    REPLACE_GETCWD=$(REPLACE_GETCWD); export REPLACE_GETCWD;             \
>> -  PATH='$(abs_top_builddir)/src$(PATH_SEPARATOR)'"$$PATH"; export PATH \
>> +  PATH='$(abs_top_builddir)/src$(PATH_SEPARATOR)'"$$PATH"; export PATH; \
>>    stderr_fileno_=9; export stderr_fileno                               \
>>    ; 9>&2

I believe this is addressed by the patch I posted last night:
  https://debbugs.gnu.org/cgi/bugreport.cgi?bug=24121#11

> On Solaris 9 Sparc the substitute for „isblank“ is missing:
>
>>   CCLD     test-c-ctype
>> Undefined                       first referenced
>>  symbol                             in file
>> isblank                             test-c-ctype.o
>
> It should be pulled in from Gnulib:
>   https://www.gnu.org/software/gnulib/manual/html_node/isblank.html
>
> On Solaris 11 Sparc I get this compilation error:
>
>>   CC       rawmemchr.o
>> rawmemchr.c: In function 'rawmemchr':
>> rawmemchr.c:49:18: error: cast increases required alignment of target type [-Werror=cast-align]
>>    longword_ptr = (const longword *) char_ptr;
>>                   ^
>> cc1: all warnings being treated as errors

Thanks!
This should fix it:
[0001-build-Solaris-9-avoid-link-failure-due-to-isblank-us.diff (text/plain, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 16:26:02 GMT) Full text and rfc822 format available.

Message #20 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Mon, 1 Aug 2016 09:25:06 -0700
[Message part 1 (text/plain, inline)]
On Sun, Jul 31, 2016 at 8:38 PM, Assaf Gordon <assafgordon <at> gmail.com> wrote:
> Hello,
>
>> On Jul 31, 2016, at 21:15, Jim Meyering <jim <at> meyering.net> wrote:
>>
>> Here's a new snapshot:
>> [...]
>> http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
>>
>> Changes in diffutils since 3.3.50-0353:
>>
>> Jim Meyering (2):
>>      tests: colors: fix a portability problem and work around a shell bug
>>      maint: remove gl/lib/reg*.c.diff; no longer needed
>
> With this update, no more failures on the tested gnu/linux system, but few failures on others (BSDs, Mac OS X, AIX, OpenSolaris).
>
> For BSD/Mac OS X/AIX, it seems the 'color' test uses 'touch --date' which is a gnu extension and is not available.
> Perhaps that leads to other failures.

Here's a patch for that GNU-ism:
[0001-tests-colors-fix-portability-problem-with-touch-date.diff (text/plain, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 16:29:02 GMT) Full text and rfc822 format available.

Message #23 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Mon, 1 Aug 2016 09:27:36 -0700
On Sun, Jul 31, 2016 at 9:35 PM, Jim Meyering <jim <at> meyering.net> wrote:
> On Sun, Jul 31, 2016 at 8:38 PM, Assaf Gordon <assafgordon <at> gmail.com> wrote:
>> Hello,
>>
>>> On Jul 31, 2016, at 21:15, Jim Meyering <jim <at> meyering.net> wrote:
>>>
>>> Here's a new snapshot:
>>> [...]
>>> http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
>>>
>>> Changes in diffutils since 3.3.50-0353:
>>>
>>> Jim Meyering (2):
>>>      tests: colors: fix a portability problem and work around a shell bug
>>>      maint: remove gl/lib/reg*.c.diff; no longer needed
>>
>> With this update, no more failures on the tested gnu/linux system, but few failures on others (BSDs, Mac OS X, AIX, OpenSolaris).
>>
>> For BSD/Mac OS X/AIX, it seems the 'color' test uses 'touch --date' which is a gnu extension and is not available.
>> Perhaps that leads to other failures.
>>
>> For OpenSolaris 5.10 (both i86pc and sparc) 'make check' does not complete (perhaps an old shell syntax issue).
>>
>> For GNU Hurd 0.7, the test 'brief-vs-stat-zero-kernel-lies' fails.
>
> Great! Thanks again for all of that quick feedback.
> So far I've only looked at the solaris 5.10 build failure, and wrote
> the attached that I hope will fix it. It upgrades to the
> infrastructure (forgot I hadn't yet done it for diffutils) that is
> used already to good effect in at least grep, sed, coreutils. Would
> you please let me know if this fixes it?

I've gone ahead and pushed that with a tweak to make envvars check one
more variable name: GREP_OPTIONS.




Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Mon, 01 Aug 2016 16:46:02 GMT) Full text and rfc822 format available.

Message #26 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Mon, 1 Aug 2016 09:44:49 -0700
[Message part 1 (text/plain, inline)]
On Mon, Aug 1, 2016 at 9:27 AM, Jim Meyering <jim <at> meyering.net> wrote:
> On Sun, Jul 31, 2016 at 9:35 PM, Jim Meyering <jim <at> meyering.net> wrote:
>> On Sun, Jul 31, 2016 at 8:38 PM, Assaf Gordon <assafgordon <at> gmail.com> wrote:
>>> Hello,
>>>
>>>> On Jul 31, 2016, at 21:15, Jim Meyering <jim <at> meyering.net> wrote:
>>>>
>>>> Here's a new snapshot:
>>>> [...]
>>>> http://meyering.net/diff/diffutils-3.3.52-e974.tar.xz
>>>>
>>>> Changes in diffutils since 3.3.50-0353:
>>>>
>>>> Jim Meyering (2):
>>>>      tests: colors: fix a portability problem and work around a shell bug
>>>>      maint: remove gl/lib/reg*.c.diff; no longer needed
>>>
>>> With this update, no more failures on the tested gnu/linux system, but few failures on others (BSDs, Mac OS X, AIX, OpenSolaris).
>>>
>>> For BSD/Mac OS X/AIX, it seems the 'color' test uses 'touch --date' which is a gnu extension and is not available.
>>> Perhaps that leads to other failures.
>>>
>>> For OpenSolaris 5.10 (both i86pc and sparc) 'make check' does not complete (perhaps an old shell syntax issue).
>>>
>>> For GNU Hurd 0.7, the test 'brief-vs-stat-zero-kernel-lies' fails.
>>
>> Great! Thanks again for all of that quick feedback.
>> So far I've only looked at the solaris 5.10 build failure, and wrote
>> the attached that I hope will fix it. It upgrades to the
>> infrastructure (forgot I hadn't yet done it for diffutils) that is
>> used already to good effect in at least grep, sed, coreutils. Would
>> you please let me know if this fixes it?
>
> I've gone ahead and pushed that with a tweak to make envvars check one
> more variable name: GREP_OPTIONS.

Thanks again for all the testing, Assaf.
Would you please see what the hurd test expected?
If you rerun it with this patch, the usual testsuite.log file will
include details:
[0001-k.diff (text/plain, attachment)]

Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Tue, 02 Aug 2016 05:19:02 GMT) Full text and rfc822 format available.

Message #29 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Assaf Gordon <assafgordon <at> gmail.com>
To: Jim Meyering <jim <at> meyering.net>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Tue, 2 Aug 2016 01:18:52 -0400
Hello,

> On Aug 1, 2016, at 12:44, Jim Meyering <jim <at> meyering.net> wrote:
> Would you please see what the hurd test expected?
> If you rerun it with this patch, the usual testsuite.log file will
> include details:
> <0001-k.diff>

Digging a bit deeper, it seems that GNU Hurd does not have a functional /proc/self link
(at least not Debian/Hurd 0.7).

  $ uname -a
  GNU hurd07 0.7 GNU-Mach 1.6+git20160114-486/Hurd-0.7 i686-AT386 GNU

  $ ls -l /proc/self
  lrwxrwxrwx 0 root root 1 Dec 31  1969 /proc/self -> 1

  $ od -ta /proc/self/cmdline
  0000000   i   n   i   t  sp   [   2   ] nul
  0000011

And so this test will never succeed as-is.

A relevant mention of /proc/self shortcomings is here:
 https://www.gnu.org/software/hurd/hurd/translator/procfs/jkoenig/discussion.html#index2h1

Hope this helps,
 - assaf







Information forwarded to bug-diffutils <at> gnu.org:
bug#24121; Package diffutils. (Tue, 02 Aug 2016 16:47:01 GMT) Full text and rfc822 format available.

Message #32 received at 24121 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 24121 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24121: new snapshot available:
 diffutils-3.3.52-e974
Date: Tue, 2 Aug 2016 09:46:24 -0700
[Message part 1 (text/plain, inline)]
On Mon, Aug 1, 2016 at 10:18 PM, Assaf Gordon <assafgordon <at> gmail.com> wrote:
> Hello,
>
>> On Aug 1, 2016, at 12:44, Jim Meyering <jim <at> meyering.net> wrote:
>> Would you please see what the hurd test expected?
>> If you rerun it with this patch, the usual testsuite.log file will
>> include details:
>> <0001-k.diff>
>
> Digging a bit deeper, it seems that GNU Hurd does not have a functional /proc/self link
> (at least not Debian/Hurd 0.7).
>
>   $ uname -a
>   GNU hurd07 0.7 GNU-Mach 1.6+git20160114-486/Hurd-0.7 i686-AT386 GNU
>
>   $ ls -l /proc/self
>   lrwxrwxrwx 0 root root 1 Dec 31  1969 /proc/self -> 1
>
>   $ od -ta /proc/self/cmdline
>   0000000   i   n   i   t  sp   [   2   ] nul
>   0000011
>
> And so this test will never succeed as-is.
>
> A relevant mention of /proc/self shortcomings is here:
>  https://www.gnu.org/software/hurd/hurd/translator/procfs/jkoenig/discussion.html#index2h1

Thank you.
The attached patch should cause that test to be skipped when
/proc/self is not useful:
[0001-tests-skip-a-proc-self-dependent-test-on-the-Hurd.diff (text/plain, attachment)]

Added tag(s) notabug. Request was from Jim Meyering <jim <at> meyering.net> to control <at> debbugs.gnu.org. (Thu, 04 May 2017 17:01:02 GMT) Full text and rfc822 format available.

bug closed, send any further explanations to 24121 <at> debbugs.gnu.org and Jim Meyering <jim <at> meyering.net> Request was from Jim Meyering <jim <at> meyering.net> to control <at> debbugs.gnu.org. (Thu, 04 May 2017 17:01:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 02 Jun 2017 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 7 years and 118 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.