Skip to content
This repository was archived by the owner on Mar 20, 2023. It is now read-only.

Minimum Boost 1.59 #846

Merged
merged 1 commit into from
Aug 9, 2022
Merged

Minimum Boost 1.59 #846

merged 1 commit into from
Aug 9, 2022

Conversation

olupton
Copy link
Contributor

@olupton olupton commented Aug 9, 2022

Description
#841 added tests that use a two-parameter form of the BOOST_AUTO_TEST_CASE macro, which was added in boostorg/test@24e5dde and exists in Boost 1.59+.

This is causing failures on CentOS7 in https://github.com/neuronsimulator/nrn-build-ci/runs/7738697401 after the NEURON submodule of CoreNEURON was updated in neuronsimulator/nrn#1942.

Use certain branches in CI pipelines.

CI_BRANCHES:NEURON_BRANCH=master,NMODL_BRANCH=master,SPACK_BRANCH=develop

@olupton
Copy link
Contributor Author

olupton commented Aug 9, 2022

@olupton olupton requested a review from alexsavulescu August 9, 2022 08:06
@olupton
Copy link
Contributor Author

olupton commented Aug 9, 2022

Launched https://github.com/neuronsimulator/nrn-build-ci/actions/runs/2823715498 using neuronsimulator/nrn@05096be.

Looks like this fixes the issue with CentOS7.
I see that Debian Buster does not have Boost installed, but that can be fixed separately if needed.

@olupton olupton merged commit aef38da into master Aug 9, 2022
@olupton olupton deleted the olupton/minimum-boost branch August 9, 2022 10:56
pramodk pushed a commit to neuronsimulator/nrn that referenced this pull request Nov 2, 2022
Based on the new solver unit test requirements.

CoreNEURON Repo SHA: BlueBrain/CoreNeuron@aef38da
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants