coldtobi commented on pull request #99:
URL: https://github.com/apache/logging-log4cxx/pull/99#issuecomment-1008185839
> The patch as given requires CMake 3.20+, but doesn't update the minimum
required version. I'd like to not have to bump the CMake version if possible.
Ack.
> I changed the `cmake_path` directive to `file(RELATIVE_PATH
LOG4CXX_REL_BIN_TO_SRC_DIR "${LOG4CXX_SOURCE_DIR}"
"${CMAKE_CURRENT_BINARY_DIR}")` on my side which seems to work fine, would you
be able to validate that?
LGTM, it build reproducible as well on my machine. I'll update my PR
accordingly.
> I don't see where the particular example that you showed is coming in
from, that only seems to be a problem with the MapFilter page.
It seems everything in site/markdown triggering the differences... Here's a
(local created) complete diff: https://pastebin.com/6BLaEa0K. (So it is quite
possible that only the INPUT configuration of Doxyfile.in would need to be
relative, but I believe making the other relative won't hurt.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
