Skip to content
This repository was archived by the owner on Aug 29, 2018. It is now read-only.

Conversation

@mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Apr 6, 2017

@stevekuznetsov this is required for openshift/origin#13585 to pass the godep restore test.

Long story short... We are not going to build Origin with these libraries as CGO deps as we will have special build tag that will disable it.. However, godep is stupid and it will try to compile the containers/images which will result in compilation error and test failure...

So.. this is only needed to fix that test. After this merge, I guess I will have to kick new base_ami build.

Xvfb \
yum-utils \
zip
gpgme \
Copy link
Contributor

Choose a reason for hiding this comment

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

alphabetize these

Copy link
Contributor Author

Choose a reason for hiding this comment

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

entire file? because that will make the diff bigger ;-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

:sort done

Copy link
Contributor

Choose a reason for hiding this comment

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

We try to keep it sorted ... not everyone is as strict ;)

@mfojtik mfojtik force-pushed the add_signature_dep branch from 43f438f to b4e2ee2 Compare April 6, 2017 14:01
@stevekuznetsov stevekuznetsov merged commit b9ebdaf into openshift:master Apr 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants