New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #31478 (closed defect: fixed)

Opened 20 months ago

Last modified 13 months ago

expect installs broken libexpect.a symlink

Reported by: ryandesign@… Owned by: markd@…
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: Cc:
Port: expect

Description

expect installs a libexpect.a symlink pointing to a file that doesn't exist, at least not at the location it's using:

$ port -q contents expect | grep /lib/ | xargs ls -l
-rw-r--r--  1 root  wheel  477592 Oct  1 18:05 /opt/local/lib/expect5.45/libexpect5.45.a
lrwxr-xr-x  1 root  wheel      15 Oct  1 18:05 /opt/local/lib/libexpect.a -> libexpect5.45.a

Change History

comment:1 Changed 20 months ago by markd@…

Not sure what the purpose of it was. Ryan, do you know? If not, we could just remove the symlink statement from post-destroot.

comment:2 follow-up: ↓ 3 Changed 20 months ago by ryandesign@…

Looks like gwright made this change in r20658. It doesn't say why but presumably the purpose of the symlink is so that other programs can link with expect by using "-lexpect" so that they don't have to know what version of expect is installed; this seems reasonable. Presumably at the time the link was pointing to the library, and in some subsequent version of expect the library got moved into that subdirectory, and the symlink was not updated. I would say either the symlink should be updated to point to the current library location, or the current library location should be changed back to what it was at that time (directly in ${prefix}/lib). I'm not sure why having an intermediate directory would be useful and I could not immediately locate any mention of that change in the upstream ChangeLog.

comment:3 in reply to: ↑ 2 Changed 13 months ago by ryandesign@…

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

Replying to ryandesign@…:

either the symlink should be updated to point to the current library location

Done in r92640.

Note: See TracTickets for help on using tickets.