Opened 12 years ago

Closed 12 years ago

#32517 closed defect (fixed)

qhull +debug post-destroot links wrong library

Reported by: michaelld (Michael Dickens) Owned by: mamoll (Mark Moll)
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: haspatch Cc:
Port: qhull

Description (last modified by michaelld (Michael Dickens))

In r87709, various symlinks were added to the post-destroot phase. When installed with +debug, the DYLIB is currently named "libqhull_d6.dylib", while that revision of the Portfile (for whatever reason) used "libqhull6.dylib". This issue is fixed by "s/libqhull6/libqhull_d6/g", but for the +debug variant only, which changes just the symlink on line 41.

Attachments (1)

qhull_Portfile.diff (881 bytes) - added by michaelld (Michael Dickens) 12 years ago.
patch to the qhull Portfile to fix bad dylib symlink

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by michaelld (Michael Dickens)

Attachment: qhull_Portfile.diff added

patch to the qhull Portfile to fix bad dylib symlink

comment:1 Changed 12 years ago by mamoll (Mark Moll)

That's odd. I don't have that dylib. These are the libraries I have:

/opt/local/lib/libqhull.a
/opt/local/lib/libqhull.dylib
/opt/local/lib/libqhull6.6.2.1.1446.dylib
/opt/local/lib/libqhull6.dylib
/opt/local/lib/libqhullcpp.a
/opt/local/lib/libqhullstatic.a
/opt/local/lib/libqhullstatic_p.a

None of the sum links are broken. I am on OS X 10.7.2. I wonder where the "_d" is coming from on your machine.

comment:2 Changed 12 years ago by michaelld (Michael Dickens)

Description: modified (diff)
Summary: qhull post-destroot links wrong libraryqhull +debug post-destroot links wrong library

I just poked around a bit & found out that this issue is with +debug only. So, I've changed this ticket to reflect this new info. The Portfile patch won't work for this case, but it's a pretty simple change to check for that variant. It would be nice if the release and debug libraries were consistently named (e.g., libfoo.dylib and libfoo_debug.dylib), but that would require more post-destroot messing.

comment:3 Changed 12 years ago by mamoll (Mark Moll)

Resolution: fixed
Status: newclosed

Fixed in r87978.

Note: See TracTickets for help on using tickets.