Opened 8 years ago

Closed 8 years ago

#14450 closed defect (invalid)

MPIR build is broken on arm with ubuntu raring

Reported by: Snark Owned by: drkirkby
Priority: critical Milestone: sage-duplicate/invalid/wontfix
Component: porting Keywords: mpir spkg as binutils
Cc: jdemeyer Merged in:
Authors: Reviewers: Jean-Pierre Flori, Julien Puydt
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

A change in the binutils on arm in ubuntu raring breaks the build (apparently the asm changed slightly).

I have seen the problem, and ubuntu has patches which solve the problem for their gmp package here: https://launchpad.net/ubuntu/+source/gmp/2%3A5.0.5%2Bdfsg-2ubuntu3 I have been able to check that they fix the compilation issue, but:

  1. I don't know who the author is (it's not documented in the patch as it would be in debian) ;
  2. I don't know if it was forwarded to upstream gmp, and even so, it perhaps wasn't realized that it was also a problem with mpir upstream.

So I'll probably have a mpir-2.6.0.p1.spkg to attach to this bug quite soon. [Notice: again, there might be a mpir-2.6.0.p123 somewhere of which I don't know about... I'm longing for the git transition!]

Attachments (1)

trac_14450.patch (1.8 KB) - added by Snark 8 years ago.
Patch to fix the problem

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by Snark

Patch to fix the problem

comment:1 Changed 8 years ago by Snark

  • Status changed from new to needs_review

The spkg is too big to attach in trac ; here is the "hg export tip" from my known-working package.

comment:2 Changed 8 years ago by jpflori

  • Authors set to Julien Puydt
  • Priority changed from major to critical
  • Reviewers set to Jean-Pierre Flori
  • Status changed from needs_review to needs_work
  • Work issues set to provide spkg based on #14399

You should not attach spkg to trac anyway, but put them somewhere online where you have space and which is publicly available and put a link in the ticket description. Request William Stein an account on boxen if you don't want to use another place for this.

According to https://launchpad.net/ubuntu/+source/gmp/2%3A5.0.5%2Bdfsg-2ubuntu3 the authir seem to be Adam Conrad as you already mention it in the patch itself, so please update the ticket descritpion.

Trac search quickly showed there's at least a p2 spkg already at #14399, so your spkg should be based on top of that.

I've seen no reports of this problem on mpir-devel by the way, so it would be nice if you could do that!

And the bug looks really stupid. Is this really invalid syntax in GMP/MPIR? or rather a broken update to as in binutils... I fear that the line two lines after the fixed one will break when as is updated again...

Ok just read the patch comment and it is indeed a bug in binutils. It would be nice to document which version is buggy, hoping we can discard the work around at some point.

comment:3 Changed 8 years ago by Snark

Since I reported this bug, ubuntu uploaded a package for binutils which contains a workaround for this problem. And as far as I know, the fix was forwarded upstream.

I propose to close this report.

comment:4 Changed 8 years ago by jpflori

  • Authors Julien Puydt deleted
  • Cc jdemeyer added
  • Keywords mpir spkg as binutils added
  • Milestone changed from sage-5.10 to sage-duplicate/invalid/wontfix
  • Reviewers changed from Jean-Pierre Flori to Jean-Pierre Flori, Julien Puydt
  • Status changed from needs_work to positive_review

Yes let's close this one.

comment:5 Changed 8 years ago by jdemeyer

  • Resolution set to invalid
  • Status changed from positive_review to closed
  • Work issues provide spkg based on #14399 deleted
Note: See TracTickets for help on using tickets.