Opened 12 years ago

Closed 12 years ago

#31478 closed defect (fixed)

expect installs broken libexpect.a symlink

Reported by: ryandesign (Ryan Carsten Schmidt) 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 (3)

comment:1 Changed 12 years 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 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

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 12 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

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.