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.

Sign up for GitHub

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

Jump to bottom

Non existing link to the LASLib / LAStools #7770

Closed
albert-github opened this issue Oct 9, 2023 · 3 comments · Fixed by #7778
Closed

Non existing link to the LASLib / LAStools #7770

albert-github opened this issue Oct 9, 2023 · 3 comments · Fixed by #7778
Labels
Doc bug
Milestone
6.0-beta

Comments

@albert-github
Copy link
Contributor

In Documentation/doc/Documentation/Third_party.txt(Manual/thirdparty.html) there is a link to:

https://rapidlasso.com/lastools

but this link does not exist anymore.
@lrineau
Copy link
Member

lrineau commented Oct 9, 2023

Maybe https://lastools.github.io/

albert-github added a commit to albert-github/cgal that referenced this issue Oct 10, 2023
@albert-github
issue CGAL#7770 Non existing link to the LASLib / LAStools
930387e 
Corrected link.
@albert-github
Copy link
Contributor Author

I've just pushed a proposed patch, pull request #7778

@albert-github
Copy link
Contributor Author

Note

In the documentation it is mentioned:

LASlib is usually distributed along with LAStools: for simplicity, CGAL provides a fork with a CMake based install procedure.

Though this fork is a bit behind the master:
image

so maybe some merging should be done or alternatively completely removed and just a link to the official version?

albert-github added a commit to albert-github/cgal that referenced this issue Oct 10, 2023
@albert-github
issue CGAL#7770 Non existing link to the LASLib / LAStools
90050dc 
Removed trailing space
@lrineau lrineau linked a pull request Oct 13, 2023 that will close this issue
issue #7770 Non existing link to the LASLib / LAStools #7778
Merged
lrineau added a commit that referenced this issue Oct 19, 2023
@lrineau
Merge pull request #7778 from albert-github/feature/issue_7770
9187974 
issue #7770 Non existing link to the LASLib / LAStools
@lrineau lrineau added this to the 6.0-beta milestone Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Doc bug
Projects
None yet
Milestone
6.0-beta
Development

Successfully merging a pull request may close this issue.

issue #7770 Non existing link to the LASLib / LAStools albert-github/cgal
2 participants
@albert-github @lrineau