New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #24197 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

simage: simage.pc and simage-default.cfg contain -arch flags

Reported by: ryandesign@… Owned by: css@…
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

simage-archflags.diff (484 bytes) - added by ryandesign@… 3 years ago.
proposed patch

Change History

Changed 3 years ago by ryandesign@…

proposed patch

comment:1 Changed 3 years ago by css@…

  • Status changed from new to closed
  • Resolution set to fixed

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.