-
Notifications
You must be signed in to change notification settings - Fork 721
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
Adding XML Parser to Docker containers #4870
Adding XML Parser to Docker containers #4870
Conversation
jenkins build docker x86 ubuntu16 |
jenkins build docker s390x ubuntu16 |
jenkins build docker ppc64le centos7 |
5cde97f
to
273a1f5
Compare
jenkins build docker ppc64le centos7 |
jenkins build docker x86 centos6.9 |
273a1f5
to
58d22c9
Compare
I got the tests to compile on:
|
@AdamBrousseau for review |
This is needed in order for our tests to work Moved the dependencies for git to the install section [skip ci] Signed-off-by: Samuel Rubin <samuel.rubin@ibm.com>
58d22c9
to
eafdca1
Compare
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.
LGTM. @smlambert for final review & merge.
I made a mistake. The expat library is needed for all of the ubuntu18s and ubuntu16 on ppc. This caused those docker files to fail their build. I have created #4916 to fix this issue |
Missing expat library caused some of the containers to fail their build Also moved the dependencies for git to the top and identified them [skip ci] Related eclipse-openj9#4870 Signed-off-by: Samuel Rubin <samuel.rubin@ibm.com>
This is needed in order for our tests to work
[skip ci]
Signed-off-by: Samuel Rubin samuel.rubin@ibm.com