GNU bug report logs - #24228
new snapshot available: diffutils-3.4.9-d647

Previous Next

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

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

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: platform-testers <at> gnu.org
Subject: new snapshot available: diffutils-3.4.9-d647
Date: Sun, 14 Aug 2016 22:33:30 -0700

There's really only one change in this delta: the diff3 workaround for
missing seq.
It looks like gnulib was updated, too solely because while it's
included in the preceding snapshot, I did not push that commit until
after the seq fix.

diffutils snapshot:
  http://meyering.net/diff/diffutils-ss.tar.xz      1.4 MB
  http://meyering.net/diff/diffutils-ss.tar.xz.sig
  http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz

Changes in diffutils since 3.4.8-c06c:

Jim Meyering (2):
      tests: diff3: work around missing seq on some systems
      gnulib: update to latest


Changes in gnulib since 3.4.8-c06c:

* gnulib e4f1a4a...1eb82ad (5):
  > update from texinfo
  > parse-datetime: add optional debug printing
  > update from texinfo
  > tests/init.sh: exclude dash with bad "local" semantics
  > update from texinfo

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: bug-diffutils <at> gnu.org
Subject: Re: [platform-testers] new snapshot available: diffutils-3.4.9-d647
Date: Mon, 15 Aug 2016 22:35:50 +0200

[Message part 1 (text/plain, inline)]

Hi Jim

> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
> 
> There's really only one change in this delta: the diff3 workaround for
> missing seq.
> It looks like gnulib was updated, too solely because while it's
> included in the preceding snapshot, I did not push that commit until
> after the seq fix.
> 
> diffutils snapshot:
>  http://meyering.net/diff/diffutils-ss.tar.xz      1.4 MB
>  http://meyering.net/diff/diffutils-ss.tar.xz.sig
>  http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz

I get this error on Solaris 10 Sparc with GCC 5.2, same for
Solaris 11 Sparc with GCC 5.2:

>   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
> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255

Compilation on Solaris 11 x86, 32 and 64 bit work fine with
Oracle Studio 12.4.

On Solaris 11 Sparc with Oracle Studio 12.4 it works for 32 bit,
for 64 bit I get an error in the testsuite:

> FAIL: test-timespec
> ===================
> 
> test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
> FAIL test-timespec (exit status: 262)

On Solaris 9 Sparc with GCC 4.6 I still get this, probably because the
change you proposed in not in the release tarball. I’d happily test if the
updated tarballs are bootstrapped:

>   CC       error.o
> error.c:112:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]

I would like to set up buildbot, to which repository? The one at Savannah?
Do you have a preferred branch?
  git://git.savannah.gnu.org/diffutils.git


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)]

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

From: Assaf Gordon <assafgordon <at> gmail.com>
To: Jim Meyering <jim <at> meyering.net>, 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: new snapshot available:
 diffutils-3.4.9-d647
Date: Mon, 15 Aug 2016 18:22:48 -0400

Hello,

On 08/15/2016 01:33 AM, Jim Meyering wrote:
> There's really only one change in this delta: the diff3 workaround for
> missing seq.
> [...]
> diffutils snapshot:
>   http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz

This tarball now passes on various systems, including the previously failed 'diff3' .


However,
only on AIX, gnulib's "test-init.sh" fails (perhaps because of the recent update? though the error seems related to 'diff -U3').
Gnulib's test do not fail on most other systems (except minix, netbsd which were discussed before and deemed as not "show stoppers").

===========================================================
GNU diffutils 3.4.9-d647: gnulib-tests/test-suite.log
===========================================================

# TOTAL: 167
# PASS:  140
# SKIP:  26
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

FAIL: test-init.sh
==================

