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

chore: update build args to arm64 #157

Merged
merged 7 commits into from
Mar 29, 2023
Merged

Conversation

shiki-tak
Copy link
Contributor

@shiki-tak shiki-tak commented Mar 28, 2023

#155

I changed the build args default to M1 mac.

To run it on an M1 mac simply run: make build-docker

For Intel mac, specify args as follows and execute.

make build-docker ARCH=x86_64 PLATFORM=amd64

Change default to intel mac.
For M1 mac, please see README.

Description

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

@shiki-tak shiki-tak requested a review from zemyblue March 28, 2023 03:06
Copy link
Member

@zemyblue zemyblue left a comment

Choose a reason for hiding this comment

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

Thank you for supporting macos M1.

And please change the target branch to rc/v1.0.0-rc0 and update changelog.

@zemyblue zemyblue linked an issue Mar 28, 2023 that may be closed by this pull request
Makefile Outdated Show resolved Hide resolved
@shiki-tak shiki-tak changed the base branch from main to rc/v1.0.0-rc0 March 28, 2023 03:18
Copy link
Member

@zemyblue zemyblue left a comment

Choose a reason for hiding this comment

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

Please update README for M1 user like below.

If your platform is macos m1, please add ARCH option in build-docker
make build-docker ARCH=aarch64

@shiki-tak
Copy link
Contributor Author

Please update README for M1 user like below.

If your platform is macos m1, please add ARCH option in build-docker
make build-docker ARCH=aarch64

I added a note to the README that the build default is Intel mac.

zemyblue
zemyblue previously approved these changes Mar 28, 2023
jaeseung-bae
jaeseung-bae previously approved these changes Mar 28, 2023
Makefile Outdated Show resolved Hide resolved
@shiki-tak shiki-tak dismissed stale reviews from jaeseung-bae and zemyblue via 2270ba1 March 29, 2023 04:06
Copy link
Contributor

@torao torao left a comment

Choose a reason for hiding this comment

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

LGTM

@shiki-tak shiki-tak merged commit f950cd8 into rc/v1.0.0-rc0 Mar 29, 2023
@shiki-tak shiki-tak deleted the feat/update-m1-build-args branch March 29, 2023 05:44
zemyblue added a commit to zemyblue/finschia that referenced this pull request Apr 21, 2023
* rc/v1.0.0-rc0: (26 commits)
  ci: add a ci to build darwin version when add tag (Finschia#180)
  chore: Release v1.0.0-rc7 (Finschia#179)
  Bump github.com/Finschia/finschia-sdk from v0.47.0-rc6 to v0.47.0-rc7 (Finschia#178)
  chore: Release v1.0.0-rc6 (Finschia#177)
  build: rename and bump up dependencies (Finschia#176)
  chore: bump up lbm-sdk from v0.47.0-rc5 to v0.47.0-rc5.0.20230414034539-489c442416cd (Finschia#174)
  chore: Release v1.0.0-rc5 (Finschia#173)
  Bump github.com/line/lbm-sdk from v0.47.0-rc4.0.20230410115856-b8421116b3f2 to v0.47.0-rc5 (Finschia#172)
  chore: apply wasmd v0.1.2 (Finschia#171)
  fix license copyright holder and typo (Finschia#170)
  Bump github.com/line/lbm-sdk from v0.47.0-rc4 to v0.47.0-rc4.0.20230410115856-b8421116b3f2 (Finschia#168)
  Bump up github.com/line/lbm-sdk from v0.47.0-rc3 to v0.47.0-rc4 (Finschia#167)
  chore: release v1.0.0-rc3 (Finschia#166)
  feat: bump up ibc-go v3.3.2 and wasmd for changing ibc light client of Ostracon to Tendermint (Finschia#164)
  chore: update license notice and code_of_conduct (Finschia#165)
  Bump github.com/line/lbm-sdk from v0.47.0-rc2 to v0.47.0-rc3 (Finschia#162)
  fix: fix `release-build` ci error occurred when adding assets after tagging (Finschia#163)
  chore: release v1.0.0-rc2 (Finschia#160)
  Bump github.com/line/lbm-sdk from v0.47.0-rc1 to v0.47.0-rc2 (Finschia#159)
  chore: update build args to arm64 (Finschia#157)
  ...
@zemyblue zemyblue mentioned this pull request Apr 25, 2023
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.

fail to build docker image as make build-docker
4 participants