Add parse error location to unparsable environment file warning#11996
Merged
mergify[bot] merged 8 commits intoJul 26, 2026
Conversation
When a GHC environment file cannot be parsed, the warning now includes the line, column, and reason from the underlying parser, instead of only stating that the file is unparsable. Fixes haskell#11963 Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
Text.Parsec.Error.ParseError has no Exception instance on older GHCs, so displayException does not type-check. Its Show instance already renders the line, column, and reason. Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
wasimat404
marked this pull request as ready for review
June 21, 2026 01:02
zlonast
reviewed
Jun 24, 2026
Assert on the full warning message (location, unexpected character, and expected tokens) rather than only the line and column, addressing review feedback on test coverage. Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
Record the full warning output to cabal.out rather than asserting on a substring, so changes to the message are visible in review. Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
wasimat404
marked this pull request as draft
June 25, 2026 13:29
Golden output recording fails across the CI matrix for this test because the install command's surrounding output is not reproducible. Assert on the key parts of the warning instead. Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
Record the install output to cabal.out per review feedback, so changes to the warning are visible in review. Signed-off-by: wasimat404 <huaweiwasim7@gmail.com>
zlonast
approved these changes
Jun 26, 2026
Contributor
Author
|
@zlonast Thanks for the nudge:) |
wasimat404
marked this pull request as ready for review
June 26, 2026 23:36
Kleidukos
approved these changes
Jul 24, 2026
philderbeast
approved these changes
Jul 24, 2026
zlonast
added
squash+merge me
mergify
Bot
added
ready and waiting
Contributor
Merge Queue Status
This pull request spent 1 hour 48 minutes 3 seconds in the queue, including 1 hour 37 minutes 14 seconds running CI. Required conditions to merge
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #11963.
When a GHC environment file cannot be parsed, cabal previously warned only that the file "is unparsable", giving no indication of where or why. The underlying parser already produces a Parsec error carrying the line, column, and the unexpected/expected tokens, but that information was discarded at the warning site in
getExistingEnvEntries. This surfaces it.Before:
After:
The reporter's case was a stray trailing newline, which the parser reports as "unexpected end of input" at the offending line, so this change points directly at it. The fix is message-only; parser behaviour is unchanged.
Added a
cabal-testsuitepackage test underPackageTests/EnvironmentFile/UnparsableWarningthat points cabal at a malformed environment file and asserts the warning includes the location. Verified the test fails on unpatched cabal (the location line is absent) and passes with the change.QA Notes
Create a malformed GHC environment file, e.g. a file
bad.envwhose first line isthis is not valid. Then run:The warning should name the parse location, for example:
Before this change, only the first line ("is unparsable") was printed, with no location.