-
-
Notifications
You must be signed in to change notification settings - Fork 305
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
Conversation
fix `wget --progress=dot:giga`
fix file not found when using darwin
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.
COME ON. The ARM platform is irrelevant, because of the BTW, container with AMD64 is running normally in macOS Silicon, yes rosetta2. ALL DONE. PLZ check. |
Made some revision:
|
gotcha, thanks. btw, conflicts resolved. |
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.
LGTM
Update: Enhanced compilation scripts
1: Add a build-time architecture parameters below:ARGsDefaultAvailableARCHamd64amd64, arm64OSlinuxlinux, darwinIf we need to build a macOS Silicon image should be thoughdocker 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
andRCON
version numbers.4: Remove the
-q
parameter ofwget
, instead ofwget --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
forRCON
.