Repository navigation
Conversation
a1b386b to
1513bf8
Compare
| -- yield 'Nothing'), so the raw output is not valid source code. Use the | ||
| -- exported 'mkPackageName' / 'mkUnitId' instead. | ||
| fixupCompilerShow :: String -> String | ||
| fixupCompilerShow = go |
There was a problem hiding this comment.
if possible, could this be done where the constructors are visible?
There was a problem hiding this comment.
As I understand it, we can use the new cabal-testsuite on older branches. However, Show and Read haven't been updated in those older branches, which is where the problem arises.
There was a problem hiding this comment.
I've added it; now we have even more CPP macros.
1513bf8 to
0875578
Compare
philderbeast
left a comment
There was a problem hiding this comment.
Is everything here needed to bump setup-depends in cabal-testsuite?
| setup-depends: Cabal ^>= 3.18.1.0, | ||
| Cabal-syntax ^>= 3.18.1.0, |
There was a problem hiding this comment.
Sorry, another force push removed the co-authorship.
There was a problem hiding this comment.
I think my only changes are these two lines, so I don't think I deserve a co-authorship for just that 😅
| $ mkVerbosity defaultVerbosityHandles | ||
| $ fromFlagOrDefault normal (configVerbosity flags) |
There was a problem hiding this comment.
This is the recurring calling pattern that #11778 adds utility functions for.
-- | The verbosity from the setup verbosity or 'normal' if the flag is not set.
setupOrNormalVerbosity :: VerbosityHandles -> CommonSetupFlags -> Verbosity
setupOrNormalVerbosity verbHandles =
mkVerbosity verbHandles . Cabal.fromFlagOrDefault normal . setupVerbosityI see that setupVerbosity is configVerbosity:
0875578 to
9d0d698
Compare
Unfortunately, yes. cabal-testsuite's It is also worth noting that Basically, our |
Fix: #12186
Template Α: This PR modifies behaviour or interface
Include the following checklist in your PR:
significance: significantin the changelog file.