Skip to content

Conversation

VictorLamoine
Copy link
Contributor

Intents to replace http://www.pointclouds.org/downloads/source.html
The links at http://www.pointclouds.org/downloads/ should be updated (don't forget #839 as well)

The tutorial is nearly a copy, I changed the version number and the formatting is not always the same.
There is no logo for POSIX so I kept the default PCL logo. Using a Tux logo is not a good idea because the tutorial also targets MacOS X users etc.

@VictorLamoine
Copy link
Contributor Author

Looks ok?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.7.2

@taketwo
Copy link
Member

taketwo commented Jan 28, 2015

Hmm, indeed!

taketwo added a commit that referenced this pull request Jan 28, 2015
Add building tutorial for POSIX systems
@taketwo taketwo merged commit f6ea211 into PointCloudLibrary:master Jan 28, 2015
@soyersoyer
Copy link
Contributor

Boost 1.54? Not 1.46?

@VictorLamoine
Copy link
Contributor Author

If building with OpenNI boost minimum version is 1.47 (first version with chrono)
Without OpenNI boost minimum version is 1.40

See #276

I'll wait for the tutorial to be online and fix the errors.

@VictorLamoine
Copy link
Contributor Author

@jspricke @taketwo Something must be wrong with the tutorial auto-update because the tutorial is not yet online.
I can generate the tutorial on my side, it looks ok.

@VictorLamoine
Copy link
Contributor Author

Friendly ping! 👻

@taketwo
Copy link
Member

taketwo commented Feb 7, 2015

Should be ok now. BTW, it occurred to me that it might make sense to rename it into "Compiling PCL from source on POSIX compliant systems" so that it stands in line with tutorials for the other systems. (The source file also, "compiling_pcl_posix.rst".)

@VictorLamoine
Copy link
Contributor Author

Yes it's online thanks
Ok I'll do that with the other fixes (boost version, maybe dead links)

Todo list

@taketwo
Copy link
Member

taketwo commented Feb 7, 2015

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants