Skip to content

Fixes to make devtest #4684

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.

Sign up for GitHub

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

Jump to bottom
Merged
fisx merged 2 commits into from
Aug 20, 2025
Merged

Fixes to make devtest #4684

fisx merged 2 commits into from
Aug 20, 2025

Conversation

@eyeinsky
Copy link
Contributor

@eyeinsky eyeinsky commented Jul 25, 2025

This fixes a help message and might fix the cabal repl used by ghcid. Might in the sense that when I run make devtest package=wire-subsystems I get the following error:

Error: [Cabal-7076]
Cannot open a repl for multiple components at onnce. The targets 'wire-subsystems' and 'wire-subsystems-tests' refer to different components..

Your compiler supports a multiple component repl but support is not enabled.
The experimental multi repl can be enabled by
  * Globally: Setting multi-repl: True in your .cabal/config
  * Project Wide: Setting multi-repl: True in your cabal.project file
  * Per Invocation: By passing --enable-multi-repl when starting the repl

Command "cabal repl wire-subsystems:wire-subsystems-tests lib:wire-subsystems" exited unexpectedly with error message: 



*** usage: make devtest package=<package>.
*** this works for wire-subsystems; for other packages, you may need to edit the cabal file.

This is on cabal version 3.14.1.1. This seems to be provided by the nix environment, so I don't really understand how the command did run for @fisx, but not for me. Adding --enable-multi-repl as suggested makes it work.

@eyeinsky eyeinsky requested a review from a team as a code owner July 25, 2025 14:48
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 25, 2025
@zebot zebot mentioned this pull request Jul 28, 2025
Merged
@supersven
Copy link
Contributor

supersven commented Aug 4, 2025

I'd guess that @fisx has multi-repl: True in his cabal.project.local file 🤔

@fisx
Copy link
Contributor

fisx commented Aug 20, 2025

I'd guess that @fisx has multi-repl: True in his cabal.project.local file 🤔

that's exactly right.

thanks for the fix!

@fisx fisx merged commit 6c26f2c into develop Aug 20, 2025
8 checks passed
@fisx fisx deleted the ml/make-devtest branch August 20, 2025 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fisx fisx fisx approved these changes

Assignees

No one assigned

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@eyeinsky @supersven @fisx @zebot