Skip to content

Fixed Dockerfile. - #1882

Merged
AngheloAlf merged 6 commits into
zeldaret:mainfrom
Zeri-Ai:main
Aug 30, 2026
Merged

Fixed Dockerfile.#1882
AngheloAlf merged 6 commits into
zeldaret:mainfrom
Zeri-Ai:main

Conversation

@Zeri-Ai

@Zeri-Ai Zeri-Ai commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Dockerfile updated per the file on https://practicerom.com/public/packages/debian/howto.txt and Debian/Ubuntu's apt-key command being deprecated.

Dockerfile updated per the file on https://practicerom.com/public/packages/debian/howto.txt
and Debian/Ubuntu's apt-key command being deprecated.
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated

@AngheloAlf AngheloAlf 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.

We are trying to get everybody who has contributed / wants to contribute to MM decomp to agree to licensing the repo before merging PRs from people who have not yet agreed to it. It is better explained here: louist103/mm-licensing#1.
Could you please read about it and post in that issue if you agree or not. Thanks!

Comment thread Dockerfile Outdated
@Zeri-Ai

Zeri-Ai commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

Removed sudo from the Dockerfile (as it is indeed not required) and agreed to the aforementioned license.

@hensldm hensldm removed the Needs-first-approval First approval label Aug 29, 2026
@AngheloAlf AngheloAlf added Merge-ready All reviewers satisfied, just waiting for CI and removed Needs-second-approval Second approval labels Aug 30, 2026
@AngheloAlf
AngheloAlf merged commit 2d43d21 into zeldaret:main Aug 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merge-ready All reviewers satisfied, just waiting for CI Tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants