Opened 2 years ago

Closed 21 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 2 years ago.

Download all attachments as: .zip

Change History (13)

Changed 2 years ago by vdelecroix

comment:1 Changed 2 years ago by vdelecroix

  • Description modified (diff)

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

  • Cc jpflori added

comment:4 Changed 2 years 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 2 years ago by vdelecroix

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

Last edited 2 years ago by vdelecroix (previous) (diff)

comment:6 in reply to: ↑ 5 Changed 2 years ago by vdelecroix

Replying to vdelecroix:

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

comment:7 Changed 2 years ago by fbissey

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

comment:8 Changed 2 years 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 2 years ago by vdelecroix

  • Description modified (diff)

comment:10 Changed 2 years ago by chapoton

  • Status changed from needs_review to positive_review

comment:11 Changed 23 months ago by chapoton

  • Priority changed from blocker to critical

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