Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PSP3] Adapting to new LASTools version #8173

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

soesau
Copy link
Member

@soesau soesau commented Apr 29, 2024

Summary of Changes

Switching to the official LAStools repository

Release Management


Edit, @lrineau, 2024/05/24: depends on LAStools/LAStools#194, that has been merged.

@lrineau
Copy link
Member

lrineau commented Apr 29, 2024

TODO: document that the "last" version of LASTools is required, and precise since which version it will work.

@lrineau lrineau added the TODO label Apr 29, 2024
@lrineau lrineau added this to the 6.0-beta milestone May 23, 2024
@sloriot
Copy link
Member

sloriot commented May 26, 2024

This PR makes the LASTools distributed by CGAL incompatible with CGAL-6.0 (see here for example). Any chance to fix that ? The reason why it would be nice to have is that we would install a single version on test platforms and it would work both with master and the release branches.

@sloriot
Copy link
Member

sloriot commented May 27, 2024

Actually, if this PR target 5.5.x-branch then we would be good with test platforms being updated.

@lrineau lrineau modified the milestones: 6.0-beta, 6.1-beta May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LAStools support
3 participants