Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#17551 closed defect (fixed)

coin needs updated X11 dependencies

Reported by: jeremyhu (Jeremy Huddleston Sequoia) Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version: 1.6.0
Keywords: X11 Cc: cssdev
Port: Coin

Description

Index: Portfile
===================================================================
--- Portfile	(revision 42954)
+++ Portfile	(working copy)
@@ -17,9 +17,13 @@
 checksums		md5 1e7cb6afa69f6eed034a64021f7d1bfc \
 			sha1 19dc61398d843bff120aec9c640ea814c827b577
 
-depends_build		lib:libX11.6:XFree86 bin:doxygen:doxygen
-depends_lib		port:simage
+depends_build bin:doxygen:doxygen
 
+depends_lib \
+	port:simage \
+	lib:libSM.6:xorg-libsm \
+	lib:libXext.6:xorg-libXext
+
 use_configure		yes
 configure.args		--with-simage=${prefix} --mandir=${prefix}/share/man
 
@@ -30,7 +34,8 @@
 variant aqua requires darwin {
 	configure.args-delete	--without-framework --enable-darwin-x11
 	configure.args-append	--with-framework=${destroot}/Library/Frameworks  --without-x
-	depends_build-delete	lib:libX11.6:XFree86
+	depends_build-delete	lib:libSM.6:xorg-libsm
+	depends_build-delete	lib:Xext.6:xorg-libXext
 }
 
 variant devel {

Attachments (1)

patch-Portfile.diff (1.8 KB) - added by cssdev 15 years ago.
Updated dependencies and added variant descriptions

Download all attachments as: .zip

Change History (11)

comment:1 Changed 15 years ago by cssdev

Status: newassigned

Bugs need to provide more than just the patch ... what failed? Which OS version and hardware are you using? If there are related tickets describing the updated dependencies, please link to them.

comment:2 Changed 15 years ago by cssdev

(or I could wade through that big unread X11 mail thread... !) :)

comment:3 Changed 15 years ago by cssdev

I'm still unclear ... why the change in dependencies?

comment:4 Changed 15 years ago by jeremyhu (Jeremy Huddleston Sequoia)

1) Your X11 dependency is just a buildtime dependency, but you actually need it at runtime as well. 2) You are not depending on libSM, but you need libSM 3) You are not depending on libXext, but you need libXext

Also, I think you should probably change the doxygen dependency to port:doxygen.

comment:5 Changed 15 years ago by jeremyhu (Jeremy Huddleston Sequoia)

See, libX11 is not enough:

$ otool -L /opt/local/lib/libCoin.dylib 
/opt/local/lib/libCoin.dylib:
	/opt/local/lib/libCoin.40.dylib (compatibility version 46.0.0, current version 46.0.0)
	/usr/X11/lib/libGL.1.dylib (compatibility version 1.2.0, current version 1.2.0)
	/usr/X11/lib/libXext.6.dylib (compatibility version 11.0.0, current version 11.0.0)
	/usr/X11/lib/libXdmcp.6.dylib (compatibility version 7.0.0, current version 7.0.0)
	/usr/X11/lib/libXau.6.dylib (compatibility version 7.0.0, current version 7.0.0)
	/usr/X11/lib/libSM.6.dylib (compatibility version 7.0.0, current version 7.0.0)
	/usr/X11/lib/libICE.6.dylib (compatibility version 10.0.0, current version 10.0.0)
	/usr/X11/lib/libX11.6.dylib (compatibility version 9.0.0, current version 9.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 111.0.0)
	/usr/lib/libstdc++.6.dylib (compatibility version 7.0.0, current version 7.4.0)
	/usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)

libSM depends on libICE

libXext depends on libX11 which depends on libXau and libXdmcp

So you need libSM and libXext.

comment:6 Changed 15 years ago by cssdev

Ah, so that helps to make sure everything is thoroughly covered. Thanks for the proactivity and providing the details!

I have a build running at the moment...

comment:7 Changed 15 years ago by cssdev

I might be able to remove doxygen as a default dependency, because it's really only used when building the documentation.

Changed 15 years ago by cssdev

Attachment: patch-Portfile.diff added

Updated dependencies and added variant descriptions

comment:8 Changed 15 years ago by jeremyhu (Jeremy Huddleston Sequoia)

That looks good to me. Push it =)

comment:9 Changed 15 years ago by cssdev

Resolution: fixed
Status: assignedclosed

Committed to r43863!

comment:10 Changed 15 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.