Skip to content
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

CMakes writes to non-writeable dir with builtin_llvm=OFF #12151

Closed
1 task done
stephanlachnit opened this issue Jan 28, 2023 · 2 comments
Closed
1 task done

CMakes writes to non-writeable dir with builtin_llvm=OFF #12151

stephanlachnit opened this issue Jan 28, 2023 · 2 comments

Comments

@stephanlachnit
Copy link
Contributor

stephanlachnit commented Jan 28, 2023

  • Checked for duplicates

Describe the bug

Build master with builtin_llvm=OFF using system LLVM. CMake configuration fails with:

-- Using LLVM external library - 13.0.1
-- Clang version: 13.0.1
-- Could NOT find Git (missing: GIT_EXECUTABLE) 
-- Not building amdgpu-arch: hsa-runtime64 not found
-- Cling version (from VERSION file): ROOT_1.0~dev
-- Cling will look for C++ headers in '/usr/include/c++/12:/usr/include/x86_64-linux-gnu/c++/12:/usr/include/c++/12/backward' at runtime.
-- And then fallback to: 'ccache'
CMake Error: Could not open file for write in copy operation /usr/lib/llvm-13/lib/cmake/cling/ClingConfig.cmake.tmp
CMake Error: : System Error: No such file or directory
CMake Error at interpreter/cling/cmake/modules/CMakeLists.txt:22 (configure_file):
  configure_file Problem configuring file

The error is thrown in interpreter/cling/cmake/modules/CMakeLists.txt:22-25 when trying to create a file in cling_cmake_builddir which is not writable.

The bug originates from interpreter/cling/cmake/modules/CMakeLists.txt:5 and was introduced in 2b283cc from #10294.

Expected behavior

I think there are three options to fix this:

  • 2b283cc is reverted (the commit mentions some fixes though)
  • CMake checks for builtin_llvm=OFF and sets the dir accordingly
  • Some other path is found that prevents the issues mentioned in 2b283cc (maybe CLING_BINARY_DIR?)

To Reproduce

Build from master with builtin_llvm=OFF

Setup

git master, Debian Sid

Additional context

/cc @hahnjo @vgvassilev @bellenot

@stephanlachnit
Copy link
Contributor Author

Closing, duplicate of #11920.

@github-actions
Copy link

Hi @stephanlachnit, @bellenot,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

@bellenot bellenot added this to Issues in Fixed in: not applicable via automation Feb 13, 2023
stephanlachnit added a commit to stephanlachnit/root that referenced this issue Jul 31, 2023
…lingConfig.cmake

This fixes root-project#12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
stephanlachnit added a commit to stephanlachnit/root that referenced this issue Jul 31, 2023
…lingConfig.cmake

This fixes root-project#12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
stephanlachnit added a commit to stephanlachnit/root that referenced this issue Jul 31, 2023
…lingConfig.cmake

This fixes root-project#12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
hahnjo pushed a commit that referenced this issue Aug 8, 2023
…lingConfig.cmake

This fixes #12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
hahnjo pushed a commit to hahnjo/root that referenced this issue Aug 14, 2023
…lingConfig.cmake

This fixes root-project#12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
(cherry picked from commit 8a45c09)
hahnjo pushed a commit that referenced this issue Aug 15, 2023
…lingConfig.cmake

This fixes #12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
(cherry picked from commit 8a45c09)
maksgraczyk pushed a commit to maksgraczyk/root that referenced this issue Jan 12, 2024
…lingConfig.cmake

This fixes root-project#12151.

Signed-off-by: Stephan Lachnit <stephanlachnit@debian.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants