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


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: 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 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.