Opened 14 years ago

Closed 14 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)

simage-archflags.diff (484 bytes) - added by ryandesign (Ryan Carsten Schmidt) 14 years ago.
proposed patch

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: simage-archflags.diff added

proposed patch

comment:1 Changed 14 years ago by cssdev

Resolution: fixed
Status: newclosed

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.