Opened 7 years ago

Closed 6 years ago

#13898 closed defect (wontfix)

OSX build failure in mercurial with FSF GCC

Reported by: vbraun Owned by: tbd
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Volker Braun Reviewers: John Palmieri
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13864, #13867 Stopgaps:

Description (last modified by zeier)

As reported on https://groups.google.com/d/topic/sage-devel/kfXamuej_Gs/discussion

gcc -DNDEBUG -g -O3 -Wall -Wstrict-prototypes -O2 -g -m64 
-I/opt/sage/sage-5.6.beta1/local/include/python2.7 -c mercurial/osutil.c 
-o build/temp.macosx-10.8-x86_64-2.7/mercurial/osutil.o 
In file included from 
/System/Library/Frameworks/Foundation.framework/Headers/NSObject.h:5:0, 
                  from /usr/include/os/object.h:74, 
                  from /usr/include/dispatch/dispatch.h:48, 
                  from 
/System/Library/Frameworks/CoreFoundation.framework/Headers/CFMessagePort.h:11, 
                  from 
/System/Library/Frameworks/CoreFoundation.framework/Headers/CoreFoundation.h:68, 
                  from 
/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/CarbonCore.h:18, 
                  from 
/System/Library/Frameworks/CoreServices.framework/Frameworks/AE.framework/Headers/AE.h:20, 
                  from 
/System/Library/Frameworks/CoreServices.framework/Headers/CoreServices.h:18, 
                  from 
/System/Library/Frameworks/ApplicationServices.framework/Headers/ApplicationServices.h:24, 
                  from mercurial/osutil.c:523: 
/System/Library/Frameworks/Foundation.framework/Headers/NSObjCRuntime.h:409:1: 
Fehler: verirrtes »@« im Programm 

Workaround: Use the following spkgs:

For updated versions see:

Attachments (5)

mercurial-p1.patch (926 bytes) - added by vbraun 7 years ago.
diff for review only
python-p4.patch (873 bytes) - added by zeier 7 years ago.
diff for review only
r-p7.patch (781 bytes) - added by zeier 7 years ago.
diff for review only
python-2.7.3.p7.diff (899 bytes) - added by zeier 7 years ago.
diff for review
r-2.15.2.p2.diff (730 bytes) - added by zeier 7 years ago.
diff for review

Download all attachments as: .zip

Change History (21)

comment:1 Changed 7 years ago by vbraun

  • Description modified (diff)
  • Status changed from new to needs_review

Changed 7 years ago by vbraun

diff for review only

comment:2 Changed 7 years ago by jdemeyer

The spkg obviously looks fine. But we need to check that this actually fixes the problem.

Changed 7 years ago by zeier

diff for review only

Changed 7 years ago by zeier

diff for review only

comment:3 Changed 7 years ago by zeier

I have tested all three patches also with sage-5.6.beta2

comment:4 Changed 7 years ago by vbraun

  • Description modified (diff)

comment:5 Changed 7 years ago by jhpalmieri

  • Dependencies set to #13864, #13867

comment:6 Changed 7 years ago by jhpalmieri

One data point: on an OS X 10.8.2 machine on which I had no problems before, everything still works after using these spkgs.

comment:7 Changed 7 years ago by zeier

I have just tested the three provided spkg files with sage-5.6.rc0 on OS X 10.8.2. Everything compiles.

Changed 7 years ago by zeier

diff for review

Changed 7 years ago by zeier

diff for review

comment:8 Changed 7 years ago by zeier

  • Description modified (diff)

Updated spkgs and diffs for python and r are included as they have changed in the meantime. The new ones build on python-2.7.3.p6 and r-2.15-2-p1. They can be applied to sage-5.9.beta5.

Last edited 7 years ago by zeier (previous) (diff)

comment:9 Changed 7 years ago by zeier

  • Description modified (diff)

comment:10 Changed 7 years ago by vbraun

The Python spkg needs to be rebased on top of #14423. And I'm not going to do it, I fixed it once and nobody reviewed it.

comment:11 Changed 7 years ago by vbraun

  • Status changed from needs_review to needs_work

comment:12 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:13 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:14 Changed 6 years ago by jhpalmieri

  • Milestone changed from sage-6.2 to sage-duplicate/invalid/wontfix
  • Status changed from needs_work to positive_review

I think this can be closed, since we no longer distribute Mercurial.

comment:15 Changed 6 years ago by jhpalmieri

  • Reviewers set to John Palmieri

comment:16 Changed 6 years ago by vbraun

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.