Page MenuHomeFreeBSD
Differential D25863

math/pdal: Update to 2.1.0
ClosedPublic
Actions

Authored by lbartoletti on Jul 28 2020, 3:59 PM.
Tags
None
Referenced Files
F162890109: D25863.id75185.diff
Sat, Jul 18, 1:28 AM
Unknown Object (File)
Tue, Jul 7, 12:43 AM
Unknown Object (File)
Tue, Jul 7, 12:42 AM
Unknown Object (File)
Sat, Jul 4, 7:57 PM
Unknown Object (File)
Thu, Jul 2, 2:06 PM
Unknown Object (File)
Wed, Jul 1, 8:30 AM
Unknown Object (File)
Wed, Jun 24, 12:10 AM
Unknown Object (File)
Tue, Jun 23, 8:33 AM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
swills
Commits
rP543828: math/pdal: Update to 2.1.0
Summary
  • Update to 2.1.0
  • Remove Sqlite and Python (removed upstream)
  • Remove patch for LASZip (fixed in archivers/laszip)
  • Reoreder options
  • Add lazperf option

Changelog:

Test Plan

poudriere (11/12/13 i386/amd64) all options tested

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lbartoletti created this revision.Jul 28 2020, 3:59 PM
Herald added a subscriber: mat. · View Herald TranscriptJul 28 2020, 3:59 PM
lbartoletti requested review of this revision.Jul 28 2020, 3:59 PM
Harbormaster completed remote builds in B32618: Diff 75074.Jul 28 2020, 3:59 PM
tcberner added inline comments.Jul 28 2020, 4:24 PM
math/pdal/Makefile
58 ↗(On Diff #75074)

^

LAZPERF_CMAKE_BOOL=WITH_LAZPERF

applies also for the other cmake-bool options:

https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/makefile-options.html#options-cmake_bool

lbartoletti updated this revision to Diff 75081.Jul 28 2020, 4:59 PM

Use _CMAKE_BOOL instead of _CMAKE_ON

Harbormaster completed remote builds in B32622: Diff 75081.Jul 28 2020, 4:59 PM
tcberner added inline comments.Jul 28 2020, 5:01 PM
math/pdal/Makefile
61 ↗(On Diff #75081)

^-D is too much here

tcberner added a child revision: D25870: [NEW PORT] math/py-pdal: PDAL Python bindings.Jul 30 2020, 4:30 PM
tcberner added a comment.Jul 30 2020, 4:36 PM

Apart from the wrong argument pointed out previously this is ready to ship -- please correct that, and the commit :)

tcberner mentioned this in D25870: [NEW PORT] math/py-pdal: PDAL Python bindings.Jul 30 2020, 4:37 PM
lbartoletti marked an inline comment as done.Jul 30 2020, 7:53 PM
lbartoletti added inline comments.
math/pdal/Makefile
61 ↗(On Diff #75081)

Done. Thanks :p

This revision was not accepted when it landed; it landed in state Needs Review.Jul 30 2020, 7:53 PM
Closed by commit rP543828: math/pdal: Update to 2.1.0 (authored by lbartoletti). · Explain Why
This revision was automatically updated to reflect the committed changes.
lbartoletti marked an inline comment as done.
lbartoletti added a commit: rP543828: math/pdal: Update to 2.1.0.

Revision Contents
Changeset List

PathSize
head/
math/
pdal/
61 lines
6 lines
files/
11 lines
184 lines

Diff 75185

View Options

head/math/pdal/Makefile

Loading...
View Options

head/math/pdal/distinfo

Loading...
View Options

head/math/pdal/files/patch-cmake_modules_FindLASzip.cmake

Loading...
View Options

head/math/pdal/pkg-plist

Loading...