Opened 13 years ago
Closed 13 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 13 years ago by markd@…
comment:2 follow-up: 3 Changed 13 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 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Replying to ryandesign@…:
either the symlink should be updated to point to the current library location
Done in r92640.
Not sure what the purpose of it was. Ryan, do you know? If not, we could just remove the symlink statement from post-destroot.