++ initial_cwd_=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests
+++ testdir_prefix_
+++ printf gt
++ pfx_=gt
+++ mktempd_ /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests gt-test-init.sh.XXXX
+++ case $# in
+++ destdir_=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests
+++ template_=gt-test-init.sh.XXXX
+++ MAX_TRIES_=4
+++ case $destdir_ in
+++ destdir_slash_=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/
+++ case $template_ in
++++ unset TMPDIR
+++ d=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
+++ case $d in
+++ :
+++ test -d /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
++++ ls -dgo /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
+++ perms='drwx------    2        256 Aug 15 11:41 /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL'
+++ case $perms in
+++ :
+++ echo /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
+++ return
++ test_dir_=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
++ cd /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
++ gl_init_sh_nl_='
'
++ IFS='
'
++ for sig_ in 1 2 3 13 15
+++ expr 1 + 128
++ eval 'trap '\''Exit 129'\'' 1'
+++ trap 'Exit 129' 1
++ for sig_ in 1 2 3 13 15
+++ expr 2 + 128
++ eval 'trap '\''Exit 130'\'' 2'
+++ trap 'Exit 130' 2
++ for sig_ in 1 2 3 13 15
+++ expr 3 + 128
++ eval 'trap '\''Exit 131'\'' 3'
+++ trap 'Exit 131' 3
++ for sig_ in 1 2 3 13 15
+++ expr 13 + 128
++ eval 'trap '\''Exit 141'\'' 13'
+++ trap 'Exit 141' 13
++ for sig_ in 1 2 3 13 15
+++ expr 15 + 128
++ eval 'trap '\''Exit 143'\'' 15'
+++ trap 'Exit 143' 15
++ trap remove_tmp_ 0
+ path_prepend_ .
+ test 1 '!=' 0
+ path_dir_=.
+ case $path_dir_ in
+ abs_path_dir_=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/.
+ case $abs_path_dir_ in
+ PATH=/home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/.:/home/agn/bin:/usr/bin:/etc:/usr/sbin:/usr/ucb:/usr/bin/X11:/sbin:/usr/java5/jre/bin:/usr/java5/bin
+ create_exe_shims_ /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/.
+ case $EXEEXT in
+ return 0
+ shift
+ test 0 '!=' 0
+ export PATH
+ fail=0
+ test_compare
+ touch empty
+ echo xyz
+ compare /dev/null /dev/null
+ test -s out
+ case $- in
+ compare /dev/null empty
+ test -s out
+ case $- in
+ compare in in
+ test -s out
+ case $- in
+ compare /dev/null in
+ cat
+ compare exp out
+ compare_dev_null_ exp out
+ test 2 = 2
+ test xexp = x/dev/null
+ test xout = x/dev/null
+ return 2
+ case $? in
+ compare_ exp out
+ diff -U3 exp out
+ rm -f diff.out
+ true
+ case $- in
+ compare empty in
+ grep @ out
+ sed 's/	.*//;/^@@/d' out
+ mv k out
+ cat
+ compare exp out
+ compare_dev_null_ exp out
+ test 2 = 2
+ test xexp = x/dev/null
+ test xout = x/dev/null
+ return 2
+ case $? in
+ compare_ exp out
+ diff -U3 exp out
+ cat diff.out
--- exp	2016-08-15 11:41:43.000000000 -0700
+++ out	2016-08-15 11:41:43.000000000 -0700
@@ -1,3 +1,3 @@
--- empty
+++ in
- +xyz
+ + xyz
+ rm -f diff.out
+ false
+ fail=1
+ case $- in
+ Exit 1
+ set +e
+ exit 1
+ exit 1
+ remove_tmp_
+ __st=1
+ cleanup_
+ :
+ cd /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests
+ chmod -R u+rwx /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
+ rm -rf /home/agn/pretest.2016-08-15.CBYCef/diffutils-3.4.9-d647.glMPj0/diffutils-3.4.9-d647/gnulib-tests/gt-test-init.sh.DTCL
+ exit 1
FAIL test-init.sh (exit status: 1)

====================================================
GNU diffutils 3.4.9-d647: tests/test-suite.log
====================================================

# TOTAL: 18
# PASS:  17
# SKIP:  1
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0

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

From: Jim Meyering <jim <at> meyering.net>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Thu, 18 Aug 2016 21:18:06 -0700

On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
...
>>  http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz

Thank you for testing.

> I get this error on Solaris 10 Sparc with GCC 5.2, same for
> Solaris 11 Sparc with GCC 5.2:
>
>>   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
>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255

In this case, I think the best advice it not to with warnings enabled
(don't use configure's --enable-gcc-warnings) unless you're using a
sufficiently modern system, probably glibc-based.

> Compilation on Solaris 11 x86, 32 and 64 bit work fine with
> Oracle Studio 12.4.
>
> On Solaris 11 Sparc with Oracle Studio 12.4 it works for 32 bit,
> for 64 bit I get an error in the testsuite:
>
>> FAIL: test-timespec
>> ===================
>>
>> test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
>> FAIL test-timespec (exit status: 262)

This one is interesting.
It would be great if you could get in with a debugger to see what
exactly is going wrong.
It's probably not an issue for diffutils, as it barely uses timespec
at all and does not use timespec_sub.

> On Solaris 9 Sparc with GCC 4.6 I still get this, probably because the
> change you proposed in not in the release tarball. I’d happily test if the
> updated tarballs are bootstrapped:
>
>>   CC       error.o
>> error.c:112:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]

This appears to be exposing some unusually old code in error.c that
should instead be relying on gnulib's own strerror_r module.
We'll probably fix it properly in the long run, but simply not using
--enable-gcc-warnings will avoid it for you now.

> I would like to set up buildbot, to which repository? The one at Savannah?
> Do you have a preferred branch?
>   git://git.savannah.gnu.org/diffutils.git

A buildbot would be great!
That is indeed the right repository, and as for branch, please use "master".
Thanks!

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: Jim Meyering <jim <at> meyering.net>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Thu, 8 Sep 2016 14:02:13 +0200

[Message part 1 (text/plain, inline)]

Hi Jim,

Am 19.08.2016 um 06:18 schrieb Jim Meyering <jim <at> meyering.net>:
> On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
> ...
>>> http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz
> 
> Thank you for testing.
> 
>> I get this error on Solaris 10 Sparc with GCC 5.2, same for
>> Solaris 11 Sparc with GCC 5.2:
>> 
>>>  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
>>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255
> 
> In this case, I think the best advice it not to with warnings enabled
> (don't use configure's --enable-gcc-warnings) unless you're using a
> sufficiently modern system, probably glibc-based.

Ok.

>> Compilation on Solaris 11 x86, 32 and 64 bit work fine with
>> Oracle Studio 12.4.
>> 
>> On Solaris 11 Sparc with Oracle Studio 12.4 it works for 32 bit,
>> for 64 bit I get an error in the testsuite:
>> 
>>> FAIL: test-timespec
>>> ===================
>>> 
>>> test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
>>> FAIL test-timespec (exit status: 262)
> 
> This one is interesting.
> It would be great if you could get in with a debugger to see what
> exactly is going wrong.
> It's probably not an issue for diffutils, as it barely uses timespec
> at all and does not use timespec_sub.

I didn’t get into looking into that yet, sorry.

>> On Solaris 9 Sparc with GCC 4.6 I still get this, probably because the
>> change you proposed in not in the release tarball. I’d happily test if the
>> updated tarballs are bootstrapped:
>> 
>>>  CC       error.o
>>> error.c:112:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]
> 
> This appears to be exposing some unusually old code in error.c that
> should instead be relying on gnulib's own strerror_r module.
> We'll probably fix it properly in the long run, but simply not using
> --enable-gcc-warnings will avoid it for you now.

Ok.

>> I would like to set up buildbot, to which repository? The one at Savannah?
>> Do you have a preferred branch?
>>  git://git.savannah.gnu.org/diffutils.git
> 
> A buildbot would be great!
> That is indeed the right repository, and as for branch, please use "master".

This is now set up:
  https://buildfarm.opencsw.org/buildbot/waterfall?tag=diffutils

I will add more build options, compilers and platforms sometime.

I can also configure sending automatic emails if a build breaks, which destination
address would you prefer?


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)]

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Dagobert Michelsen <dam <at> opencsw.org>, Jim Meyering <jim <at> meyering.net>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: bug#24228: [platform-testers] new
 snapshot available: diffutils-3.4.9-d647
Date: Thu, 8 Sep 2016 09:30:07 -0700

On 09/08/2016 05:02 AM, Dagobert Michelsen wrote:
>>>> FAIL: test-timespec
>>>> >>>===================
>>>> >>>
>>>> >>>test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
>>>> >>>FAIL test-timespec (exit status: 262)
>> >
>> >This one is interesting.
>> >It would be great if you could get in with a debugger to see what
>> >exactly is going wrong.
>> >It's probably not an issue for diffutils, as it barely uses timespec
>> >at all and does not use timespec_sub.
> I didn’t get into looking into that yet, sorry.
>

If the failure is in line test-timespec.c line 138 it is more likely a 
bug in timespec_cmp.

For what it's worth I cannot reproduce the problem on Solaris 10 sparc 
when compiling with Oracle Studio 12.5 with the -m64 flag. I used the 
current version of gnulib (commit 
7da1068a87c331793af73359d8637d140d08a674), created the test with 
'./gnulib-tool --create-testdir --dir foo timespec', built it with 
'./configure CC="cc -m64"' and ran it with 'make check'.

Perhaps it was a compiler bug, fixed in Oracle Studio 12.5? One can 
always hope.

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

From: Jim Meyering <jim <at> meyering.net>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Thu, 8 Sep 2016 17:25:37 -0700

On Thu, Sep 8, 2016 at 5:02 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
> Hi Jim,
>
> Am 19.08.2016 um 06:18 schrieb Jim Meyering <jim <at> meyering.net>:
>> On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
>> ...
>>>> http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz
>>
>> Thank you for testing.
>>
>>> I get this error on Solaris 10 Sparc with GCC 5.2, same for
>>> Solaris 11 Sparc with GCC 5.2:
>>>
>>>>  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
>>>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255
>>
>> In this case, I think the best advice it not to with warnings enabled
>> (don't use configure's --enable-gcc-warnings) unless you're using a
>> sufficiently modern system, probably glibc-based.
>
> Ok.
>
>>> Compilation on Solaris 11 x86, 32 and 64 bit work fine with
>>> Oracle Studio 12.4.
>>>
>>> On Solaris 11 Sparc with Oracle Studio 12.4 it works for 32 bit,
>>> for 64 bit I get an error in the testsuite:
>>>
>>>> FAIL: test-timespec
>>>> ===================
>>>>
>>>> test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
>>>> FAIL test-timespec (exit status: 262)
>>
>> This one is interesting.
>> It would be great if you could get in with a debugger to see what
>> exactly is going wrong.
>> It's probably not an issue for diffutils, as it barely uses timespec
>> at all and does not use timespec_sub.
>
> I didn’t get into looking into that yet, sorry.
>
>>> On Solaris 9 Sparc with GCC 4.6 I still get this, probably because the
>>> change you proposed in not in the release tarball. I’d happily test if the
>>> updated tarballs are bootstrapped:
>>>
>>>>  CC       error.o
>>>> error.c:112:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>>
>> This appears to be exposing some unusually old code in error.c that
>> should instead be relying on gnulib's own strerror_r module.
>> We'll probably fix it properly in the long run, but simply not using
>> --enable-gcc-warnings will avoid it for you now.
>
> Ok.
>
>>> I would like to set up buildbot, to which repository? The one at Savannah?
>>> Do you have a preferred branch?
>>>  git://git.savannah.gnu.org/diffutils.git
>>
>> A buildbot would be great!
>> That is indeed the right repository, and as for branch, please use "master".
>
> This is now set up:
>   https://buildfarm.opencsw.org/buildbot/waterfall?tag=diffutils
>
> I will add more build options, compilers and platforms sometime.
>
> I can also configure sending automatic emails if a build breaks, which destination
> address would you prefer?

