Fix dead links in Walkthrough tutorial

[taketwo]

Fixes #2517.

Not sure what to do about "Range Image library". We don't have such library anymore 😅

[SergioRAgostinho]

Since we can only provide an up-to-date version of this tutorial I would remove the range image module. The location paths and what-not are also a little bit redundant to have them generated for every single module. Plus, the path prefix mentions pcl-1.6. It should be updated to the current version 1.8(.1).

Edit: So apparently it was not removed but moved to common. Don't remove it but put a note that it is not longer a module and it was merged to common.

[SergioRAgostinho]

Sorry for the extra hassle. When I mentioned to replace the path prefixes my idea was to have CMake automatically filling in that version prefix. Otherwise we will "need" to constantly update those paths with every minor version.

[taketwo]

IMHO that would be too much of a trouble, the usefulness of these paths is questionable anyway. What I can do is replace 1.6 with $(PCL_VERSION).

[SergioRAgostinho]

On 09/10/2018 13:21, Sergey Alexandrov wrote: IMHO that would be too much of a trouble, the usefulness of these paths is questionable anyway. What I can do is replace |1.6| with |$(PCL_VERSION)|. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2532 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABoUVvOue-mQ7z18JP-AtIH3rsVu9Rnxks5ujIbJgaJpZM4XLwYm>.

|PCL_VERSION| also includes the patch version, which is not added to the install prefix. New proposal: erase all the paths. ​

[taketwo]

$(PCL_VERSION) is not really a variable or anything, just a "symbol" that can have any meaning. In this case it will mean "major.minor".

[SergioRAgostinho]

Sounds good. Go with that.

[GeneralAdmin]

Dead link!

[kunaltyagi]

Please create a PR :)