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

Install instructions for m1 macbook #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jared-bloomer
Copy link

These are a set of repeatable install instructions I have put together for compiling and installing on an Apple m1 macbook

@jared-bloomer
Copy link
Author

Documentation is finalized i it's current form

jamestelfer
jamestelfer previously approved these changes May 3, 2023
Copy link
Member

@jamestelfer jamestelfer left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution, and sorry it's languished. I did see it not too long ago, and it gave me some impetus to update the tool and release ARM for Linux and Mac (#20).

Would you like to kick the tyres on the new binary? If it works for you, it would probably change the M1 instructions a fair bit.


8. Mark the current directory as safe in your global git config with the command `git config --global --add safe.directory /usr/local/share/parameter-store-exec`

9. Compile the binary with the command `GOOS=linux GOARCH=arm64 go build -ldflags="-X main.Version=$(git describe --tags --candidates=1 --dirty) -s -w" -o $@ github.com/cultureamp/parameter-store-exec`
Copy link
Member

Choose a reason for hiding this comment

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

This will build for linux rather than darwin -- is that the intent?

@jamestelfer jamestelfer dismissed their stale review May 3, 2023 03:32

Accidental approval

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.

None yet

2 participants