Opened 7 years ago
Closed 7 years ago
#55346 closed update (fixed)
LASzip @2.2.0: update to 3.1.1
Reported by: | l2dy (Zero King) | Owned by: | Veence (Vincent) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | |
Port: | LASzip |
Description
It compiles, but...
$ otool -D /opt/local/lib/liblaszip.dylib /opt/local/lib/liblaszip.dylib: @executable_path/../lib/liblaszip.3.dylib $ otool -D /opt/local/lib/liblaszip_api.dylib /opt/local/lib/liblaszip_api.dylib: @executable_path/../lib/liblaszip_api.3.dylib
Attachments (3)
Change History (11)
Changed 7 years ago by l2dy (Zero King)
Attachment: | 0001-LASzip-update-to-3.1.1.patch added |
---|
comment:1 Changed 7 years ago by l2dy (Zero King)
Keywords: | haspatch added |
---|
comment:2 follow-up: 3 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | LASzip-3.1.1.diff added |
---|
Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | install_name.patch added |
---|
comment:3 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Replying to ryandesign:
I'm more concerned that the library version has decreased from 6 (in version 2.2.0) to 3 (in 3.1.1).
comment:4 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Owner: | set to Veence |
---|---|
Status: | new → assigned |
laszip was updated to 3.2.0 in [e3710d6800e803ebaa9528d3bdb38fb2fcade513/macports-ports], without fixing the library version problem or the install_name problem. When either or both of these problems are fixed, the ports that link with the laszip library need to be rebuilt by increasing their revisions.
comment:5 Changed 7 years ago by Veence (Vincent)
I’ve committed a fix to the wrong library ID problem, and revbump'ed saga and pdal which depend on it. I’ve no idea what to do for the version mishandling, though.
comment:6 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
laszip 3.2.1 has been released which should fix the library version, setting it to 8. However, the laszip 3.2.1 release files are named wrong, with version number 3.2.0 instead of 3.2.1; let's wait and see if upstream will fix that before we update.
comment:7 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
comment:8 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
That's easy enough to fix. I'm more concerned that the library version has decreased from 6 (in version 2.2.0) to 3 (in 3.1.1).
The two programs, laszip-config and laszippertest, are also no longer included in 3.1.1. They're in the examples directory now, but not installed anymore. Not sure if we should try to install them.