Conversation
external hipo
fix HIPO_LIBS
fix(ci): update actions and dependencies
Add ccdb/rcdb test
fix(ci): tab -> space
since it triggers updates for unrelated packages, and sometimes failures such as [this one, seen in `coatjava` CI](https://github.com/JeffersonLab/coatjava/actions/runs/17418515756/job/49452235710) on the other hand, `apt update` is recommended by the CI documentation
fix(ci): don't `apt upgrade`
Use a CLAS12 software container, which includes dependencies such as: - ROOT - RCDB - CCDB - QADB - HIPO For more information, see https://code.jlab.org/hallb/clas12/container-forge
this is because I suspect that `$CLAS12ROOT` is not actually being set in the same step if we just add it to `$GITHUB_ENV`; on the other hand, if we define it first, we can use it in the same step to form additional variables, such as `$PATH`.
fix(ci): use `container-forge` image instead of building dependencies here
- new URIs pointing to code.jlab.org repos - update submodule versions
Add build option `WITH_HIPO4_WRAPPER` to control whether or not to build code in `hipo4`. Default is set to `OFF` (see JeffersonLab#105 (comment)).
Don't call `make`, which will not work for non-Makefile generators; instead use `cmake --build` and `cmake --install`.
chore: update git submodules
build: optionally build `hipo4/` HIPO wrapper
…nerator fix(build): don't assume the CMake generator is a Makefile generator
…ed-lib-ext Fix/detect shared lib ext
Otherwise on some systems, it may try to install at the system level (which needs `root` permission).
fix(build): set installation prefix in `installC12Root`
remove line with explicit .so
build: bump qadb submodule to 3.4.0
Change integer thresholds to double in mesonex_trigger
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.