Skip to content

Conversation

@pranaysashank
Copy link
Member

@pranaysashank pranaysashank commented Aug 27, 2022

  • Made separate modules for each ghc version in the
    hopes that maintaining them would become easier
    than the many nested if, else we have in cpp right now.
  • There is some redundancy and CPP usage in Plugin.hs,
    which seems a bit to hard to eliminate without duplicating
    efforts.
  • In the CI, 8.4.4 fails I don't understand the error message
    there.
    So just module A in the file is not valid syntax,
    wonder why I thought it was.
    And 9.4.2 needs Debug.Trace but previous versions
    don't need it in the imports, where is it coming from?

Opened for CI. Turning it back to draft, after verifying that
this approach makes sense by fixing GhcHead I'll revert it.

@pranaysashank pranaysashank force-pushed the ghc-module-per-version branch 3 times, most recently from 3e986e1 to 0c68bb3 Compare August 27, 2022 13:44
- Made separate modules for each ghc version in the
  hopes that maintaining them would become easier
  than the many nested if, else we have in cpp right now.
@pranaysashank pranaysashank force-pushed the ghc-module-per-version branch from 0c68bb3 to 0aa9fad Compare August 27, 2022 13:50
@pranaysashank pranaysashank marked this pull request as draft August 27, 2022 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants