Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
sdk arm64 images #2521
sdk arm64 images #2521
Changes from all commits
680f316
0281ab4
a4b2b5c
2a94635
1c4fa4c
ba446c3
bc21738
00192de
2718501
a568924
c55df8c
a16e560
fd8f24e
b07d874
d9a344c
7e72b16
8ddbbdd
1993529
2e0db94
adef581
7d0b00e
1b3ee2a
12019d4
4661fb6
a5c8069
9b6cf0d
1fe21fb
853e164
0f360d9
588e182
218c81f
191d411
9920f73
65bd25b
f7c9481
3fa1b38
bbd1490
7623c4d
54a6d8f
a7610da
11891ef
415a44a
1c36f27
5b5f4d9
f54efc5
439fdce
e12de03
49020b9
f064809
0a7fd8d
2b477a1
01c06cf
4f8d5e8
c8faf7c
9416890
980ea1b
c1d2765
7d176ce
a684f1c
db76522
be2015f
8489fa2
52ddde2
09a3b15
2239529
2600247
0a0e913
b37a284
d006212
6367cc5
a0fe3d3
42ebbf2
c299380
aa9772b
ba62fd8
6f735d1
b258453
49d7503
37874bb
0278654
404738e
1f3a184
c72abe0
b59ccf0
244671b
26972b0
4632cbe
01b1098
7dcfa4b
a481d42
2b85666
5726bf5
45c4ec2
06b4f10
a24e7fd
c69489a
5249ef3
a667bd1
2b27ace
c2ed4ed
fa32f51
42b5645
c71e622
b0a3095
c391778
0c087c6
cdf9e95
e4e10d9
5fe6db7
f57db57
0de6411
5746995
6ebd31b
f88e15e
97c9e83
822c4c3
6d9e0ef
140816b
2d91f88
2c4574e
9986756
030b09b
e80be72
dae4c85
af45e9d
3b1e6c4
ec06756
7a0c947
6e2082d
855d17d
09f19f8
01b960a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Oh boy this is fun. I hope we don't have to include another architecture down the line 😆
Not for this PR, but I wonder if we could eventually do some kind of a list, so that you could do something more like make
make build ARCH=windows,arm64,amd64
rather than having all theseWITH_*
statements.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.
I believe that this should depend on
build-agones-sdk-image-amd64
instead of having the build image. I tried running this target (having modified the name of the image locally so it wouldn't exist) and got an error:make: *** [Makefile:521: push-agones-sdk-linux-image-amd64] Error 1
For some reason the arm and windows targets above didn't have this issue, and I'm not sure why (maybe because they use buildx?).
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.
I tested this change locally and it had the desired effect.
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.
I'm going to merge this PR as-is and will send a follow up PR to tweak this line.
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.
#2564