-
Notifications
You must be signed in to change notification settings - Fork 219
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
Patch documentation build for OSAL API file #624
Comments
2 tasks
dzbaker
added a commit
that referenced
this issue
Dec 6, 2022
Fix #624, add compatibility for CFS app doc build
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist (Please check before submitting)
Is your feature request related to a problem? Please describe.
Most CFS app documentation builds now depend on a file produced by the OSAL documentation build, but there is not (yet) a dependency encoded into the CMakeLists.txt file to ensure this gets produced before attempting the doc build.
Describe the solution you'd like
The correct solution is to add a dependency for the doc build into CMakeList.txt to each app that does have this dependency.
In the meantime, the failing workflow can be worked around by first forcing build of the OSAL API doxyfile.
Additional context
This would be just a short term patch to make the workflows run. Should be removed once the CFS apps are fixed.
Requester Info
Joseph Hickey, Vantage Systems, Inc.
The text was updated successfully, but these errors were encountered: