Skip to content
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

Re-vendor, primarily for https://github.com/containers/storage/pull/11 #331

Merged
merged 1 commit into from
May 12, 2017

Conversation

mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Apr 28, 2017

containers/storage got new dependencies, so we will need to re-vendor eventually anyway, and having this separate from other major work is cleaner.

But the primary goal of this commit is to see whether it makes skopeo buildable on OS X.

@mtrmac mtrmac mentioned this pull request Apr 28, 2017
@runcom
Copy link
Member

runcom commented Apr 29, 2017

make check failed

@mtrmac
Copy link
Contributor Author

mtrmac commented Apr 29, 2017

Yeah, I somehow managed to persuade git that there is a submodule. Hopefully fixed.

(And no, we can’t trivially add a macOS target to Travis CI, because Travis does not support Docker on macOS. I suppose we could test make binary-local at least… but I’ll leave that to someone actually using macOS for skopeo development.)

@mtrmac
Copy link
Contributor Author

mtrmac commented May 3, 2017

@runcom tests pass now.

@runcom
Copy link
Member

runcom commented May 3, 2017

is there anyone willing to give this a try for MacOS? @hferentschik perhaps?

containers/storage got new dependencies, so we will need to re-vendor
eventually anyway, and having this separate from other major work is
cleaner.

But the primary goal of this commit is to see whether it makes skopeo
buildable on OS X.
@mtrmac mtrmac merged commit c5adc4b into containers:master May 12, 2017
@mtrmac mtrmac deleted the storage-rebase branch May 12, 2017 16:50
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants