[MacPorts] #31053: qhull fails to build using g++-4.0, e.g., leopard
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: octave gcc | Port: qhull ------------------------------+--------------------------------------------- {{{ [ 48%] Building CXX object CMakeFiles/qhullcpp.dir/src/libqhullcpp/Qhull.cpp.o /usr/bin/g++-4.0 -Dqh_QHpointer -O2 -arch i386 -O3 -DNDEBUG -arch i386 -isysroot / -mmacosx-version-min=10.5 \ -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_math_qhull/qhull/\ work/qhull-2011.1/src/libqhullcpp \ -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_math_qhull/qhull/\ work/qhull-2011.1/src \ -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_math_qhull/qhull/\ work/qhull-2011.1/src/libqhull \ -o CMakeFiles/qhullcpp.dir/src/libqhullcpp/Qhull.cpp.o \ -c /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_math_qhull/qhull/\ work/qhull-2011.1/src/libqhullcpp/Qhull.cpp /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_math_qhull/qhull/\ work/qhull-2011.1/src/libqhullcpp/QhullPointSet.h:114: error: using typedef-name 'orgQhull::QhullSet<double*>::const_iterator' after 'class' }}} also [http://paste.lisp.org/+2NVH]. The author attributes this to a bug in g++-4.0 [http://paste.lisp.org/+2NVH/1] for a patch. Qhull is used, e.g., by octave, which uses gcc44 by default. Should qhull do the same? -- Ticket URL: <https://trac.macports.org/ticket/31053> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Changes (by macsforever2000@…): * keywords: octave gcc => * owner: macports-tickets@… => mmoll@… -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Comment(by mmoll@…): Patch committed in r83413. Please confirm that this works, since I don't have access to a Leopard machine. -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Comment(by gnwiii@…): Replying to [comment:2 mmoll@…]:
Patch committed in r83413. Please confirm that this works, since I don't have access to a Leopard machine.
Thanks for the fast response. There was a minor problem with the change to the Portfile: {{{ port upgrade qhull ---> Computing dependencies for qhull ---> Fetching qhull ---> Attempting to fetch # from http://www.qhull.org/download }}} After editing the Partfile to remove the trailing comment "{{{# patch for g++-4.0 / Leopard, see #31053}}}" I was able to build qhull on Leopard. There was another issue: {{{ ---> Staging qhull into destroot Warning: violation by /opt/local/man Warning: qhull violates the layout of the ports-filesystems! Warning: Please fix or indicate this misbehavior (if it is intended), it will be an error in future releases! }}} I'm guessing the man pages are supposed to go in {{{/opt/local/share/man}}}. For the record, I haven't tested the new version yet because octave-devel looks for {{{qhull/qhull_a.h:}}}, and the headers are now in {{{libqhull}}}. I'll file a report against {{{octave-devel}}}. -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Comment(by mmoll@…): Fix committed in r83436. I created a symlink from ${prefix}/include/libqhull to ${prefix}/include/qhull. Hopefully that's sufficient to stay backwards compatible. -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Comment(by gnwiii@…): Thanks. The symlink may help some other dependent packages, but octave- devel links against {{{libqhull.dylib}}}. I tried a symbolic link {{{libqhull6.dylib --> libqhull.dylib}}}, but the new dylib is called {{{libqhull6.dylib}}}. Another symlink, but then configure wants {{{qhull/qhull.h}}}. The upstream octave bug report neglects to mention this, but does mentions a problem when using qhull-2011.1 -- could the upstream reporter have mixed old and new headers? -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard ------------------------------+--------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Keywords: | Port: qhull ------------------------------+--------------------------------------------- Comment(by takeshi@…): I updated for 2011.2 and created a few symlinks for backward compatibility. A better solution may be update the octave source. I was able to build octave against qhull with this fix. See also #31062. -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
#31053: qhull fails to build using g++-4.0, e.g., leopard -------------------------------+-------------------------------------------- Reporter: gnwiii@… | Owner: mmoll@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.0.1 Resolution: fixed | Keywords: Port: qhull | -------------------------------+-------------------------------------------- Changes (by mmoll@…): * status: new => closed * resolution: => fixed Comment: Committed in r87709. Thanks for the patch! -- Ticket URL: <https://trac.macports.org/ticket/31053#comment:7> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts