[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: PDAL with E57 support



On 2/11/20 5:02 PM, Howard Butler wrote:
>> On Feb 11, 2020, at 9:26 AM, Sebastiaan Couwenberg <sebastic@xs4all.nl> wrote:
>>
>> On 2/11/20 2:03 PM, José Miguel Gonçalves wrote:
>>> Is this lack of E57 support on purpose or something that was missed in
>>> the package build?
>>
>>> From the build log:
>>
>> -- The following features have been disabled:
>>
>> ...
>>   * E57 plugin, read/write data to and from e57 format
>>
>> https://buildd.debian.org/status/fetch.php?pkg=pdal&arch=amd64&ver=2.0.1%2Bds-1%2Bb1&stamp=1578659865&raw=0
>>
>> The CMakeLists.txt for the plugin shows that it requires XercesC which
>> is not documented:
>>
>> https://pdal.io/development/compilation/dependencies.html
> 
> I'll make a pass through this doc.
> 
> E57 didn't configure correctly as released anyway IIRC, but we pushed some fixes to master and the maintenance branch to fix it up.

Let's see what happens sooner, the python3.8 transition or PDAL 2.1 release.

>> The plugin also needs to be explicitly enabled which the package did not
>> do, since most plugins require dependencies that are not available in
>> Debian.
>>
>> The pdal package build is currently broken due to python3.8, so we
>> cannot test the changes for the E57 plugin.
> 
> Is this something that PDAL needs to fix?

I think it's a CMake issue, it picks up python3.8 despite it not being
the default yet. python3.8 is only partially pulled in via some of the
build dependencies, pdal itself only build depends on python3-dev which
provides 3.7 for now.

> btw, for PDAL 2.1, we are removing the Python embedded stuff entirely off to the PDAL extension. The setup.py of the extension will install those embedded plugins in the PDAL_DRIVER_PATH if it is set or fetchable via pdal-config

I known, and am looking forward to it.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1


Reply to: