GNU bug report logs - #70248
[PATCH] gnu: Add pdal.

Previous Next

Package: guix-patches;

Reported by: Vinicius Monego <monego <at> posteo.net>

Date: Sat, 6 Apr 2024 23:02:01 UTC

Severity: normal

Tags: patch

Done: Vinicius Monego <monego <at> posteo.net>

To reply to this bug, email your comments to 70248 AT debbugs.gnu.org.
There is no need to reopen the bug first.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#70248; Package guix-patches. (Sat, 06 Apr 2024 23:02:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Vinicius Monego <monego <at> posteo.net>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Sat, 06 Apr 2024 23:02:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Vinicius Monego <monego <at> posteo.net>
To: guix-patches <at> gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH] gnu: Add pdal.
Date: Sat,  6 Apr 2024 23:00:21 +0000
* gnu/packages/geo.scm (pdal): New variable.

Change-Id: I7c24fec69b4f0c19a1c246fab39b994e17e36f3a
---
 gnu/packages/geo.scm | 41 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/gnu/packages/geo.scm b/gnu/packages/geo.scm
index 4884a3803b..95ce83c869 100644
--- a/gnu/packages/geo.scm
+++ b/gnu/packages/geo.scm
@@ -101,6 +101,7 @@ (define-module (gnu packages geo)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages java)
   #:use-module (gnu packages kde)
+  #:use-module (gnu packages libunwind)
   #:use-module (gnu packages libusb)
   #:use-module (gnu packages linux)
   #:use-module (gnu packages lua)
@@ -1229,6 +1230,46 @@ (define-public spatialite-gui
     (home-page "https://www.gaia-gis.it/fossil/spatialite_gui/index")
     (license license:gpl3+)))
 
+(define-public pdal
+  (package
+    (name "pdal")
+    (version "2.7.1")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/PDAL/PDAL")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0gg5lcshlmn3wwak42xr0b8a8gdr4572d7hrcvxn2291kp2c3096"))))
+    (build-system cmake-build-system)
+    (arguments
+     (list #:phases
+           #~(modify-phases %standard-phases
+               (replace 'check
+                 (lambda* (#:key tests? #:allow-other-keys)
+                   (when tests?
+                     (invoke "ctest" "-E" ;; This test hangs.
+                             "pdal_io_stac_reader_test")))))))
+    (native-inputs (list python))
+    (inputs (list gdal
+                  h3
+                  libgeotiff
+                  libunwind
+                  libxml2
+                  nlohmann-json
+                  proj
+                  utfcpp
+                  xz
+                  `(,zstd "lib")))
+    (home-page "https://pdal.io/")
+    (synopsis "Point Data Abstraction Library")
+    (description "PDAL is a C++ library for translating and manipulating point
+cloud data.  It is very much like the GDAL library which handles raster and
+vector data.")
+    (license license:bsd-3)))
+
 (define-public gdal
   (package
     (name "gdal")

base-commit: 12fd64c9f952189d82a8a6ba91f2178c479eb890
-- 
2.39.2





Information forwarded to guix-patches <at> gnu.org:
bug#70248; Package guix-patches. (Mon, 08 Apr 2024 13:54:02 GMT) Full text and rfc822 format available.

Message #8 received at 70248 <at> debbugs.gnu.org (full text, mbox):

From: Guillaume Le Vaillant <glv <at> posteo.net>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 70248 <at> debbugs.gnu.org
Subject: Re: [bug#70248] [PATCH] gnu: Add pdal.
Date: Mon, 08 Apr 2024 13:53:38 +0000
[Message part 1 (text/plain, inline)]
LGTM.
(I saw that some optional features are disabled because hdf, openssl or
postgresql are not in the inputs, but I don't know if these features are
important for usual users of PDAL).
[signature.asc (application/pgp-signature, inline)]

Reply sent to Vinicius Monego <monego <at> posteo.net>:
You have taken responsibility. (Tue, 09 Apr 2024 00:34:02 GMT) Full text and rfc822 format available.

Notification sent to Vinicius Monego <monego <at> posteo.net>:
bug acknowledged by developer. (Tue, 09 Apr 2024 00:34:02 GMT) Full text and rfc822 format available.

Message #13 received at 70248-done <at> debbugs.gnu.org (full text, mbox):

From: Vinicius Monego <monego <at> posteo.net>
To: Guillaume Le Vaillant <glv <at> posteo.net>
Cc: 70248-done <at> debbugs.gnu.org
Subject: Re: [bug#70248] [PATCH] gnu: Add pdal.
Date: Tue,  9 Apr 2024 00:33:05 +0000
Em 08/04/2024 10:53, Guillaume Le Vaillant escreveu:
> LGTM.
> (I saw that some optional features are disabled because hdf, openssl or
> postgresql are not in the inputs, but I don't know if these features are
> important for usual users of PDAL).

OpenSSL is for access to Google Cloud Storage.

While I don't think it's too important, the size of openssl doesn't seem 
to hurt, so I added it to the inputs. I don't know what is the purpose 
of hdf or postgres in PDAL.

There are are a couple other disabled features which I have no idea what 
they are. I pushed the patch with the addition of openssl to the inputs.

PDAL is a mandatory dependency for the latest versions of QGIS, which is 
why I am packaging it.

Thanks,

Vinicius





This bug report was last modified 25 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.