-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Implement support for packwiz modpacks #1163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This is a working PR, and still needs testing. |
|
Tested PR build with the following options: docker run --rm -it -v $(PWD)/temp:/data \
-e 'TYPE=FABRIC' \
-e 'EULA=TRUE' \
-e 'PACKWIZ_URL=https://pepsidogs.github.io/pepsipack/pack.toml' \
docker.io/itzg/minecraft-server:test-itzg-1545903355First run successful: Second run successful: |
|
Tested overriding pack definitions (and adding additional mods) using the common docker run --rm -it -v $(PWD)/temp:/data \
-e 'TYPE=FABRIC' \
-e 'EULA=TRUE' \
-e 'PACKWIZ_URL=https://pepsidogs.github.io/pepsipack/pack.toml' \
-e 'MODPACK=https://pepsi.dog/minecraft/files/mods_v1.zip' \
docker.io/itzg/minecraft-server:test-itzg-1545903355Successful: |
|
@itzg I'm happy with the support of this in place so far. Happy to make changes to the code location and or execution order if desired! |
See https://packwiz.infra.link/ for packwiz information This commit adds support for packwiz server modpack definitions provided by the PACKWIZ_URL environment variable. Packwiz modpack definitions are processed before other mod definitions (MODPACK, MODS, etc.) to allow for additional processing/overrides the operator may want to perform (in case of mods not available via Modrinth/CurseForge, or where the pack is not maintained by the operator). We attempt to retrieve the latest bootstrap jar from Github if it doesn't already exist. If for any reason retrieving release information from Github isn't possible, we'll use the existing bootstrap jar if already available. Packwiz is pre-configured to show no GUI and only download server mods.
4c85286 to
d90221a
Compare
|
Reworked that commit to match the 2-space formatting used elsewhere in the file. |
itzg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Thanks for adding this and introducing me to packwiz.
|
This is now merged and published for all of the image variants. |
|
Awesome! I took packwiz for a spin due to a new server with friends for 1.18, and the string of mod updates meant an automated solution was needed. The nice thing here is that the same set of fabric mods can be used against client/server, so automated updates are easier to handle. Created a PR (packwiz/packwiz-website#8) against the packwiz documentation to refer to the section in the docs here. |
|
👀 Thank you for contributing this! I use docker-minecraft-server personally so this is very useful for me :) |
See https://packwiz.infra.link/ for packwiz information
This PR adds support for packwiz server modpack definitions provided via the
PACKWIZ_URLenvironment variable.Packwiz modpack definitions are processed before other mod definitions (
MODPACK,MODS, etc.) to allow for additional processing/overrides the operator may want to perform (in case of mods not available via Modrinth/CurseForge, or where the pack is not maintained by the operator).We attempt to retrieve the latest bootstrap jar from Github if it doesn't already exist. If for any reason retrieving release information from Github isn't possible, we'll use the existing bootstrap jar if already available.
Packwiz is pre-configured to show no GUI and only download server mods. A relevant note to
README.mdhas been added to clarify this, and informs operators to check their packwiz definition is correct in case client mods are downloaded and cause such issues.