Opened 13 years ago
Closed 7 years ago
#7034 closed defect (fixed)
PolyBoRi pass GNU specific options to the Sun compiler
Reported by: | drkirkby | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | porting: Solaris | Keywords: | SCons Solaris |
Cc: | polybori@…, AlexanderDreyer | Merged in: | |
Authors: | Reviewers: | Jeroen Demeyer | |
Report Upstream: | Fixed upstream, in a later stable release. | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Using
- Solaris 10 update 7 on SPARC
- sage-4.1.2.alpha2
- Sun Studio 12.1
- An updated configure script to allow the Sun compiler to be used #7021
CC was set to the Sun C compiler, and CXX to the Sun C++ compiler,
It would appear that Using PolyBoRi? correctly uses the value of CXX as the C++ compiler, but will pass GNU specific options to that compiler.
polybori-0.6.3-20090827/.hgignore Finished extraction **************************************************** Host system uname -a: SunOS swan 5.10 Generic_139555-08 sun4u sparc SUNW,Sun-Blade-1000 **************************************************** **************************************************** CC Version /opt/xxxsunstudio12.1/bin/cc -v usage: cc [ options] files. Use 'cc -flags' for details **************************************************** mkdir: Failed to make directory "src/boost_1_34_1.cropped"; File exists Starting build... Removing old PolyBoRi install... Done removing old PolyBoRi install. Running build_polybori... scons: Reading SConscript files ... Sun linker detected. Checking for C header file gd.h... no Checking for C++ header file unordered_map... no Checking for C++ header file tr1/unordered_map... no Checking for C++ header file ext/hash_map... no Warning: No LaTeX to html converter found, Tutorial will not be installed Checking for C library m4ri... no Checking for C header file gd.h... no Symlinking to M4RI/m4ri ... no python extension scons: done reading SConscript files. scons: Building targets ... /opt/xxxsunstudio12.1/bin/CC -o polybori/src/BoolePolyRing.o -c -O3 -Wno-long-long -Wreturn-type -g -fPIC -ftemplate-depth-100 -g -fPIC -O3 -Wno-long-long -Wreturn-type -g -fPIC -DNDEBUG -DPACKED -DHAVE_M4RI -DHAVE_IEEE_754 -DBSD -I/export/home/drkirkby/sage/gcc32/sage-4.1.2.alpha2/spkg/build/polybori-0.6.3-20090827/src/boost_1_34_1.cropped -I/export/home/drkirkby/sage/gcc32/sage-4.1.2.alpha2/local/include -I/export/home/drkirkby/sage/gcc32/sage-4.1.2.alpha2/local/include/python2.6 -Ipolybori/include -IM4RI -ICudd/obj -ICudd/util -ICudd/cudd -ICudd/mtr -ICudd/st -ICudd/epd polybori/src/BoolePolyRing.cc CC: Warning: Option -Wno-long-long passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -Wreturn-type passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -fPIC passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -ftemplate-depth-100 passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -fPIC passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -Wno-long-long passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -Wreturn-type passed to ld, if ld is invoked, ignored otherwise CC: Warning: Option -fPIC passed to ld, if ld is invoked, ignored otherwise "polybori/include/CDDManager.h", line 103: Warning: Last line in file "polybori/include/cacheopts.h" is not terminated with a newline. "polybori/include/CCuddZDD.h", line 308: Warning (Anachronism): Formal argument func of type DdNode*(*)(DdManager*,DdNode*,int) in call to polybori::CCuddDDBase<polybori::CCuddZDD>::apply(DdNode*(*)(DdManager*,DdNode*,int), int) const is being passed extern "C" DdNode*(*)(DdManager*,DdNode*,int). "polybori/include/CCuddZDD.h", line 308: Warning (Anachronism): Formal argument func of type DdNode*(*)(DdManager*,DdNode*,int) in call to polybori::CCuddDDBase<polybori::CCuddZDD>::apply(DdNode*(*)(DdManager*,DdNode*,int), int) const is being passed extern "C" DdNode*(*)(DdManager*,DdNode*,int). "polybori/include/CCuddZDD.h", line 308: Warning (Anachronism): Formal argument func of type DdNode*(*)(DdManager*,DdNode*,int) in call to polybori::CCuddDDBase<polybori::CCuddZDD>::apply(DdNode*(*)(DdManager*,DdNode*,int), int) const is being passed extern "C" DdNode*(*)(DdManager*,DdNode*,int).
PolyBoRi? uses SCons, and the use of SCons in Sage seems to cause countless problems.
Change History (14)
comment:1 Changed 13 years ago by
- Summary changed from Using * Solaris 10 update 7 on SPARC * sage-4.1.2.alpha2 * Sun Studio 12.1 * An updated configure script to allow the Sun compiler to be used http://sagetrac.org/sage_trac/ticket/7021 CC was set to the Sun C compiler, and CXX to the Sun C++ compiler, uses GNU flags with the Sun compiler to PolyBoRi pass GNU specific options to the Sun compiler. * Solaris 10 update 7 on SPARC * sage-4.1.2.alpha2 * Sun Studio 12.1 * An updated configure script to allow the Sun compiler to be used http://sagetrac.org/sage_trac/ticket/7021 CC was set to the Sun C compiler, and CXX to the Sun C++ compiler, uses GNU flags with the Sun compiler
comment:2 Changed 13 years ago by
- Summary changed from PolyBoRi pass GNU specific options to the Sun compiler. * Solaris 10 update 7 on SPARC * sage-4.1.2.alpha2 * Sun Studio 12.1 * An updated configure script to allow the Sun compiler to be used http://sagetrac.org/sage_trac/ticket/7021 CC was set to the Sun C compiler, and CXX to the Sun C++ compiler, uses GNU flags with the Sun compiler to PolyBoRi pass GNU specific options to the Sun compiler
comment:3 Changed 12 years ago by
- Report Upstream set to Reported upstream. Little or no feedback.
comment:4 Changed 12 years ago by
From polybori-discuss: In fact, Sage forces PolyBoRi? to do so. Sage's spkg contains a custom.py file, which overwrites PolyBoRi?'s settings: Looking at polybori-0.6.3.r1647-20091028, one can see:
dreyer@lts035 [999] (0) [...patches]cat custom.py import os import sys
CCFLAGS=-O3 -Wno-long-long -Wreturn-type -g -fPIC? CXXFLAGS=CCFLAGS+-ftemplate-depth-100 -g -fPIC? [...]
The -W options can be dropped without any problems for all platforms. But the remaining options were set intentionally in Sage, as far as I know. (I don't know the reason for -fPIC). Does PolyBoRi? compile with the sun-compiler, if -ftemplate-depth-100 and -fPIC is dropped, i.e. if the two lines from custom.py read as follows:
CCFLAGS=-O3 -g? CXXFLAGS=[""] (previous line corrected since polybori-discuss)
In addition (see http://developers.sun.com/solaris/articles/cmp_stlport_libCstd.html) in order to have C++98 available, one has to build with -library=stlport4 .
But there are still some lines of code (related to default template parameters), which do not compile using the solaris compiler. I'll try to sort this out.
Regards,
Alexander
comment:5 Changed 12 years ago by
- Cc polybori@… added
comment:6 Changed 11 years ago by
- Description modified (diff)
comment:7 Changed 10 years ago by
- Report Upstream changed from Reported upstream. Little or no feedback. to Reported upstream. No feedback yet.
comment:8 Changed 10 years ago by
- Cc AlexanderDreyer added
- Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, in a later stable release.
- Status changed from new to needs_info
The issue was discussed in polybori-discuss and the flags were corrected upstream in the meantime. I have no knowledge whether recent spkgs were tested w.r.t. sun's compiler.
comment:9 Changed 9 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:10 Changed 8 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:11 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:12 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:13 Changed 7 years ago by
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Reviewers set to Jeroen Demeyer
- Status changed from needs_info to positive_review
Let's assume this is fixed by the new brial
package.
comment:14 Changed 7 years ago by
- Resolution set to fixed
- Status changed from positive_review to closed
I emailed this to polybori-discuss at lists.sourceforge.net on 27th November 2009.