[Bug 207059] math/pdal: Update to 1.1.0

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=207059

Raphael Kubo da Costa <rakuco at FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #168587|                            |maintainer-approval?(coder@
              Flags|                            |tuxfamily.org)

--- Comment #10 from Raphael Kubo da Costa <rakuco at FreeBSD.org> ---
Created attachment 168587
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=168587&action=edit
Patch v2

OK, I've finally taken some time to investigate this and the attached patch
builds on HEAD-amd64 and 9.3-i386.

* Use USES=cmake:outsource instead of USES=cmake. Out-of-source builds are
cleaner and showed that several build directories had been added to the package
and the plist.
* Implemented my suggestions (USES=compiler:c++11-lib and USES=pathfix).
* Those many patches commenting out the portable_endian.hpp were just working
around the fact that portable_endian.hpp was broken and needed to be fixed (the
#defines translating FreeBSD functions into Linux ones were inverted).
* The PointViewTest.cpp patch was also working around a problem in the embedded
gtest, that was lacking a platform definition for FreeBSD.
* The nanoflann.hpp patch only needs the cstdlib include, the rest is not
necessary.

Kurt and coder at tuxfamily.org, please take a look at this new patch and let me
know if it's OK for you.

-- 
You are receiving this mail because:
You are the assignee for the bug.