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

Bugfix: Build system fixes #309

Merged
merged 3 commits into from
Jun 21, 2024
Merged

Bugfix: Build system fixes #309

merged 3 commits into from
Jun 21, 2024

Conversation

nathanwbrei
Copy link
Collaborator

  • jana-config now adds $JANA_INSTALL_DIR/lib to rpath
  • jana-config now understands static vs dynamic library linking
  • .gitignore no longer accidentally ignores src/programs
  • examples/SubeventCUDAExample and examples/UnitTestingExample now install to bin instead of programs
  • .gitignore now uses wildcards for build and install directories

@nathanwbrei nathanwbrei force-pushed the nbrei_build_system_fixes branch from ddbec8b to 8991bf3 Compare June 21, 2024 19:45
Top-level 'programs' pattern was incorrectly matching against src/programs. Also made the pattern for '/build' into a wildcard so that it includes build_podio, etc.
Some examples previously installed to CMAKE_INSTALL_PREFIX/programs, which isn't in the FHS. They now install to CMAKE_INSTALL_PREFIX/bin instead.
@nathanwbrei nathanwbrei force-pushed the nbrei_build_system_fixes branch from 8991bf3 to 8f09ea4 Compare June 21, 2024 19:45
@nathanwbrei nathanwbrei merged commit a56fbc7 into master Jun 21, 2024
3 checks passed
@nathanwbrei nathanwbrei deleted the nbrei_build_system_fixes branch June 21, 2024 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants