Skip to content

Add back +legacy-comparison to the validate project - #12394

Open
philderbeast wants to merge 47 commits into
haskell:masterfrom
cabalism:test/validate-comparing-parsers
Open

philderbeast wants to merge 47 commits into
haskell:masterfrom
cabalism:test/validate-comparing-parsers

Conversation

@philderbeast

@philderbeast philderbeast commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

A follow on from #12139 (depends-on: 12139), adding back +legacy-comparison to the validate project, something that was removed in #11773.

$ git diff HEAD~1 HEAD~0
diff --git a/cabal.validate.project b/cabal.validate.project
index d1d57e0fe..bf2552af2 100644
--- a/cabal.validate.project
+++ b/cabal.validate.project
@@ -13,7 +13,7 @@ program-options
 -- The +legacy-comparison flag is to set the default project file parser to compare
 -- the result of the legacy and parsec parser and fail if they are not equal.
 package cabal-install
-  flags: +git-rev
+  flags: +git-rev +legacy-comparison
 
 package Cabal
   flags: +git-rev

@philderbeast philderbeast changed the title Adding back +legacy-comparison to the validate project Add back +legacy-comparison to the validate project Sep 29, 2026
@philderbeast
philderbeast force-pushed the test/validate-comparing-parsers branch from 1f6f086 to 36190b2 Compare October 8, 2026 13:25
@philderbeast
philderbeast force-pushed the test/validate-comparing-parsers branch 2 times, most recently from 36bf3ca to 66cb9c0 Compare November 3, 2026 20:22
@mergify mergify Bot added the 👻 haunting the review queue Set by Mergify: no review from anyone with write access two weeks after opening label Nov 6, 2026
@philderbeast
philderbeast force-pushed the test/validate-comparing-parsers branch from 944d199 to e2ac483 Compare November 11, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👻 haunting the review queue Set by Mergify: no review from anyone with write access two weeks after opening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant