cmake hacks for windows #68
Conversation
|
Based on your changes, I have some updates lined up for ARPA2CM 0.6 and QD to fix this more generally. See also branch python-wrangling in the repo right now. |
|
moet ; niet worden geescaped? |
cmake/PythonSupport.cmake
Outdated
| # License-Filename: LICENSES/BSD-2-Clause.degroot | ||
|
|
||
| if (WIN32) | ||
| if("${CMAKE_HOST_SYSTEM}" MATCHES ".*Windows.*") |
There was a problem hiding this comment.
https://cmake.org/cmake/help/v3.13/variable/WIN32.html?highlight=win32
Maar goed, jij checkt host en niet target system. Daar is wat voor te zeggen, in dit geval. Het is meer idiomatisch om if(CMAKE_HOST_SYSTEM MATCHES "Windows") te schrijven -- variable expansion is implicit, en de match is niet anchored en hoeft niet de hele string te matchen. Dus "ergens komt Windows voor in de host system" hoeft geen sterretjes.
There was a problem hiding this comment.
Er .. language settings :) If you specifically want to check the host system (and I guess that makes sense, since that's where the build is happening and you need to run python during the build, on the host) then that can be done more idiomatically as ..
Why would ; need to be escaped, and where? |
| macro(add_asn1_document _docname _groupname) | ||
| add_custom_command (OUTPUT doc/${_docname}.md | ||
| COMMAND ${CMAKE_SOURCE_DIR}/python/scripts/asn1literate ${CMAKE_CURRENT_SOURCE_DIR}/${_docname}.asn1 ${_docname}.md | ||
| COMMAND ${PYTHON_EXECUTABLE} ${CMAKE_SOURCE_DIR}/python/scripts/asn1literate ${CMAKE_CURRENT_SOURCE_DIR}/${_docname}.asn1 ${_docname}.md |
There was a problem hiding this comment.
This is a straight-up bug in the module, and doesn't need to be hidden in a hacks-branch
2 participants
cmake/MacroASN1Module.cmake:
lib/CMakeLists.txt
On windows shared libraries need ARCHIVE DESTINATION: https://cmake.org/pipermail/cmake/2008-September/023876.html