-
Notifications
You must be signed in to change notification settings - Fork 12
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
Missing file KExport.hpp #45
Comments
@chasbryant Line 960 in c22793d
The file is saved under ${CMAKE_CURRENT_BINARY_DIR}/include/KDIS (visible thanks to target_include_directories ).The generation procedure is the same as previously except for the location (old was ${CMAKE_CURRENT_SOURCE_DIR}/include/KDIS ). The latter allowed removing an entry in .gitignore .Therefore, it should compile without any problem as before. |
It wasn’t for me…. All the added export stuff was flagged on build…. Just fyiChuck BryantOn Jan 17, 2025, at 16:47, Carlo Corradini ***@***.***> wrote:
@chasbryant KExport.hpp is automatically generated by CMake (see https://github.com/karljj1/kdis/blob/c22793d3005c602860304cd45c63fbe39499d792/CMakeLists.txt#L960) using GenerateExportHeader.
The file is saved under ${CMAKE_CURRENT_BINARY_DIR}/include/KDIS (visible thanks to target_include_directories).
The generation procedure is the same as previously except for the location (old was ${CMAKE_CURRENT_SOURCE_DIR}/include/KDIS). The latter allowed removing an entry in .gitignore.
Therefore, it should compile without any problem as before.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@chasbryant What? |
I must have been pulling includes from the src folder then…. I was building to put into a docker container.
… On Jan 17, 2025, at 4:47 PM, Carlo Corradini ***@***.***> wrote:
@chasbryant <https://github.com/chasbryant> KExport.hpp is automatically generated by CMake (see https://github.com/karljj1/kdis/blob/c22793d3005c602860304cd45c63fbe39499d792/CMakeLists.txt#L960) using GenerateExportHeader <https://cmake.org/cmake/help/latest/module/GenerateExportHeader.html>.
The file is saved under ${CMAKE_CURRENT_BINARY_DIR}/include/KDIS (visible thanks to target_include_directories).
The generation procedure is the same as previously except for the location (old was ${CMAKE_CURRENT_SOURCE_DIR}/include/KDIS). The latter allowed removing an entry in .gitignore.
Therefore, it should compile without any problem as before.
—
Reply to this email directly, view it on GitHub <#45 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALHBHENU6ST5XFLWRVEVK5L2LGCADAVCNFSM6AAAAABVMRPQV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOJZGMZTGNBWGY>.
You are receiving this because you were mentioned.
|
@chasbryant Always pull from a specified tag since it's safer 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I cannot compile the master branch due to this file:
#include "KDIS/KExport.hpp"
missing....
The text was updated successfully, but these errors were encountered: