Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Adam D. Barratt]

Control: tags -1 + pending

On Sat, 2017-01-28 at 17:31 +, Steve McIntyre wrote:
> On Sat, Jan 28, 2017 at 05:01:36PM +, Adam Barratt wrote:
> >Control: tags -1 + confirmed
> >
> >On Thu, 2017-01-12 at 13:23 +, Steve McIntyre wrote:
> >> Binutils in jessie is building (some) broken packages on arm64,
> >> particularly those using gold. See #850814 for more details - I've
> >> scanned the archive for affected packages.
> >> 
> >> There's a simple fix proposed with a single change backported from
> >> upstream. I've built and tested 2.25-5+deb8u1.1 locally on amd64 and
> >> arm64. Debdiff attached.
> >
> >+deb8u1, please. Other than that, please go ahead.
> 
> Rebuilt with just that one change, and in incoming now.

Thanks; flagged for acceptance.

Regards,

Adam

Processed: Re: Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Debian Bug Tracking System]

Processing control commands:

> tags -1 + pending
Bug #851151 [release.debian.org] jessie-pu: package binutils/2.25-5+deb8u1.1
Added tag(s) pending.

-- 
851151: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=851151
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Steve McIntyre]

On Sat, Jan 28, 2017 at 05:01:36PM +, Adam Barratt wrote:
>Control: tags -1 + confirmed
>
>On Thu, 2017-01-12 at 13:23 +, Steve McIntyre wrote:
>> Binutils in jessie is building (some) broken packages on arm64,
>> particularly those using gold. See #850814 for more details - I've
>> scanned the archive for affected packages.
>> 
>> There's a simple fix proposed with a single change backported from
>> upstream. I've built and tested 2.25-5+deb8u1.1 locally on amd64 and
>> arm64. Debdiff attached.
>
>+deb8u1, please. Other than that, please go ahead.

Rebuilt with just that one change, and in incoming now.

>> Once this is in the archive, we'll need some binnmus scheduling to fix
>> the packages in that list in #850814.
>
>As mentioned on IRC, it won't currently help with backports rebuilds
>until after 8.8 (or until someone persuades whichever combination of DSA
>and buildd it needs to change the sources.list build for backports
>chroots).

ACK.

-- 
Steve McIntyre, Cambridge, UK.st...@einval.com
"The problem with defending the purity of the English language is that
 English is about as pure as a cribhouse whore. We don't just borrow words; on
 occasion, English has pursued other languages down alleyways to beat them
 unconscious and rifle their pockets for new vocabulary."  -- James D. Nicoll


signature.asc
Description: Digital signature

Processed: Re: Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Debian Bug Tracking System]

Processing control commands:

> tags -1 + confirmed
Bug #851151 [release.debian.org] jessie-pu: package binutils/2.25-5+deb8u1.1
Added tag(s) confirmed.

-- 
851151: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=851151
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Adam D. Barratt]

Control: tags -1 + confirmed

On Thu, 2017-01-12 at 13:23 +, Steve McIntyre wrote:
> Binutils in jessie is building (some) broken packages on arm64,
> particularly those using gold. See #850814 for more details - I've
> scanned the archive for affected packages.
> 
> There's a simple fix proposed with a single change backported from
> upstream. I've built and tested 2.25-5+deb8u1.1 locally on amd64 and
> arm64. Debdiff attached.

+deb8u1, please. Other than that, please go ahead.

> OK to upload?
> 
> Once this is in the archive, we'll need some binnmus scheduling to fix
> the packages in that list in #850814.

As mentioned on IRC, it won't currently help with backports rebuilds
until after 8.8 (or until someone persuades whichever combination of DSA
and buildd it needs to change the sources.list build for backports
chroots).

Regards,

Adam

Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Steve McIntyre]

On Thu, Jan 12, 2017 at 01:23:39PM +, Steve McIntyre wrote:
>Package: release.debian.org
>Severity: normal
>Tags: jessie
>User: release.debian@packages.debian.org
>Usertags: pu
>
>Hi folks,
>
>Binutils in jessie is building (some) broken packages on arm64,
>particularly those using gold. See #850814 for more details - I've
>scanned the archive for affected packages.
>
>There's a simple fix proposed with a single change backported from
>upstream. I've built and tested 2.25-5+deb8u1.1 locally on amd64 and
>arm64. Debdiff attached.
>
>OK to upload?
>
>Once this is in the archive, we'll need some binnmus scheduling to fix
>the packages in that list in #850814.

Forgot to highlight - I'm not the binutils maintainer but doko has
already acked my patch and said he's happy for me to upload this - see

  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=850814#22

-- 
Steve McIntyre, Cambridge, UK.st...@einval.com
Into the distance, a ribbon of black
Stretched to the point of no turning back

Bug#851151: jessie-pu: package binutils/2.25-5+deb8u1.1

[Steve McIntyre]

Package: release.debian.org
Severity: normal
Tags: jessie
User: release.debian@packages.debian.org
Usertags: pu

Hi folks,

Binutils in jessie is building (some) broken packages on arm64,
particularly those using gold. See #850814 for more details - I've
scanned the archive for affected packages.

There's a simple fix proposed with a single change backported from
upstream. I've built and tested 2.25-5+deb8u1.1 locally on amd64 and
arm64. Debdiff attached.

OK to upload?

Once this is in the archive, we'll need some binnmus scheduling to fix
the packages in that list in #850814.

-- System Information:
Debian Release: 8.6
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.8.0-0.bpo.2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
diff -u binutils-2.25/debian/changelog binutils-2.25/debian/changelog
--- binutils-2.25/debian/changelog
+++ binutils-2.25/debian/changelog
@@ -1,3 +1,11 @@
+binutils (2.25-5+deb8u1.1) stable; urgency=medium
+
+  * NMU
+  * Apply patch from upstream to fix gold on arm64. The ABI specifies
+using a pagesize of 64k for ELF binaries.
+
+ -- Steve McIntyre <93...@debian.org>  Thu, 12 Jan 2017 10:36:22 +
+
 binutils (2.25-5) unstable; urgency=medium
 
   * Remove '*.rej' files in the source package. Closes: #775679.
diff -u binutils-2.25/debian/patches/series binutils-2.25/debian/patches/series
--- binutils-2.25/debian/patches/series
+++ binutils-2.25/debian/patches/series
@@ -46,0 +47 @@
+gold_arm64_pagesize.patch
only in patch2:
unchanged:
--- binutils-2.25.orig/debian/patches/gold_arm64_pagesize.patch
+++ binutils-2.25/debian/patches/gold_arm64_pagesize.patch
@@ -0,0 +1,47 @@
+commit 3b0357dadaf2366cc418ec725dec55b1cea1a2e7
+Author: Andreas Schwab 
+Date:   Thu Oct 1 12:30:18 2015 +0200
+
+gold: fix ABI pagesize for aarch64
+
+* aarch64.cc (aarch64_info): Set abi_pagesize to 64K.
+
+diff a/gold/aarch64.cc b/gold/aarch64.cc
+--- a/gold/aarch64.cc	2017-01-12 10:43:43.948866937 +
 b/gold/aarch64.cc	2017-01-12 10:44:21.913133322 +
+@@ -2232,7 +2232,7 @@
+   '\0',			// wrap_char
+   "/lib/ld.so.1",	// program interpreter
+   0x40,		// default_text_segment_address
+-  0x1000,		// abi_pagesize (overridable by -z max-page-size)
++  0x1,		// abi_pagesize (overridable by -z max-page-size)
+   0x1000,		// common_pagesize (overridable by -z common-page-size)
+   false,// isolate_execinstr
+   0,// rosegment_gap
+@@ -2259,7 +2259,7 @@
+   '\0',			// wrap_char
+   "/lib/ld.so.1",	// program interpreter
+   0x40,		// default_text_segment_address
+-  0x1000,		// abi_pagesize (overridable by -z max-page-size)
++  0x1,		// abi_pagesize (overridable by -z max-page-size)
+   0x1000,		// common_pagesize (overridable by -z common-page-size)
+   false,// isolate_execinstr
+   0,// rosegment_gap
+@@ -2286,7 +2286,7 @@
+   '\0',			// wrap_char
+   "/lib/ld.so.1",	// program interpreter
+   0x40,		// default_text_segment_address
+-  0x1000,		// abi_pagesize (overridable by -z max-page-size)
++  0x1,		// abi_pagesize (overridable by -z max-page-size)
+   0x1000,		// common_pagesize (overridable by -z common-page-size)
+   false,// isolate_execinstr
+   0,// rosegment_gap
+@@ -2313,7 +2313,7 @@
+   '\0',			// wrap_char
+   "/lib/ld.so.1",	// program interpreter
+   0x40,		// default_text_segment_address
+-  0x1000,		// abi_pagesize (overridable by -z max-page-size)
++  0x1,		// abi_pagesize (overridable by -z max-page-size)
+   0x1000,		// common_pagesize (overridable by -z common-page-size)
+   false,// isolate_execinstr
+   0,// rosegment_gap