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

Fix documentation install when Doxygen is not found #1624

Closed
wants to merge 1 commit into from

Conversation

@Crunkle
Copy link
Contributor

Crunkle commented Jan 14, 2020

If Doxygen is not installed then the install target will fail as the documentation was not built. This simply updates the check to only include the docs directory if Doxygen was found.

@Crunkle Crunkle requested a review from elmindreda as a code owner Jan 14, 2020
@elmindreda elmindreda self-assigned this Jan 15, 2020
@elmindreda elmindreda added the verified label Jan 15, 2020
@elmindreda elmindreda added this to the 3.3.2 milestone Jan 15, 2020
elmindreda added a commit that referenced this pull request Jan 16, 2020
elmindreda added a commit that referenced this pull request Jan 16, 2020
Related to #1624.
elmindreda added a commit that referenced this pull request Jan 17, 2020
Related to #1624.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.