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

Use latest version and fix release file name in Dockerfile.template #558

Merged
merged 2 commits into from
Apr 24, 2024

Conversation

drskullster
Copy link
Contributor

Change-type: patch

@drskullster drskullster marked this pull request as draft April 23, 2024 10:19
@drskullster drskullster force-pushed the drskullster/fix-release-name-in-dockerfile branch from aba810b to 28c91c7 Compare April 24, 2024 10:15
@rahul-thakoor rahul-thakoor force-pushed the drskullster/fix-release-name-in-dockerfile branch 3 times, most recently from 55c2827 to 0aa806e Compare April 24, 2024 11:07
@drskullster drskullster force-pushed the drskullster/fix-release-name-in-dockerfile branch from f4c09b3 to 2d4a18a Compare April 24, 2024 12:25
@rahul-thakoor rahul-thakoor force-pushed the drskullster/fix-release-name-in-dockerfile branch 15 times, most recently from d02fb7f to 0b9e419 Compare April 24, 2024 16:30
rahul-thakoor and others added 2 commits April 24, 2024 20:35
Change-type: patch
Co-authored-by: drskullster <[email protected]>
Signed-off-by: Rahul Thakoor <[email protected]>
We currently define the pre-build commands to run individually for each
target. All the commands are same, and we do not run target specific
commands. This commit defines the pre-build commands using the
[build config](https://github.com/cross-rs/cross/wiki/Configuration#build)

Signed-off-by: Rahul Thakoor <[email protected]>
@rahul-thakoor rahul-thakoor force-pushed the drskullster/fix-release-name-in-dockerfile branch from 0b9e419 to c0eb92d Compare April 24, 2024 16:38
@rahul-thakoor rahul-thakoor changed the title Bump version and fix release file name in Dockerfile.template Use latest version and fix release file name in Dockerfile.template Apr 24, 2024
@rahul-thakoor rahul-thakoor marked this pull request as ready for review April 24, 2024 16:59
@rahul-thakoor rahul-thakoor requested a review from pipex April 24, 2024 17:03
BINARY_ARCH_NAME="aarch64-unknown-linux-gnu" ;; \
"amd64") \
BINARY_ARCH_NAME="x86_64-unknown-linux-gnu" ;;\
"armv7hf") \
Copy link
Contributor

Choose a reason for hiding this comment

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

No i386 😬?

@pipex
Copy link
Contributor

pipex commented Apr 24, 2024

LGTM!

@rahul-thakoor rahul-thakoor merged commit 08b9d71 into master Apr 24, 2024
50 checks passed
@rahul-thakoor rahul-thakoor deleted the drskullster/fix-release-name-in-dockerfile branch April 24, 2024 17:17
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