Skip to content

fix: update rest of the Dockerfiles with the generator, introduce generator shortcut script - #160

Merged
develop7 merged 1 commit into
haskell:masterfrom
develop7:push-xwrswlzlxqxy
Oct 10, 2025
Merged

develop7 merged 1 commit into
haskell:masterfrom
develop7:push-xwrswlzlxqxy

Conversation

@develop7

@develop7 develop7 commented Oct 3, 2025 •

Copy link
Copy Markdown
Collaborator

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
Comment thread 9.10/slim-bullseye/Dockerfile
@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

develop7 commented Oct 7, 2025

Copy link
Copy Markdown
Collaborator Author

@chreekat @jhrcek please review

@jhrcek jhrcek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Comment thread .gitignore

Copy link
Copy Markdown
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)

@chreekat

Copy link
Copy Markdown

@develop7 can you rebase on master and drop the duplicate commit?

@develop7

Copy link
Copy Markdown
Collaborator Author

@chreekat done, merging

@develop7
develop7 merged commit 0beea79 into haskell:master Oct 10, 2025
14 of 15 checks passed
@develop7
develop7 deleted the push-xwrswlzlxqxy branch November 3, 2025 14:15
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