Paths

Table of Contentst

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

	Unknown Object (File)
	Tue, Nov 25, 6:16 AM

	Unknown Object (File)
	Mon, Nov 24, 7:49 PM

	Unknown Object (File)
	Sun, Nov 23, 5:11 AM

	Unknown Object (File)
	Fri, Nov 21, 2:37 AM

	Unknown Object (File)
	Wed, Nov 19, 8:49 AM

	Unknown Object (File)
	Sun, Nov 16, 10:13 AM

	Unknown Object (File)
	Sun, Nov 16, 3:24 AM

	Unknown Object (File)
	Wed, Nov 12, 8:01 AM

View All 100 Files

Subscribers

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:

https://github.com/PDAL/PDAL/releases/tag/2.1.0

Test Plan

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

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 32622
Build 30078: arc lint + arc unit

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
56	^ 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

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	^`-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

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	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

Path

Size

math/

pdal/

61 lines

6 lines

files/

patch-cmake_modules_FindLASzip.cmake

184 lines

Diff 75081

math/pdal/Makefile

Loading...

math/pdal/distinfo

Loading...

math/pdal/files/patch-cmake_modules_FindLASzip.cmake

Loading...

math/pdal/pkg-plist

Loading...