Skip to content

Fix build-cli-bin - #4876

Merged
kleimkuhler merged 2 commits into
linkerd:mainfrom
aliariff:fix-build-cli-bin
Aug 13, 2020
Merged

Fix build-cli-bin#4876
kleimkuhler merged 2 commits into
linkerd:mainfrom
aliariff:fix-build-cli-bin

Conversation

@aliariff

Copy link
Copy Markdown
Contributor

Signed-off-by: Ali Ariff <ali.ariff12@gmail.com>
Signed-off-by: Ali Ariff <ali.ariff12@gmail.com>
@aliariff
aliariff requested a review from a team as a code owner August 13, 2020 12:32

@alpeb alpeb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Works for me, thanks @aliariff 👍
Would be awesome if the os+arch detection could be moved into a separate library (say ./bin/_os.sh) to avoid the repetition 😉 😉 (if you're up to it that'd be nice to do in a separate PR).

@aliariff

Copy link
Copy Markdown
Contributor Author

@alpeb yap that's the plan, after both of the PR is merged, then the duplication will be addressed

@kleimkuhler kleimkuhler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome thanks @aliariff this works well for me 👍

@kleimkuhler
kleimkuhler merged commit 66d2c6b into linkerd:main Aug 13, 2020
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.

Update bin/build-cli-bin to build binary based off architecture

3 participants