search

techoe / ceres-solver

Automatically exported from code.google.com/p/ceres-solver
Other
0 stars 0 forks source link

Build documentation #64

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Patch provided

Being able to generate the documentation by turning ON one option in CMake. The 
new target is added to the ALL target (but can be removed if people preferred).

Original issue reported on code.google.com by arnaudge...@gmail.com on 15 Aug 2012 at 8:00

Attachments:

GoogleCodeExporter commented 9 years ago 
Hi Arnaud,
Thanks for the patch. Two things.

1. I know the gerrit docs are a bit sparse but it would be great to have this 
patch there. Please have a look at the wiki page and see if the instructions 
are enough, keir and I are happy to provide more.
1. Are you assuming that pygmentize is already installed? It would be nice to 
detect its presence and fail early.

Sameer

Original comment by sameerag...@google.com on 15 Aug 2012 at 1:55

GoogleCodeExporter commented 9 years ago 
I have just pushed the patch on gerrit.

I have assumed pygmentize was already installed...

You're right, it should fail early if pygmentize is not installed (but it can 
be done in a subsequent patch)!

Arnaud

Original comment by arnaudge...@gmail.com on 15 Aug 2012 at 2:52

GoogleCodeExporter commented 9 years ago 
done in the last patch on gerrit...

Original comment by arnaudge...@gmail.com on 15 Aug 2012 at 3:21

GoogleCodeExporter commented 9 years ago 
Closing as the rest of the discussion is on gerrit.

Original comment by sameerag...@google.com on 15 Aug 2012 at 4:46