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

Testsuite: pass -i argument to runghc invocations (backport #9518) #10003

Merged
merged 1 commit into from
May 13, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 13, 2024

This commit modifies the generation of runghc commands to pass an additional -i argument to account for the change in working directory.

This ensures that, in the testsuite, runghc is able to see other modules. For example, an invocation of runghc to run a Custom Setup script will now properly see the modules imported, e.g. if one has a directory structure like

  test.cabal
  Setup.hs
  SetupDep.hs

and Setup.hs imports SetupDep.hs.


This is an automatic backport of pull request #9518 done by Mergify.

@ulysses4ever ulysses4ever added the merge me Tell Mergify Bot to merge label May 13, 2024
This commit modifies the generation of runghc commands to pass an
additional -i argument to account for the change in working directory.

This ensures that, in the testsuite, runghc is able to see other modules.
For example, an invocation of runghc to run a Custom Setup script will
now properly see the modules imported, e.g. if one has a directory
structure like

  test.cabal
  Setup.hs
  SetupDep.hs

and Setup.hs imports SetupDep.hs.

(cherry picked from commit b2b9c2f)
@mergify mergify bot merged commit ece298a into 3.12 May 13, 2024
47 of 48 checks passed
@mergify mergify bot deleted the mergify/bp/3.12/pr-9518 branch May 13, 2024 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport merge me Tell Mergify Bot to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants