Skip to content

Conversation

@mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Jan 2, 2017

Migrate the syscall to golang.org/x/sys/unix (see: https://golang.org/pkg/syscall).
Also use Mtimespec instead of Mtim so this library compile on darwin.

@mfojtik
Copy link
Contributor Author

mfojtik commented Jan 2, 2017

@nalind PTAL

@nalind
Copy link
Member

nalind commented Jan 3, 2017

LGTM, merging.

@nalind nalind merged commit f8371fb into containers:master Jan 3, 2017
@mfojtik
Copy link
Contributor Author

mfojtik commented Jan 4, 2017

@nalind you should not trust me :-) i forgot to try to run this on linux and it breaks for some reason... Can you please revert this?

@runcom
Copy link
Member

runcom commented Jan 4, 2017

This seems to be why PRs in c/image are failing containers/image#195

@mfojtik
Copy link
Contributor Author

mfojtik commented Jan 4, 2017

@runcom for some reason this works on OSX but not on Linux :-/ I wonder if there a way to make this work on both, but for now I think we should revert this commit

@nalind
Copy link
Member

nalind commented Jan 4, 2017

Can I get you to look over #11 and see if it works for you?

@nalind
Copy link
Member

nalind commented Jan 6, 2017

Okay, reverted this one and readded it as the first patch in #11.

Sign up for free to 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.

3 participants