-
Notifications
You must be signed in to change notification settings - Fork 723
Add Cabal-tests workaround project #9923
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
Conversation
cc435da to
34bac5c
Compare
|
A ping about #9901 (comment) in case it goes under the radar. :) |
34bac5c to
e0fcb97
Compare
|
Note With #9578 and later versions of @grayjay the effort we put into #9578 may be paying dividends. |
|
@philderbeast: thank you for you prompt action. The reporter of the issue is on holidays, so let me add another reviewer. |
e0fcb97 to
db3f9a8
Compare
|
@Mikolaj, I've renamed the workaround project so that it will be picked up automatically from the |
|
Yay, so does this solve all plausible cases of this failure? Or is there a common workflow that does not enter that directory? Can the reviewers/reporters comment? |
It fixes the reproduction of #9901 for
For |
ed719be to
b0d85e2
Compare
|
I am not sure this fixes the regression, you are still unable to run |
- Add testRelativePath - Update Cabal-tests/README.md
b0d85e2 to
df63967
Compare
|
@Mikolaj & @mpickering , I'm closing this now in anticipation of the imminent |
A workaround for #9901 so that we can use released versions of
cabal-install, such ascabal-3.8.*andcabal-3.10.*, withCabal-tests:parser-testsandCabal-tests:check-teststest suites whencabal runrunning them to accept updated test outputs.To repeat the problem; when we run
cabal test Cabal-tests, the working directory will beCabal-testsbut when we runcabal run Cabal-tests:parser-testsorCabal-tests:check-testsit will be the current directory. To update the expected outputs we need to first change directory toCabal-tests. That is good for the package but not for the rootcabal.projectthat will be unable to resolve imports failing with;I deliberately named
Cabal-tests/cabal.workaround.projectso that we might notice to delete it at a later time. This is at the cost of having to do the following with earlier versions of cabal;Note
With #9578 and later versions of
cabal-install, the above problem does not occur and the workaround project is not needed. I did not set out to fix this exact problem but left the comment "I also fix some bugs around project imports" on that pull request. Related, we should check the behavior before and after #9578 to see if it helps with #8795.Cabal-testsand put it in that package's directory, up one from where it was before.cabal testandcabal runmake no mention of working directory.Cabal-tests --acceptandcabal-testsuite --accept. With the former we need to be in theCabal-testsdirectory for it to work but for the later we need to be in the root of the project and running--acceptfrom thecabal-testsuitedirectory fails. Currently it fails for the same reason asCabal-testsdoes but prior to the project changes of Use package groups #9565, it also fails;Template Β: This PR does not modify
cabalbehaviour (documentation, tests, refactoring, etc.)Include the following checklist in your PR: