jose icon indicating copy to clipboard operation
jose copied to clipboard

Add sphinx documentation of jose

Open tiran opened this issue 8 years ago • 10 comments

Jose lacks API documentation. I started sphinx documentation with breathe and doxygen for C API.

Signed-off-by: Christian Heimes [email protected]

tiran avatar Oct 05 '16 09:10 tiran

This looks great! Before I merge, can you add optional doc building/install to autoconf/automake?

npmccallum avatar Oct 05 '16 12:10 npmccallum

I'm not an AM expert but my Google Fu was strong enough to figure it out.

tiran avatar Oct 05 '16 12:10 tiran

We need more than this. Let me define what I'm looking for. We should look for our documentation requirements during configure. If they are found, we should build the documentation.

Additionally, there should be a switch to configure to force on the documentation requirements. See, for instance, --with-openssl.

Finally, this needs to be added to the test suite via the --with-* flags. This will reveal in Travis CI which platforms can't build the docs.

npmccallum avatar Oct 05 '16 12:10 npmccallum

Check for

  • doxygen command
  • either
    • sphinx-build-2 command
    • python2 -c "import breathe"
  • or
    • sphinx-build-3 command
    • python3 -c "import breathe"
  • run make -C docs html SPHINXBUILD=sphinx-build-[23]

tiran avatar Oct 05 '16 13:10 tiran

Can you rebase this?

npmccallum avatar Oct 14 '16 17:10 npmccallum

Done

tiran avatar Oct 17 '16 11:10 tiran

Will you be integrating this into the build system as I requested?

npmccallum avatar Oct 19 '16 13:10 npmccallum

Is it really worth the extra effort? I'm going to hook up our documentation with readthedocs to get it automatically build.

tiran avatar Oct 21 '16 10:10 tiran

@tiran, do you think you'll update this PR to work on current master ?

simo5 avatar Nov 13 '19 14:11 simo5

@tiran: any update on this? Are you planning to resolve conflicting files?

sarroutbi avatar Oct 06 '21 12:10 sarroutbi

It is been since this PR has no activity. Closing it. Please, feel free to reopen it in case you consider necessary

sarroutbi avatar Jan 15 '24 10:01 sarroutbi