-
Notifications
You must be signed in to change notification settings - Fork 88
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
Added support for ppc64le #255
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
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.
Seems like we could support ALL arches with just
x86_64) ARCH="amd64";; \
*) ARCH="$1";; \
Signed-off-by: Shivani Junawane <shivanij@us.ibm.com>
@nickboldt PR now supports all architectures as suggested... |
@@ -23,6 +23,7 @@ ADD . /go/src/github.com/eclipse/che-operator | |||
RUN case $(uname -m) in \ | |||
x86_64) ARCH="amd64" ;; \ | |||
s390x) ARCH="s390x";; \ | |||
ppc64le) ARCH="ppc64le";; \ |
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 might suggest making this less hard-coded and more generic... for s390x, ppc64le, and even future arches...
x86_64) ARCH="amd64" ;; \
*) ARCH="$1" ;; \
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.
why merging instead of having the suggested proper fix for other arches ?
@@ -23,6 +23,7 @@ ADD . /go/src/github.com/eclipse/che-operator | |||
RUN case $(uname -m) in \ | |||
x86_64) ARCH="amd64" ;; \ | |||
s390x) ARCH="s390x";; \ | |||
ppc64le) ARCH="ppc64le";; \ | |||
esac | |||
|
|||
# do no break RUN lines when building with UBI base images. https://projects.engineering.redhat.com/browse/OSBS-7398 & OSBS-7399 |
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.
FYI this comment is no longer correct. Broken lines are allowed in Brew.
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.
force pushed change is the same as the original PR but whatever.
Followup #266 |
What does this PR do?
I have added support for ppc64le architecture with reference to the PR merged for s390x architecture.