Skip to content

Commit

Permalink
Change image to use clang and remove target_include_directories
Browse files Browse the repository at this point in the history
  • Loading branch information
jmcarcell committed Nov 14, 2023
1 parent 6f3c00f commit c7478ab
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pre-commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
- uses: cvmfs-contrib/github-action-cvmfs@v3
- uses: aidasoft/run-lcg-view@v4
with:
release-platform: LCG_104/x86_64-centos7-gcc12-opt
release-platform: LCG_103/x86_64-centos7-clang12-opt
run: |
STARTDIR=$(pwd)
echo "::group::Build podio"
Expand Down
3 changes: 0 additions & 3 deletions edm4hep/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@ set_target_properties(edm4hep PROPERTIES ALIAS_GLOBAL true)

file(GLOB_RECURSE top_headers ${PROJECT_SOURCE_DIR}/include/*.h)
target_sources(edm4hep PUBLIC FILE_SET headers TYPE HEADERS BASE_DIRS ${PROJECT_SOURCE_DIR}/include FILES ${top_headers})
target_include_directories(edm4hep PUBLIC
$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>)

if (nlohmann_json_FOUND)
target_compile_definitions(edm4hep PUBLIC PODIO_JSON_OUTPUT)
Expand Down

0 comments on commit c7478ab

Please sign in to comment.