Skip to content

Replacearch with apk --print-arch in Dockerfile-alpine.template - #1142

Merged
PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:update-functions.sh
Oct 31, 2019
Merged

Replacearch with apk --print-arch in Dockerfile-alpine.template#1142
PeterDaveHello merged 1 commit into
nodejs:masterfrom
PeterDaveHello:update-functions.sh

Conversation

@PeterDaveHello

Copy link
Copy Markdown
Member

@PeterDaveHello
PeterDaveHello requested a review from a teamOctober 29, 2019 07:29
@PeterDaveHello

Copy link
Copy Markdown
MemberAuthor

cc #1026

Comment threadfunctions.sh Outdated
function get_arch() {
local arch
case $(uname -m) in
case $(apk --print-arch) in

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.

this only works for alpine, yes?

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.

I think so

@tianon

Copy link
Copy Markdown
Contributor

I meant in the Dockerfile itself -- I'm not sure what this script is for?

@PeterDaveHelloPeterDaveHello changed the title Use apk --print-arch instead of uname -m in functions.shReplacearch with apk --print-arch in Dockerfile-alpine.templateOct 29, 2019
@PeterDaveHello

Copy link
Copy Markdown
MemberAuthor

I misunderstood, should be fixed now.

@nschonni

Copy link
Copy Markdown
Member

Should all the alpine images be updated, or just wait for the next versions?

@PeterDaveHello

Copy link
Copy Markdown
MemberAuthor

I think it's for the next version, no need to change it immediately. @tianon what do you think?

@PeterDaveHello
PeterDaveHello merged commit ac369e6 into nodejs:masterOct 31, 2019
@PeterDaveHello
PeterDaveHello deleted the update-functions.sh branch October 31, 2019 06:50
Sign up for freeto 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.

5 participants

@PeterDaveHello@tianon@nschonni@LaurentGoderre@SimenB