Opened 15 years ago
Closed 15 years ago
#24197 closed defect (fixed)
simage: simage.pc and simage-default.cfg contain -arch flags
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | cssdev |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.8.2 |
Keywords: | Cc: | ||
Port: | simage |
Description
$ port installed simage The following ports are currently installed: simage @1.6.1_4+universal (active) $ port contents simage | sed 1d | xargs grep -- -arch /opt/local/share/Coin/conf/simage-default.cfg:ldflags="-L${exec_prefix}/lib -arch x86_64 -arch i386"
$ port installed simage The following ports are currently installed: simage @1.7.0_0+universal (active) $ port contents simage | sed 1d | xargs grep -- -arch /opt/local/lib/pkgconfig/simage.pc:Libs: -L${libdir} -L${exec_prefix}/lib -arch x86_64 -arch i386 -lsimage -logg -lvorbis -lvorbisfile -Wl,-framework,CoreFoundation -Wl,-framework,ApplicationServices /opt/local/share/Coin/conf/simage-default.cfg:ldflags="-L${exec_prefix}/lib -arch x86_64 -arch i386"
Those -arch flags should not be there; they will cause problems for other ports using simage as a dependency. The attached patch should fix this. If you don't combine this with a version update, don't forget to increase the revision as well.
Attachments (1)
Change History (2)
Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | simage-archflags.diff added |
---|
comment:1 Changed 15 years ago by cssdev
Resolution: | → fixed |
---|---|
Status: | new → closed |
Thanks for the patch! Fixed in r65328. I've never known much about pkgconfig, so I'll try to keep an eye out for that in the future.
Note: See
TracTickets for help on using
tickets.
proposed patch