Task #10159 (closed)
Opened 7 years ago
Closed 5 years ago
Bug: OmeroCpp: Wrong C++ compiler version detected
| Reported by: | rleigh | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 5.x |
| Component: | OmeroCpp | Version: | 4.4.9 |
| Keywords: | n.a. | Cc: | cpp@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description (last modified by rleigh)
The default compiler version is 4.7.2. The actual compiler used for building is 4.4.7. While 4.4 is definitely used by scons for the build, it detects the default 4.7.2 as the compiler version (which is wrong), and this version is erroneously embedded in the name of the generated zipfile.
I can't see in the source where CXXVERSION is actually set for real; is this done by scons? CXXVERSION should be derived from CXX.
$ g++-4.4 --version g++-4.4 (Debian 4.4.7-2) 4.4.7 $ g++ --version g++ (Debian 4.7.2-4) 4.7.2 $ c++ --version c++ (Debian 4.7.2-4) 4.7.2 CXX=g++-4.4
$ ./build.py build-cpp ==> components/tools/OmeroCpp/scons.log 'CXX': 'g++-4.4', 'CXXCOM': '$CXX -o $TARGET -c $CXXFLAGS $CCFLAGS $_CCCOMCOM $SOURCES', 'CXXCOMSTR': 'Compiling $TARGET', 'CXXFILESUFFIX': '.cc', 'CXXFLAGS': [], 'CXXVERSION': '4.7.2', ==> components/tools/OmeroCpp/compiler.log # # Scons Compile Log : Thu Jan 10 17:49:59 2013 # CPPFLAGS=['-Wall', '-ansi', '-Wno-long-long', '-Wnon-virtual-dtor', '-Wno-unused-parameter', '-Wno-unused-function', '-Wunused-variable', '-Wunused-value', '-O0', '-g'] CPPPATH=['src', 'target', '/mnt/rleigh/code/ome/components/blitz/generated', '/usr/share/Ice/include'] CXX=gcc CXXVERSION=4.7.2 LIBPATH=['/mnt/rleigh/code/ome/components/tools/OmeroCpp', '/usr/share/Ice/lib'] ARCH=64 RELEASE=dbg PLATFORM=posix
Change History (14)
comment:1 Changed 7 years ago by rleigh
- Description modified (diff)
comment:2 Changed 7 years ago by jmoore
comment:3 Changed 7 years ago by jmoore
- Cc jmoore added
comment:4 Changed 7 years ago by rleigh
This is a scons bug:
http://scons.tigris.org/ds/viewMessage.do?dsForumId=1271&dsMessageId=52948
http://markmail.org/thread/dsc4ci66vnjq6efl
Could we update the version of scons we use to see if it fixes this?
comment:5 Changed 7 years ago by jmoore
I'd think that's appropriate, Roger. You up for doing it?
comment:6 Changed 7 years ago by rleigh
No problem.
comment:7 Changed 7 years ago by rleigh
Still broken even with the scons 2.2.0 update...
comment:8 Changed 7 years ago by jmoore
Referencing ticket #1835 has changed sprint.
comment:9 Changed 7 years ago by jmoore
- Milestone changed from Unscheduled to OMERO-5
- Owner set to jcorrington-x
comment:10 Changed 6 years ago by jmoore
- Cc cpp@… added; jmoore removed
- Owner jcorrington-x deleted
comment:11 Changed 6 years ago by jamoore
- Milestone changed from OMERO-5 to OMERO-4.4.8
comment:12 Changed 6 years ago by jamoore
Referencing ticket #1835 has changed sprint.
comment:13 Changed 6 years ago by jamoore
- Milestone changed from OMERO-4.4.x to 5.0.0
- Version set to 4.4.9
comment:14 Changed 5 years ago by rleigh
- Resolution set to fixed
- Status changed from new to closed
Fixed by move to cmake.
Roger: off-hand I don't have any pointers, so this is likely scons internal. Though there are a few items possibly of interest from "git grep -i cxx".