Skip to content
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

Update: Enhanced compilation scripts #263

Merged
merged 8 commits into from
Feb 2, 2024
Merged

Conversation

celaraze
Copy link
Contributor

@celaraze celaraze commented Feb 2, 2024

1: Add a build-time architecture parameters below:

ARGs Default Available
ARCH amd64 amd64, arm64
OS linux linux, darwin

If we need to build a macOS Silicon image should be though docker build --build-arg ARCH=arm64 --build-arg OS=darwin -t user/image:tag . .
2: Add priority to determine whether the current architecture is supported, and exit after an exception.
3: Controllable SUPERCRONIC and RCON version numbers.
4: Remove the -q parameter of wget, instead of wget --progress=dot:giga, which is especially important for the visualization of logs during the compilation process.
5: Unified, highly readable code and comments.
6: Add md5sum for RCON.

celaraze and others added 3 commits February 2, 2024 11:35
fix `wget --progress=dot:giga`
fix file not found when using darwin
@Technetium1
Copy link

Why remove checksum validation?

@celaraze
Copy link
Contributor Author

celaraze commented Feb 2, 2024

Why remove checksum validation?

It's hard-coded in the script and only the amd64 binaries are validated, and I'm constantly updating this PR so that multiple binaries across platforms can be checked.

fix sha1sum for supercronic,
but checksum still mssing.
@celaraze
Copy link
Contributor Author

celaraze commented Feb 2, 2024

COME ON. The ARM platform is irrelevant, because of the cm2network/steamcmd image is only amd64 architecture.
deleted for 1 and 2.

BTW, container with AMD64 is running normally in macOS Silicon, yes rosetta2.

ALL DONE. PLZ check.

@thijsvanloef thijsvanloef changed the title Update: Enhanced compilation scripts for multi-platform Update: Enhanced compilation scripts Feb 2, 2024
@thijsvanloef
Copy link
Owner

Made some revision:

  • Add labels to the docker image
  • Sort package installs alphabetically
  • Split Rcon and Supercronic install

@celaraze
Copy link
Contributor Author

celaraze commented Feb 2, 2024

Made some revision:

  • Add labels to the docker image
  • Sort package installs alphabetically
  • Split Rcon and Supercronic install

gotcha, thanks.

btw, conflicts resolved.

Copy link
Owner

@thijsvanloef thijsvanloef left a comment

Choose a reason for hiding this comment

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

LGTM

@thijsvanloef thijsvanloef merged commit 9a5453f into thijsvanloef:main Feb 2, 2024
4 checks passed
@celaraze celaraze mentioned this pull request Feb 22, 2024
3 tasks
MusclePr pushed a commit to MusclePr/palworld-server-docker that referenced this pull request Jun 19, 2024
Update: Enhanced compilation scripts
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