#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 19 months ago.

Download all attachments as: .zip

Change History (13)

Changed 19 months ago by vdelecroix

comment:1 Changed 19 months ago by vdelecroix

  • Description modified (diff)

comment:2 Changed 19 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 19 months ago by vdelecroix

  • Cc jpflori added

comment:4 Changed 19 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 19 months ago by vdelecroix

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

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

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

Replying to vdelecroix:

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

comment:7 Changed 19 months ago by fbissey

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

comment:8 Changed 19 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 19 months ago by vdelecroix

  • Description modified (diff)

comment:10 Changed 19 months ago by chapoton

  • Status changed from needs_review to positive_review

comment:11 Changed 18 months ago by chapoton

  • Priority changed from blocker to critical

comment:12 Changed 16 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.