#40038 closed submission (fixed)
Port Submission: libLAS - LAS 1.0/1.1/1.2 ASPRS LiDAR data translation toolset
Reported by: | venabled@… | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | ||
Port: | liblas |
Description
This is a new port submission for libLAS - LAS 1.0/1.1/1.2 ASPRS LiDAR data translation toolset, a C/C++ Library for manipulating data from Laser Scanners.
Attachments (3)
Change History (13)
comment:1 Changed 11 years ago by venabled@…
comment:2 follow-up: 3 Changed 11 years ago by mf2k (Frank Schima)
Keywords: | gis removed |
---|---|
Version: | 2.2.0 |
Thanks for the submission. Here are some quick notes:
- The revision line needs to be deleted so it starts at 0.
- Since it uses cmake, the cmake portgroup should be used instead.
- When using a portgroup, the depends_lib line should be changed to depends_lib-append to not overwrite dependencies added from the portgroup.
- A sha256 checksum should be added and possibly replace the sha1 checksum.
- Formatting could be improved to be consistent.
- The first line should not start with any information. Change it to:
# $Id:$
comment:3 Changed 11 years ago by venabled@…
I made a revision according to the suggestions (hopefully correctly). I cleaned and resinstalled/tested.
comment:4 follow-up: 5 Changed 11 years ago by mf2k (Frank Schima)
Since you are using the cmake portgroup, you should be able to remove the 3 configure
lines.
Changed 11 years ago by venabled@…
Modified portfile, made changes according to macsforever2000@ ...
comment:5 Changed 11 years ago by venabled@…
Replying to macsforever2000@…:
Since you are using the cmake portgroup, you should be able to remove the 3
configure
lines.
I went ahead and removed 2 of the lines, and changed the last configure line from configure.args to configure.args-append with the specific flags for compiling against GeoTIFF
comment:6 Changed 11 years ago by mf2k (Frank Schima)
Finally, can you please determine and add the license line? Sorry for not mentioning this sooner.
comment:8 Changed 11 years ago by mf2k (Frank Schima)
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:9 Changed 11 years ago by venabled@…
Sorry, quick correction. LibLas is BSD licensed. I was working with another portfile simultaneously and got them confused.
Attached Portfile