Opened 2 years ago

Closed 2 years ago

#30594 closed defect (fixed)

ecl build with Xcode 12

Reported by: John Palmieri Owned by:
Priority: blocker Milestone: sage-9.2
Component: packages: standard Keywords:
Cc: Dima Pasechnik, Marius Gerbershagen Merged in:
Authors: John Palmieri Reviewers: Dima Pasechnik
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: e5aeb62 (Commits, GitHub, GitLab) Commit: e5aeb62b9e0c22a057b5e323c04bc1ad4c1df821
Dependencies: Stopgaps:

Status badges

Description (last modified by Dima Pasechnik)

The configure script for ecl fails with Xcode 12, with this kind of error in config.log:

conftest.c:4:3: error: implicitly declaring library function 'exit' with type 'void (int) __attribute__((noreturn))' [-Werror,-Wimplicit-function-declaration]
  exit(0);
  ^
conftest.c:4:3: note: include the header <stdlib.h> or explicitly provide a declaration for 'exit'
1 error generated.

upstream, in the develop branch, this is fixed in https://gitlab.com/embeddable-common-lisp/ecl/-/merge_requests/231

Change History (10)

comment:1 Changed 2 years ago by John Palmieri

We can imitate the changes in #30593. Here is a patch to do that. It works, but is there a better way?

comment:2 Changed 2 years ago by John Palmieri

Branch: u/jhpalmieri/ecl-vs-xcode12

comment:3 Changed 2 years ago by John Palmieri

Authors: John Palmieri
Commit: e5aeb62b9e0c22a057b5e323c04bc1ad4c1df821
Status: newneeds_review

New commits:

e5aeb62trac 30594: allow ecl to build with Xcode 12.

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

Cc: Dima Pasechnik Marius Gerbershagen added

comment:5 Changed 2 years ago by Dima Pasechnik

this seems to come from an autoconf macro, not from configure.ac itself, but which one needs to be dug up.

aclocal.m4 has hand-written macros with C/C++ programs calling exit(), and not including stdlib.h (which, by right, must be done, according to man 3 exit) - I suppose it is from there.

Once Apple in its infinite wisdom allows the download of Xcode 12 on my machine to complete, I'll have a look (assuming it is reproducible on macOS 10).

Last edited 2 years ago by Dima Pasechnik (previous) (diff)

comment:6 Changed 2 years ago by Dima Pasechnik

Is this on macOS 10, or only on Big Sur?

comment:7 Changed 2 years ago by Dima Pasechnik

Priority: blockercritical

should not be a blocker, as only beta tools are affected.

comment:8 Changed 2 years ago by Dima Pasechnik

Report Upstream: N/ACompletely fixed; Fix reported upstream

comment:9 Changed 2 years ago by Dima Pasechnik

Description: modified (diff)
Priority: criticalblocker
Report Upstream: Completely fixed; Fix reported upstreamFixed upstream, but not in a stable release.
Reviewers: Dima Pasechnik
Status: needs_reviewpositive_review

the PR has been merged by upstream.

comment:10 Changed 2 years ago by Volker Braun

Branch: u/jhpalmieri/ecl-vs-xcode12e5aeb62b9e0c22a057b5e323c04bc1ad4c1df821
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.