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

Enable checktvarinvariant on CI #756

Open
edsko opened this issue Sep 20, 2019 · 2 comments
Open

Enable checktvarinvariant on CI #756

edsko opened this issue Sep 20, 2019 · 2 comments
Labels
better-tests Ideas to improve the tests continuous integration

Comments

@edsko
Copy link
Contributor

edsko commented Sep 20, 2019

In IntersectMBO/ouroboros-network#1040 I added an invariant to enable assertions on StrictTVars. We should enable this on CI.

@mrBliss
Copy link
Contributor

mrBliss commented Dec 16, 2019

Maybe somebody with Nix experience can help us out? We need the test-storage and test-consensus testsuites from ouroboros-consensus to run with the checktvarinvariant flag (for the io-sim-classes package) enabled.

Running the tests with that flag enabled takes quite long, so it should not be part of the CI checks for PRs. That would slow down the merge process even more. We want a nightly CI job for this.

@nc6 You wrote https://github.com/input-output-hk/ouroboros-network/blob/master/nix/validate-mainnet.nix, maybe you know what to do?

@mrBliss mrBliss removed their assignment Dec 16, 2019
@mrBliss mrBliss self-assigned this Jan 6, 2020
@mrBliss
Copy link
Contributor

mrBliss commented Jan 6, 2020

I have received some pointers:

Update:

  • @angerman: I don’t think you can have two different instances of the same component in one build plan. I agree that there are cases where I wanted to specify a special flavour it some package as a build dependency at some point but that would require cabal to understand and support something like build-depends: <pkg>[:<comp>][[+|-]<flag>]* or something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
better-tests Ideas to improve the tests continuous integration
Projects
None yet
Development

No branches or pull requests

3 participants