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.

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

Several targets #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Several targets #1

wants to merge 2 commits into from

Conversation

pthom
Copy link

@pthom pthom commented May 16, 2020

First, thanks a lot for your article and for this repo. It helps a lot!

I made some simple modifications to your projet, in order to make it easier to use in the case of a standard project that would use a top "src" + "include" folder and demonstrate a use case with several targets.

This version should be compatible with the merged or not merged header project layout conventions, since it searches for headers in both folders (except src//internal src//details and src/**/private)

I do not think this version would be compatible with your article, so I do not think it should be merged into master. However - and if you are interested - may be you could add a branch with this version.

The resulting doc on readthedocs:

https://pthom-cpp-documentation-example.readthedocs.io/en/latest/

@pthom
Copy link
Author

pthom commented May 16, 2020

The advantage of this PR is that you can take cmake/FindSphinx and doc/ as they are and plug them into a new project that uses the standard "src/" and "include/" top folders, it works from the start, with no additional modification (it did on one of my project at least)

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.

1 participant