Updated to makefile for new wolfHSM/docs dir#235
Conversation
|
Need to wait for wolfSSL/wolfHSM#256 to be merged |
bigbrett
left a comment
There was a problem hiding this comment.
@jackctj117 This PR should also remove this documentation source from the wolfHSM directory, no? We don't want two copies of the documentation source floating around.
Also ensure that the documentation builds and renders properly both locally and in our nightly job using the relocated source before this gets merged
|
Yes was waiting for the other PR to be merged before removing those directories so the nightly wouldn't fail |
|
@jackctj117 its merged |
|
@jackctj117 thanks. Have you verified that the docs render properly? Specifically the client API doxygen as part of the manual? Just want a set of eyes on it before this merges and potentially breaks our customer-facing docs |
|
@bigbrett confirmed this works locally and produces the client API doxygen as part of the manual. |
This pull request updates the
wolfHSM/Makefileto improve documentation build robustness and support a transition to a new documentation source directory. The changes ensure the correct documentation source is used, handle symlinked repositories for local testing, and guarantee that the externalwolfhsmrepo is cloned before documentation is built. Overrides to setup targets are introduced to inject these dependencies.