Opened 12 months ago

Closed 9 months ago

#24902 closed defect (wontfix)

optional package deformation fails to build

Reported by: vdelecroix Owned by:
Priority: critical Milestone: sage-duplicate/invalid/wontfix
Component: packages: optional Keywords:
Cc: jpflori Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by vdelecroix)

On archlinux 

/usr/bin/ld: -r and -pie may not be used together

See the more complete log deformation-d05941b.log​.

Upstream report: ​https://github.com/jpflori/pydeformation/issues/5

A patch is provided in #24575.

Attachments (1)

deformation-d05941b.log​ (8.0 KB) - added by vdelecroix 12 months ago.

Download all attachments as: .zip

Change History (13)

Changed 12 months ago by vdelecroix

comment:1 Changed 12 months ago by vdelecroix

  • Description modified (diff)

comment:2 Changed 12 months ago by fbissey

Flint had that issue some time ago. This is caused by the hardening of the toolchain. Look at what has been done for flint here ​https://github.com/sagemath/sage/blob/master/build/pkgs/flint/patches/flint-pie-hardening-conflict.patch. Basically you give -r directly to the compiler but don't pass it directly to the linker as you would with -Wl,-r.

comment:3 Changed 12 months ago by vdelecroix

  • Cc jpflori added

comment:4 Changed 12 months ago by fbissey

The culprit is this line 

Makefile.subdirs:       $(QUIET_CC) $(CC) $(ABI_FLAG) -Wl,-r $^ -o $@ -nostdlib

Change -Wl,-r to plain -r should get you in business. Can someone also please update SPKG.txt with upstream location? Where did you get that tarball from? Has it been processed before being put on the mirror?

comment:5 follow-up: Changed 12 months ago by vdelecroix

A patch is provided in #24575, should I move it here?

Last edited 12 months ago by vdelecroix (previous) (diff)

comment:6 in reply to: ↑ 5 Changed 12 months ago by vdelecroix

Replying to vdelecroix:

A patch is provided in #24575, should I move it here?

comment:7 Changed 12 months ago by fbissey

No need we can just mark this ticket as a "duplicate" of #24575.

comment:8 Changed 12 months ago by vdelecroix

  • Description modified (diff)
  • Milestone changed from sage-8.2 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

comment:9 Changed 12 months ago by vdelecroix

  • Description modified (diff)

comment:10 Changed 12 months ago by chapoton

  • Status changed from needs_review to positive_review

comment:11 Changed 10 months ago by chapoton

  • Priority changed from blocker to critical

comment:12 Changed 9 months ago by vdelecroix

  • Resolution set to wontfix
  • Status changed from positive_review to closed

closing positively reviewed duplicates

Note: See TracTickets for help on using tickets.