pdal version 1.1.0 is available, the port should be updated. This is also needed because old version doesn't support upcoming boost 1.60 (though it may be mitigated by switching to embedded boost, it may cause conflicts for consumers).
(see https://wiki.freebsd.org/BoostPortingProject/1.55-to-1.60 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=199601 )
Created attachment 167742 [details] Update pdal to 1.1.0
Hello, Can you check if it's ok with this patch? Thanks.
Testbuild on 9.3a fails, see http://people.freebsd.org/~pi/logs/math__pdal-93a-1457284628.txt 11a, 10.2a look fine, 10.2i still running.
9.3 is failing because the port uses headers not present in base's libstdc++. Try USES=compiler:c++11-lib instead. While here, I think files/patch-apps_CMakeLists.txt can be replaced by USES=pathfix.
I tried to build it on 9.3a with your suggested changes, fails to build in poudriere, see http://people.freebsd.org/~pi/logs/math__pdal-93a Any ideas ?
(In reply to Kurt Jaeger from comment #6) > I tried to build it on 9.3a with your suggested changes, fails to build in > poudriere, see > > http://people.freebsd.org/~pi/logs/math__pdal-93a > > Any ideas ? This looks like yet another case of bug 193528. This should fix it: # Unhide std::to_string() to fix build with GCC (see ports/193528 for details) CXXFLAGS+= -D_GLIBCXX_USE_C99
Thanks, testbuild@work
(In reply to Raphael Kubo da Costa from comment #7) I tried it, fails for 9.3a again. See http://people.freebsd.org/~pi/logs/math__pdal-93a-v2 But this time it looks like a type mismatch.
Created attachment 168587 [details] 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@tuxfamily.org, please take a look at this new patch and let me know if it's OK for you.
testbuilds@work
Testbuilds are all fine
I'll go ahead and commit my version of the update patch. Kurt and I have both verified it works.
A commit references this bug: Author: rakuco Date: Mon Mar 28 19:21:18 UTC 2016 New revision: 412068 URL: https://svnweb.freebsd.org/changeset/ports/412068 Log: Update to 1.1.0. Includes compatibility with the upcoming Boost 1.60. Original patch from the maintainer himself, and additional adjustments by me. Also thanks to pi@ for helping test-build the update. PR: 207059 Submitted by: coder@tuxfamily.org (maintainer, first version) Approved by: coder@tuxfamily.org (maintainer) Changes: head/math/pdal/Makefile head/math/pdal/distinfo head/math/pdal/files/patch-git_080e614 head/math/pdal/files/patch-git_3c7e997 head/math/pdal/files/patch-git_5a0b123 head/math/pdal/files/patch-git_c8a1f88 head/math/pdal/files/patch-src-Utils.cpp head/math/pdal/files/patch-src_PluginManager.cpp head/math/pdal/files/patch-vendor_gtest-1.7.0_include_gtest_internal_gtest-port.h head/math/pdal/pkg-plist
Thanks everyone!