Opened 6 years ago

Closed 6 years ago

#17204 closed defect (fixed)

OSX Yosemite libtool version detection

Reported by: vbraun Owned by:
Priority: major Milestone: sage-6.4
Component: packages: standard Keywords: yosemite
Cc: Merged in:
Authors: Volker Braun Reviewers: John Palmieri
Report Upstream: Fixed upstream, in a later stable release. Work issues:
Branch: 193836c (Commits) Commit: 193836c7f8ec3a103c4c6d5c91a6fbe77b6a7325
Dependencies: Stopgaps:

Description (last modified by vbraun)

libtool picks the wrong linker flags as it misinterprets the 10.10 version number

Note: the libtool.m4 macro may be found munged into the auto-generated aclocal.m4 and configure.

The fix will be in libtool-2.4.3 to be released shortly.

Change History (15)

comment:1 Changed 6 years ago by vbraun

  • Component changed from PLEASE CHANGE to packages: standard
  • Dependencies set to #17169
  • Type changed from PLEASE CHANGE to defect

comment:2 Changed 6 years ago by vbraun

  • Description modified (diff)

comment:3 Changed 6 years ago by vbraun

  • Branch set to u/vbraun/osx_yosemite_libtool_version_detection

comment:4 Changed 6 years ago by vbraun

  • Commit set to 193836c7f8ec3a103c4c6d5c91a6fbe77b6a7325
  • Keywords yosemite added

New commits:

​193836cUse 10.9 as MACOSX_DEPLOYMENT_TARGET on 10.10

comment:5 Changed 6 years ago by vbraun

There will be a libtool release soon: ​http://lists.gnu.org/archive/html/libtool/2014-10/msg00000.html

Regardless, I think we should work around this by setting MACOSX_DEPLOYMENT_TARGET to 10.9. By the time 10.11 rolls out we'll be in better shape to push this out to upstream projects.

Last edited 6 years ago by vbraun (previous) (diff)

comment:6 Changed 6 years ago by vbraun

  • Report Upstream changed from N/A to Fixed upstream, in a later stable release.

comment:7 Changed 6 years ago by vbraun

For the record, all testsuites pass except the following 

SAGE_CHECK_PACKAGES=!python,!zeromq,!pari,!gsl,!flint,!m4rie,!iml

The iml one is taken care of in #16706, I believe.

comment:8 Changed 6 years ago by vbraun

  • Authors set to Volker Braun
  • Status changed from new to needs_review

comment:9 Changed 6 years ago by vbraun

  • Description modified (diff)

comment:10 Changed 6 years ago by vbraun

  • Description modified (diff)

comment:11 Changed 6 years ago by jhpalmieri

  • Reviewers set to John Palmieri
  • Status changed from needs_review to positive_review

Looks good to me, builds on OS X 10.10 Yosemite.

comment:12 Changed 6 years ago by jhpalmieri

On OS X 10.9, flint and m4rie still fail their test suites with this ticket. (I assume that Python does also, but I never check that one. Same with iml.) pari passes. zeromq passes with the change from #17212 applied.

comment:13 Changed 6 years ago by vbraun

  • Dependencies #17169 deleted

comment:14 Changed 6 years ago by jhpalmieri

Of course this ticket shouldn't change anything with OS X 10.9. The failures for flint and m4rie probably come from #17169.

comment:15 Changed 6 years ago by vbraun

  • Branch changed from u/vbraun/osx_yosemite_libtool_version_detection to 193836c7f8ec3a103c4c6d5c91a6fbe77b6a7325
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.