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

Don't copy headers when not building docs. #855

Closed
wants to merge 3 commits into from
Closed

Conversation

jtv
Copy link
Owner

@jtv jtv commented Jun 23, 2024

In order to help Doxygen find all the headers, I had the docs target
copy all of them into the build tree (if that differed from the source
tree).

But it turns out that that was happening even on non-documentation
builds. Which makes development a little hard, because you're always
working with copies of the headers, stuck in the build tree!

For now, just skip the copying when not configured to generate
documentation. Later we can look into copying the headers somewhere
into the docs directory instead, so they don't get into the way of
the source build itself.

jtv added 3 commits June 24, 2024 01:04
In order to help Doxygen find all the headers, I had the `docs` target
copy all of them into the build tree (if that differed from the source
tree).

But it turns out that that was happening even on non-documentation
builds.  Which makes development a little hard, because you're always
working with _copies_ of the headers, stuck in the build tree!

For now, just skip the copying when not configured to generate
documentation.  Later we can look into copying the headers somewhere
into the `docs` directory instead, so they don't get into the way of
the source build itself.
@jtv jtv closed this Jun 23, 2024
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