llnl / llnl/smith

Enable Codevelop build for blueos

Open
#748 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug build CS gpu testing TPL
Dominant language
C++
Stars
245
Forks
36
Avg merge
8d 23h
Merged PRs (30d)
4

Description

Currently getting a compile flag that is ill formed:

-Xcompiler= -Xcompiler=-fopenmp=libomp

the first -Xcompiler= causes the rest to be past to the host compiler, which cannot understand the second -Xcompiler=.

Also there are a lot of CMake warnings about our built zlib package. For example:

CMake Warning at cmake/blt/cmake/BLTMacros.cmake:770 (add_executable):
  Cannot generate a safe runtime search path for target
  simple_conduction_with_input_file because files in some directories may
  conflict with libraries in implicit directories:

    runtime library [libz.so.1] in /usr/lib64 may be hidden by files in:
      /usr/WS2/smithdev/libs/serac/blueos_3_ppc64le_ib_p9/2022_06_01_21_11_45/clang-10.0.1/zlib-1.2.12-i6lqmmtb2os46rfflvu43zelz5sv2lcb/lib

  Some of these libraries may not be found correctly.
Call Stack (most recent call first):
  examples/simple_conduction/CMakeLists.txt:7 (blt_add_executable)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with examples/simple_conduction/CMakeLists.txt and the BLT add_executable path in cmake/blt/cmake/BLTMacros.cmake, then reproduce the BlueOS build to inspect how the compiler flags are assembled. Done means the malformed -Xcompiler option is gone and the built zlib package no longer produces the reported CMake runtime search path warnings.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.