mirror of
https://github.com/microsoft/vcpkg.git
synced 2024-11-24 02:19:00 +08:00
47633daa65
Avoid wrong use of popular but ambiguous variables (`WIN32` etc.) in scripts and portfiles by detection in CI. Sometimes the variables are used wrongly, and sometimes this isn't caught in PR review. This PR tries to catch those variables (when in the active code path in script mode). [This can happen to every contributor](https://github.com/microsoft/vcpkg/pull/34356#discussion_r1360074122), even if he/she knows the rules ... vcpkg is raising the bar higher than usual, having to deal with targets and hosts even in script mode. (`b2-options.cmake` (boost fragment) shows where we get if we don't pay attention: the same code being used in script mode (`WIN32` meaning host) and in project mode (`WIN32` meaning target).) The new check doesn't break any user builds because it relies on command line options. However it changes the ABI hashes.
194 lines
8.9 KiB
CMake
194 lines
8.9 KiB
CMake
function(vcpkg_build_make)
|
|
z_vcpkg_get_cmake_vars(cmake_vars_file)
|
|
include("${cmake_vars_file}")
|
|
|
|
# parse parameters such that semicolons in options arguments to COMMAND don't get erased
|
|
cmake_parse_arguments(PARSE_ARGV 0 arg
|
|
"ADD_BIN_TO_PATH;ENABLE_INSTALL;DISABLE_PARALLEL"
|
|
"LOGFILE_ROOT;BUILD_TARGET;SUBPATH;MAKEFILE;INSTALL_TARGET"
|
|
"OPTIONS"
|
|
)
|
|
|
|
if(DEFINED arg_UNPARSED_ARGUMENTS)
|
|
message(WARNING "vcpkg_make_build was passed extra arguments: ${arg_UNPARSED_ARGUMENTS}")
|
|
endif()
|
|
|
|
if(NOT DEFINED arg_LOGFILE_ROOT)
|
|
set(arg_LOGFILE_ROOT "build")
|
|
endif()
|
|
|
|
if(NOT DEFINED arg_BUILD_TARGET)
|
|
set(arg_BUILD_TARGET "all")
|
|
endif()
|
|
|
|
if (NOT DEFINED arg_MAKEFILE)
|
|
set(arg_MAKEFILE Makefile)
|
|
endif()
|
|
|
|
if(NOT DEFINED arg_INSTALL_TARGET)
|
|
set(arg_INSTALL_TARGET "install")
|
|
endif()
|
|
|
|
if(CMAKE_HOST_WIN32)
|
|
set(Z_VCPKG_INSTALLED "${CURRENT_INSTALLED_DIR}")
|
|
else()
|
|
string(REPLACE " " "\ " Z_VCPKG_INSTALLED "${CURRENT_INSTALLED_DIR}")
|
|
endif()
|
|
|
|
vcpkg_list(SET make_opts)
|
|
vcpkg_list(SET install_opts)
|
|
if (CMAKE_HOST_WIN32)
|
|
set(path_backup "$ENV{PATH}")
|
|
vcpkg_add_to_path(PREPEND "${SCRIPTS}/buildsystems/make_wrapper")
|
|
if(NOT DEFINED Z_VCPKG_MAKE)
|
|
vcpkg_acquire_msys(MSYS_ROOT)
|
|
find_program(Z_VCPKG_MAKE make PATHS "${MSYS_ROOT}/usr/bin" NO_DEFAULT_PATH REQUIRED)
|
|
endif()
|
|
set(make_command "${Z_VCPKG_MAKE}")
|
|
vcpkg_list(SET make_opts ${arg_OPTIONS} -j ${VCPKG_CONCURRENCY} --trace -f ${arg_MAKEFILE} ${arg_BUILD_TARGET})
|
|
vcpkg_list(SET no_parallel_make_opts ${arg_OPTIONS} -j 1 --trace -f ${arg_MAKEFILE} ${arg_BUILD_TARGET})
|
|
|
|
string(REPLACE " " [[\ ]] vcpkg_package_prefix "${CURRENT_PACKAGES_DIR}")
|
|
string(REGEX REPLACE [[([a-zA-Z]):/]] [[/\1/]] vcpkg_package_prefix "${vcpkg_package_prefix}")
|
|
vcpkg_list(SET install_opts -j ${VCPKG_CONCURRENCY} --trace -f ${arg_MAKEFILE} ${arg_INSTALL_TARGET} DESTDIR=${vcpkg_package_prefix})
|
|
vcpkg_list(SET no_parallel_install_opts -j 1 --trace -f ${arg_MAKEFILE} ${arg_INSTALL_TARGET} DESTDIR=${vcpkg_package_prefix})
|
|
#TODO: optimize for install-data (release) and install-exec (release/debug)
|
|
|
|
else()
|
|
if(VCPKG_HOST_IS_FREEBSD OR VCPKG_HOST_IS_OPENBSD)
|
|
find_program(Z_VCPKG_MAKE gmake REQUIRED)
|
|
else()
|
|
find_program(Z_VCPKG_MAKE make REQUIRED)
|
|
endif()
|
|
set(make_command "${Z_VCPKG_MAKE}")
|
|
vcpkg_list(SET make_opts ${arg_OPTIONS} V=1 -j ${VCPKG_CONCURRENCY} -f ${arg_MAKEFILE} ${arg_BUILD_TARGET})
|
|
vcpkg_list(SET no_parallel_make_opts ${arg_OPTIONS} V=1 -j 1 -f ${arg_MAKEFILE} ${arg_BUILD_TARGET})
|
|
vcpkg_list(SET install_opts -j ${VCPKG_CONCURRENCY} -f ${arg_MAKEFILE} ${arg_INSTALL_TARGET} DESTDIR=${CURRENT_PACKAGES_DIR})
|
|
vcpkg_list(SET no_parallel_install_opts -j 1 -f ${arg_MAKEFILE} ${arg_INSTALL_TARGET} DESTDIR=${CURRENT_PACKAGES_DIR})
|
|
endif()
|
|
|
|
# Since includes are buildtype independent those are setup by vcpkg_configure_make
|
|
vcpkg_backup_env_variables(VARS LIB LIBPATH LIBRARY_PATH LD_LIBRARY_PATH CPPFLAGS CFLAGS CXXFLAGS RCFLAGS)
|
|
|
|
z_vcpkg_configure_make_common_definitions()
|
|
|
|
foreach(buildtype IN ITEMS "debug" "release")
|
|
if (buildtype STREQUAL "debug" AND _VCPKG_MAKE_NO_DEBUG)
|
|
continue()
|
|
endif()
|
|
if(NOT DEFINED VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL "${buildtype}")
|
|
string(TOUPPER "${buildtype}" cmake_buildtype)
|
|
set(short_buildtype "${short_name_${cmake_buildtype}}")
|
|
set(path_suffix "${path_suffix_${cmake_buildtype}}")
|
|
|
|
set(working_directory "${CURRENT_BUILDTREES_DIR}/${TARGET_TRIPLET}-${short_buildtype}/${arg_SUBPATH}")
|
|
message(STATUS "Building ${TARGET_TRIPLET}-${short_buildtype}")
|
|
|
|
if("libtool-link-pass-target" IN_LIST VCPKG_BUILD_MAKE_FIXUP)
|
|
# Pass --target to the linker, e.g. for Android
|
|
file(GLOB_RECURSE libtool_files "${working_directory}/libtool")
|
|
foreach(file IN LISTS libtool_files)
|
|
vcpkg_replace_string("${file}" [[-xtarget=*|]] [[-xtarget=*|--target=*|]])
|
|
endforeach()
|
|
endif()
|
|
|
|
z_vcpkg_configure_make_process_flags("${cmake_buildtype}")
|
|
|
|
# Setup environment
|
|
set(ENV{CPPFLAGS} "${CPPFLAGS_${cmake_buildtype}}")
|
|
set(ENV{CFLAGS} "${CFLAGS_${cmake_buildtype}}")
|
|
set(ENV{CXXFLAGS} "${CXXFLAGS_${cmake_buildtype}}")
|
|
set(ENV{RCFLAGS} "${VCPKG_DETECTED_CMAKE_RC_FLAGS_${cmake_buildtype}}")
|
|
set(ENV{LDFLAGS} "${LDFLAGS_${cmake_buildtype}}")
|
|
vcpkg_list(APPEND lib_env_vars LIB LIBPATH LIBRARY_PATH) # LD_LIBRARY_PATH)
|
|
foreach(lib_env_var IN LISTS lib_env_vars)
|
|
if(EXISTS "${Z_VCPKG_INSTALLED}${path_suffix}/lib")
|
|
vcpkg_host_path_list(PREPEND ENV{${lib_env_var}} "${Z_VCPKG_INSTALLED}${path_suffix}/lib")
|
|
endif()
|
|
if(EXISTS "${Z_VCPKG_INSTALLED}${path_suffix}/lib/manual-link")
|
|
vcpkg_host_path_list(PREPEND ENV{${lib_env_var}} "${Z_VCPKG_INSTALLED}${path_suffix}/lib/manual-link")
|
|
endif()
|
|
endforeach()
|
|
unset(lib_env_vars)
|
|
|
|
if(LINK_ENV_${cmake_buildtype})
|
|
set(config_link_backup "$ENV{_LINK_}")
|
|
set(ENV{_LINK_} "${LINK_ENV_${cmake_buildtype}}")
|
|
else()
|
|
unset(config_link_backup)
|
|
endif()
|
|
|
|
if(arg_ADD_BIN_TO_PATH)
|
|
set(env_backup_path "$ENV{PATH}")
|
|
vcpkg_add_to_path(PREPEND "${CURRENT_INSTALLED_DIR}${path_suffix}/bin")
|
|
endif()
|
|
|
|
vcpkg_list(SET make_cmd_line ${make_command} ${make_opts})
|
|
vcpkg_list(SET no_parallel_make_cmd_line ${make_command} ${no_parallel_make_opts})
|
|
|
|
if (arg_DISABLE_PARALLEL)
|
|
vcpkg_execute_build_process(
|
|
COMMAND ${no_parallel_make_cmd_line}
|
|
WORKING_DIRECTORY "${working_directory}"
|
|
LOGNAME "${arg_LOGFILE_ROOT}-${TARGET_TRIPLET}-${short_buildtype}"
|
|
)
|
|
else()
|
|
vcpkg_execute_build_process(
|
|
COMMAND ${make_cmd_line}
|
|
NO_PARALLEL_COMMAND ${no_parallel_make_cmd_line}
|
|
WORKING_DIRECTORY "${working_directory}"
|
|
LOGNAME "${arg_LOGFILE_ROOT}-${TARGET_TRIPLET}-${short_buildtype}"
|
|
)
|
|
endif()
|
|
|
|
file(READ "${CURRENT_BUILDTREES_DIR}/${arg_LOGFILE_ROOT}-${TARGET_TRIPLET}-${short_buildtype}-out.log" logdata)
|
|
if(logdata MATCHES "Warning: linker path does not have real file for library")
|
|
message(FATAL_ERROR "libtool could not find a file being linked against!")
|
|
endif()
|
|
|
|
if (arg_ENABLE_INSTALL)
|
|
message(STATUS "Installing ${TARGET_TRIPLET}-${short_buildtype}")
|
|
vcpkg_list(SET make_cmd_line ${make_command} ${install_opts})
|
|
vcpkg_list(SET no_parallel_make_cmd_line ${make_command} ${no_parallel_install_opts})
|
|
vcpkg_execute_build_process(
|
|
COMMAND ${make_cmd_line}
|
|
NO_PARALLEL_COMMAND ${no_parallel_make_cmd_line}
|
|
WORKING_DIRECTORY "${working_directory}"
|
|
LOGNAME "install-${TARGET_TRIPLET}-${short_buildtype}"
|
|
)
|
|
endif()
|
|
|
|
if(DEFINED config_link_backup)
|
|
set(ENV{_LINK_} "${config_link_backup}")
|
|
endif()
|
|
|
|
if(arg_ADD_BIN_TO_PATH)
|
|
set(ENV{PATH} "${env_backup_path}")
|
|
endif()
|
|
endif()
|
|
|
|
vcpkg_restore_env_variables(VARS LIB LIBPATH LIBRARY_PATH)
|
|
endforeach()
|
|
|
|
if (arg_ENABLE_INSTALL)
|
|
string(REGEX REPLACE "([a-zA-Z]):/" "/\\1/" Z_VCPKG_INSTALL_PREFIX "${CURRENT_INSTALLED_DIR}")
|
|
file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}_tmp")
|
|
file(RENAME "${CURRENT_PACKAGES_DIR}" "${CURRENT_PACKAGES_DIR}_tmp")
|
|
file(RENAME "${CURRENT_PACKAGES_DIR}_tmp${Z_VCPKG_INSTALL_PREFIX}" "${CURRENT_PACKAGES_DIR}")
|
|
file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}_tmp")
|
|
endif()
|
|
|
|
# Remove libtool files since they contain absolute paths and are not necessary.
|
|
file(GLOB_RECURSE libtool_files "${CURRENT_PACKAGES_DIR}/**/*.la")
|
|
if(libtool_files)
|
|
file(REMOVE ${libtool_files})
|
|
endif()
|
|
|
|
if (CMAKE_HOST_WIN32)
|
|
set(ENV{PATH} "${path_backup}")
|
|
endif()
|
|
|
|
vcpkg_restore_env_variables(VARS LIB LIBPATH LIBRARY_PATH LD_LIBRARY_PATH CPPFLAGS CFLAGS CXXFLAGS RCFLAGS)
|
|
unset(_VCPKG_MAKE_NO_DEBUG PARENT_SCOPE)
|
|
endfunction()
|