Great!
Will get back to you shortly with an address. It'll probably be a new
mailing list.

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

From: Jim Meyering <jim <at> meyering.net>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Fri, 9 Sep 2016 20:10:23 -0700

On Thu, Sep 8, 2016 at 5:25 PM, Jim Meyering <jim <at> meyering.net> wrote:
> On Thu, Sep 8, 2016 at 5:02 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>> Hi Jim,
>>
>> Am 19.08.2016 um 06:18 schrieb Jim Meyering <jim <at> meyering.net>:
>>> On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>>>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
>>> ...
>>>>> http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz
>>>
>>> Thank you for testing.
>>>
>>>> I get this error on Solaris 10 Sparc with GCC 5.2, same for
>>>> Solaris 11 Sparc with GCC 5.2:
>>>>
>>>>>  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
>>>>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255
>>>
>>> In this case, I think the best advice it not to with warnings enabled
>>> (don't use configure's --enable-gcc-warnings) unless you're using a
>>> sufficiently modern system, probably glibc-based.
>>
>> Ok.
>>
>>>> Compilation on Solaris 11 x86, 32 and 64 bit work fine with
>>>> Oracle Studio 12.4.
>>>>
>>>> On Solaris 11 Sparc with Oracle Studio 12.4 it works for 32 bit,
>>>> for 64 bit I get an error in the testsuite:
>>>>
>>>>> FAIL: test-timespec
>>>>> ===================
>>>>>
>>>>> test-timespec.c:138: assertion 'cmp (a, b) == sign (i - j)' failed
>>>>> FAIL test-timespec (exit status: 262)
>>>
>>> This one is interesting.
>>> It would be great if you could get in with a debugger to see what
>>> exactly is going wrong.
>>> It's probably not an issue for diffutils, as it barely uses timespec
>>> at all and does not use timespec_sub.
>>
>> I didn’t get into looking into that yet, sorry.
>>
>>>> On Solaris 9 Sparc with GCC 4.6 I still get this, probably because the
>>>> change you proposed in not in the release tarball. I’d happily test if the
>>>> updated tarballs are bootstrapped:
>>>>
>>>>>  CC       error.o
>>>>> error.c:112:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>>>
>>> This appears to be exposing some unusually old code in error.c that
>>> should instead be relying on gnulib's own strerror_r module.
>>> We'll probably fix it properly in the long run, but simply not using
>>> --enable-gcc-warnings will avoid it for you now.
>>
>> Ok.
>>
>>>> I would like to set up buildbot, to which repository? The one at Savannah?
>>>> Do you have a preferred branch?
>>>>  git://git.savannah.gnu.org/diffutils.git
>>>
>>> A buildbot would be great!
>>> That is indeed the right repository, and as for branch, please use "master".
>>
>> This is now set up:
>>   https://buildfarm.opencsw.org/buildbot/waterfall?tag=diffutils
>>
>> I will add more build options, compilers and platforms sometime.
>>
>> I can also configure sending automatic emails if a build breaks, which destination
>> address would you prefer?
>
> Great!
> Will get back to you shortly with an address. It'll probably be a new
> mailing list.

Hi Dagobert,
I've just created a new mailing list: diffutils-devel <at> gnu.org
Please use that for your new cibiuld tool, assuming it will not spam
the list too much, of course.

This new list is mainly to have a forum in which to discuss
development without incurring the administrative cost of an
auto-created debbugs ticket per thread. Of course, if you have a bug,
or a patch for which issue tracking is a good idea, please continue to
use the bug-diffutils mailing list.

Note that I'll post announcements to the new list from now on, not to
the bug-diffutils list.

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

From: Dagobert Michelsen <dam <at> opencsw.org>
To: Jim Meyering <jim <at> meyering.net>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Mon, 12 Sep 2016 15:41:45 +0200

[Message part 1 (text/plain, inline)]

Hi Jim,

Am 10.09.2016 um 05:10 schrieb Jim Meyering <jim <at> meyering.net>:
> On Thu, Sep 8, 2016 at 5:25 PM, Jim Meyering <jim <at> meyering.net> wrote:
>> On Thu, Sep 8, 2016 at 5:02 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>> Hi Jim,
>>> 
>>> Am 19.08.2016 um 06:18 schrieb Jim Meyering <jim <at> meyering.net>:
>>>> On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>>>>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
>>>> ...
>>>>>> http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz
>>>> 
>>>> Thank you for testing.
>>>> 
>>>>> I get this error on Solaris 10 Sparc with GCC 5.2, same for
>>>>> Solaris 11 Sparc with GCC 5.2:
>>>>> 
>>>>>> 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
>>>>>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255
>>>> 
>>>> In this case, I think the best advice it not to with warnings enabled
>>>> (don't use configure's --enable-gcc-warnings) unless you're using a
>>>> sufficiently modern system, probably glibc-based.
>>> 
>>> Ok.

I added this to the buildbot and everything is green now.

> I've just created a new mailing list: diffutils-devel <at> gnu.org
> Please use that for your new cibiuld tool, assuming it will not spam
> the list too much, of course.

There will only be an email send when it transitions from good to bad or
vice versa.


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)]

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

From: Jim Meyering <jim <at> meyering.net>
To: Dagobert Michelsen <dam <at> opencsw.org>
Cc: 24228 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24228: [platform-testers] new snapshot
 available: diffutils-3.4.9-d647
Date: Mon, 12 Sep 2016 07:29:03 -0700

On Mon, Sep 12, 2016 at 6:41 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
> Hi Jim,
>
> Am 10.09.2016 um 05:10 schrieb Jim Meyering <jim <at> meyering.net>:
>> On Thu, Sep 8, 2016 at 5:25 PM, Jim Meyering <jim <at> meyering.net> wrote:
>>> On Thu, Sep 8, 2016 at 5:02 AM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>>> Hi Jim,
>>>>
>>>> Am 19.08.2016 um 06:18 schrieb Jim Meyering <jim <at> meyering.net>:
>>>>> On Mon, Aug 15, 2016 at 1:35 PM, Dagobert Michelsen <dam <at> opencsw.org> wrote:
>>>>>>> Am 15.08.2016 um 07:33 schrieb Jim Meyering <jim <at> meyering.net>:
>>>>> ...
>>>>>>> http://meyering.net/diff/diffutils-3.4.9-d647.tar.xz
>>>>>
>>>>> Thank you for testing.
>>>>>
>>>>>> I get this error on Solaris 10 Sparc with GCC 5.2, same for
>>>>>> Solaris 11 Sparc with GCC 5.2:
>>>>>>
>>>>>>> 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
>>>>>>> gmake[2]: *** [Makefile:1784: rawmemchr.o] Error 255
>>>>>
>>>>> In this case, I think the best advice it not to with warnings enabled
>>>>> (don't use configure's --enable-gcc-warnings) unless you're using a
>>>>> sufficiently modern system, probably glibc-based.
>>>>
>>>> Ok.
>
> I added this to the buildbot and everything is green now.
>
>> I've just created a new mailing list: diffutils-devel <at> gnu.org
>> Please use that for your new cibiuld tool, assuming it will not spam
>> the list too much, of course.
>
> There will only be an email send when it transitions from good to bad or
> vice versa.

Great! Thanks again for setting this up.

Previous Next

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