New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #33559 (closed defect: fixed)

Opened 15 months ago

Last modified 15 months ago

levmar: port test fails to find local library

Reported by: lockhart@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.0.4
Keywords: haspatch maintainer Cc: ryandesign@…, pixilla@…
Port: levmar

Description

The current levmar port fails to run "port test" if the port is not already installed. The locally-built lmdemo program does not find liblevmar.2.dylib since the install_name for the library is @rpath/liblevmar.2.dylib and the required rpath is not known to lmdemo.

Attachments

Portfile.lmdemo.patch (1.3 KB) - added by lockhart@… 15 months ago.
Patch on the latest Portfile to allow "port test" to run successfully. Also requires slight change to the Makefile.so patch which is posted here separately.
patch-Makefile.so.diff (2.4 KB) - added by lockhart@… 15 months ago.
Replacement for current patch file to allow running lmdemo locally for "port test". Adds a local soft link for the major-versioned library.

Change History

Changed 15 months ago by lockhart@…

Patch on the latest Portfile to allow "port test" to run successfully. Also requires slight change to the Makefile.so patch which is posted here separately.

Changed 15 months ago by lockhart@…

Replacement for current patch file to allow running lmdemo locally for "port test". Adds a local soft link for the major-versioned library.

comment:1 in reply to: ↑ description Changed 15 months ago by lockhart@…

Replying to lockhart@…:

The current levmar port fails to run "port test" if the port is not already installed. The locally-built lmdemo program does not find liblevmar.2.dylib since the install_name for the library is @rpath/liblevmar.2.dylib and the required rpath is not known to lmdemo.

The Portfile patch and replacement for the makefile patch allow "port test" to run successfully. afaict this is ready to publish. Thanks!

comment:2 Changed 15 months ago by pixilla@…

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

Thanks Tom. See r90600

comment:3 Changed 15 months ago by pixilla@…

  • Keywords maintainer added

comment:4 follow-up: ↓ 5 Changed 15 months ago by pixilla@…

Tom, it looked like this patch "may" result in changes to installed files so added a revbump.

comment:5 in reply to: ↑ 4 Changed 15 months ago by lockhart@…

Replying to pixilla@…:

Tom, it looked like this patch "may" result in changes to installed files so added a revbump.

OK. I'd be putting a rev number into every one of these packages.

Note: See TracTickets for help on using tickets.