Skip to content

Conversation

develop7
Copy link
Collaborator

@develop7 develop7 commented Oct 3, 2025

This is in fact a whole pack of changes, here they go:

  • extracted GHC & cabal-install-related data to separate YAML files
  • introduced generate.sh, a wrapper script to run the generator
  • introduced support for additional distributions (e.g. 9.4/*bullseye)
  • tweaked Dockerfile template to improve readability further
  • regenerated rest of Dockerfiles against updated template

@develop7 develop7 requested a review from jhrcek October 3, 2025 13:25
@develop7 develop7 changed the title fix: update rest of the Dockerfiles with the generator fix: update rest of the Dockerfiles with the generator, introduce generator shortcut script Oct 7, 2025
@develop7 develop7 force-pushed the push-xwrswlzlxqxy branch 2 times, most recently from 390ae0b to 1bd8dc1 Compare October 7, 2025 15:15
@develop7 develop7 marked this pull request as ready for review October 7, 2025 15:26
@develop7 develop7 requested a review from chreekat October 7, 2025 15:26
@develop7
Copy link
Collaborator Author

develop7 commented Oct 7, 2025

@chreekat @jhrcek please review

Copy link
Collaborator

@jhrcek jhrcek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this change is desirable given the comment on the line above.. (though I have no idea what purpose those two '\r's are supposed to have)

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.

3 participants