From d89c5ae8577264f5dd660906f12577c5fdadf49e Mon Sep 17 00:00:00 2001 From: Shilei Tian Date: Wed, 16 Sep 2020 18:54:11 -0400 Subject: [PATCH] [Flang] Fixed installation permission of the "binary" flang Under current configuration, the permission of `flang` after installation is 700. This could bring a problem for system administrators who build and install flang for other users, which only the user who builds LLVM can execute it, and others can not. In this patch, the explicit permission setting in the `install` command is removed, and let CMake determine what perssion to be used like other components. Reviewed By: DavidTruby Differential Revision: https://reviews.llvm.org/D87783 --- flang/tools/f18/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/flang/tools/f18/CMakeLists.txt b/flang/tools/f18/CMakeLists.txt index b92733d..64ccf12 100644 --- a/flang/tools/f18/CMakeLists.txt +++ b/flang/tools/f18/CMakeLists.txt @@ -84,4 +84,4 @@ set(FLANG_INTRINSIC_MODULES_DIR ${CMAKE_INSTALL_PREFIX}/include/flang) configure_file(${CMAKE_CURRENT_SOURCE_DIR}/flang.sh.in ${FLANG_BINARY_DIR}/bin/flang-install.sh @ONLY) configure_file(${CMAKE_CURRENT_SOURCE_DIR}/f18_version.h.in ${CMAKE_CURRENT_BINARY_DIR}/f18_version.h @ONLY) -install(PROGRAMS ${FLANG_BINARY_DIR}/bin/flang-install.sh DESTINATION bin RENAME flang PERMISSIONS OWNER_EXECUTE OWNER_READ OWNER_WRITE) +install(PROGRAMS ${FLANG_BINARY_DIR}/bin/flang-install.sh DESTINATION bin RENAME flang) -- 2.7.4