binutils on armel fails to build due to regression test failure on 64 Bytes alignment test for gas/arm/arm.exp

Bug #398732 reported by Oliver Grawert
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
binutils
Invalid
Medium
binutils (Ubuntu)
Fix Released
High
Unassigned

Bug Description

Binary package hint: binutils

i am not sure how related this is to #385501, seems the testsuite was completely rewritten between 2.19.51.20090622 and 2.19.51.20090704 though ...

which appears to result in the build failure seen at http://launchpadlibrarian.net/28683610/buildlog_ubuntu-karmic-armel.binutils_2.19.51.20090704-1ubuntu1_FAILEDTOBUILD.txt.gz

....
Running /build/buildd/binutils-2.19.51.20090704/gas/testsuite/gas/arm/arm.exp ...
FAIL: 64 Bytes alignment test
....
Test results, compared with installed binutils:
W: [gas/arm/arm.exp] REGRESSION (PASS -> FAIL): 64 Bytes alignment test
1 REGRESSIONS (0.10%).
...

Tags: armel

Related branches

Oliver Grawert (ogra)
tags: added: armel
Changed in binutils (Ubuntu):
importance: Undecided → High
Revision history for this message
In , Oliver Grawert (ogra) wrote :

apparently the gas testsuite was rewritten between .20090622 and 20090704.
building with gcc-4.4 makes the gas/arm/arm.exp test fail in the 64 Bytes
alignment test.

details and a full build log can be found at
https://bugs.launchpad.net/bugs/398732

Revision history for this message
Oliver Grawert (ogra) wrote :

looking at the debian PTS the same version of binutils seems to build fine there with the default gcc-4.3

Changed in binutils:
status: Unknown → Confirmed
Loïc Minier (lool)
Changed in binutils (Ubuntu):
status: New → Confirmed
Revision history for this message
Matthias Klose (doko) wrote : Re: [Bug 398732] Re: binutils on armel fails to build due to regression test failure on 64 Bytes alignment test for gas/arm/arm.exp

Oliver Grawert schrieb:
> looking at the debian PTS the same version of binutils seems to build
> fine there with the default gcc-4.3

the Debian package doesn't have this patch.

Revision history for this message
In , Matthias Klose (doko) wrote :

this is a patch not yet included in trunk.
posted at http://sourceware.org/ml/binutils/2009-06/msg00289.html

Revision history for this message
In , Loïc Minier (lool) wrote :

Sorry, this issue is specific to Ubuntu as we added a patch to change the
default alignment to 64 bits. I tried building tip, it builds at least up to
gas and passes the gas testsuite; with the patch applied the test suite fails as
follows:
...
Running target unix
Using /usr/share/dejagnu/baseboards/unix.exp as board description file for target.
Using /usr/share/dejagnu/config/unix.exp as generic interface file for target.
Using /home/lool/scratch/binutils/binutils/gas/testsuite/config/default.exp as
tool-and-target-specific interface file.
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/all/gas.exp ...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/alpha/alpha.exp ...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/arc/arc.exp ...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/arc/warn.exp ...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/arm/arm.exp ...
FAIL: 64 Bytes alignment test
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/bfin/bfin.exp ...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/cfi/cfi.exp ...
...
Running /home/lool/scratch/binutils/binutils/gas/testsuite/gas/z8k/z8k.exp ...

                === gas Summary ===

# of expected passes 319
# of unexpected failures 1
# of unsupported tests 4
../as-new 2.19.51.20090703

make[2]: *** [check-DEJAGNU] Error 1

Revision history for this message
In , Loïc Minier (lool) wrote :

err 64 bytes not bits

Revision history for this message
In , Matthias Klose (doko) wrote :

closing as invalid

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package binutils - 2.19.51.20090713-0ubuntu1

---------------
binutils (2.19.51.20090713-0ubuntu1) karmic; urgency=low

  * Snapshot, taken from the trunk 20090713.
  * debian/*.shlibs: Update to the version from the trunk.
  * 128_arm_eabi_align64.dpatch: Adjust expected output to changed objdump
    output. LP: #398732.

 -- Matthias Klose <email address hidden> Mon, 13 Jul 2009 13:21:56 -0400

Changed in binutils (Ubuntu):
status: Confirmed → Fix Released
Changed in binutils:
status: Confirmed → Invalid
Changed in binutils:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.