Opened 3 years ago

Closed 3 years ago

#29089 closed defect (fixed)

Also gfortran needs the crti.o fix - for Fedora 31/32

Reported by: Matthias Köppe Owned by:
Priority: major Milestone: sage-9.1
Component: packages: standard Keywords:
Cc: Dima Pasechnik, Erik Bray, Volker Braun, John Palmieri, Travis Scrimshaw Merged in:
Authors: Matthias Koeppe Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: 3646c06 (Commits, GitHub, GitLab) Commit: 3646c0679e0316103daeeeabd4ba818539ae17b6
Dependencies: Stopgaps:

Status badges

Description (last modified by Matthias Köppe)

#27907 - "Building gcc 7.2.0 fails on CentOS / Red Hat / Fedora Linux systems because of crti.o" added code in spkg-postinst to copy crti.o.

gfortran needs the same - as observed on fedora-31-minimal (https://github.com/mkoeppe/sage/runs/411500550) and fedora-32-minimal (https://github.com/mkoeppe/sage/runs/411500596)

Tests with the fix are at: https://github.com/mkoeppe/sage/pull/1

Unrelated error: #29162 - gfortran compile error on debian-jessie

Change History (14)

comment:1 Changed 3 years ago by Matthias Köppe

Description: modified (diff)
Summary: Also gfortran needs the crti.o fix - for Fedora 31Also gfortran needs the crti.o fix - for Fedora 31/32

comment:2 Changed 3 years ago by Matthias Köppe

Authors: Matthias Koeppe

comment:3 Changed 3 years ago by Matthias Köppe

Branch: u/mkoeppe/also_gfortran_needs_the_crti_o_fix___for_fedora_31_32

comment:4 Changed 3 years ago by Matthias Köppe

Commit: 79979908a2c6e6d5f12ff701a710deed1e087112
Status: newneeds_review

New commits:

6d52bc6crti.o fix for gfortran
9379d20fixup: use set-library-path
174f051build/pkgs/gfortran/spkg-install: Work around lib/lib64 problem
7997990Fixup finding crt dest dir

comment:5 Changed 3 years ago by Matthias Köppe

Description: modified (diff)

comment:6 Changed 3 years ago by Matthias Köppe

Fixes fedora-31-minimal, fedora-32-minimal, as can be seen in the tests at https://github.com/mkoeppe/sage/pull/1

comment:7 Changed 3 years ago by Matthias Köppe

Status: needs_reviewneeds_work

comment:8 Changed 3 years ago by git

Commit: 79979908a2c6e6d5f12ff701a710deed1e0871123646c0679e0316103daeeeabd4ba818539ae17b6

Branch pushed to git repo; I updated commit sha1. New commits:

3646c06More workaround - so as not to break build on macOS

comment:9 Changed 3 years ago by Matthias Köppe

Status: needs_workneeds_review

comment:10 Changed 3 years ago by Matthias Köppe

Description: modified (diff)

comment:11 Changed 3 years ago by Matthias Köppe

Cc: John Palmieri Travis Scrimshaw added

Anyone interested in reviewing this ticket?

comment:12 Changed 3 years ago by Volker Braun

Reviewers: Volker Braun
Status: needs_reviewpositive_review

comment:13 Changed 3 years ago by Matthias Köppe

Thanks!

comment:14 Changed 3 years ago by Volker Braun

Branch: u/mkoeppe/also_gfortran_needs_the_crti_o_fix___for_fedora_31_323646c0679e0316103daeeeabd4ba818539ae17b6
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.