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
Create pkg/system and move stuff there from archive #4278
Conversation
@crosbymichael will this conflict with your branch ? |
import "syscall" | ||
|
||
func getLastAccess(stat *syscall.Stat_t) syscall.Timespec { | ||
return stat.Atimespec |
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.
This will probably not compile on darwin and BSD. You might want to return an empty Timespec
or even better return am error when it is not supported
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.
Like 47c36f2?
Yep, it does not conflict . |
Running ---> Making bundle: cross (in bundles/0.8.1-dev/cross)
Created symlinks: /go/src/github.com/dotcloud/docker/bundles/0.8.1-dev/cross/linux/amd64/docker-0.8.1-dev
Created binary: /go/src/github.com/dotcloud/docker/bundles/0.8.1-dev/cross/linux/386/docker-0.8.1-dev
Created binary: /go/src/github.com/dotcloud/docker/bundles/0.8.1-dev/cross/linux/arm/docker-0.8.1-dev
# github.com/dotcloud/docker/pkg/system
pkg/system/utimes_unsupported.go:8: undefined: ErrNotImplemented
pkg/system/utimes_unsupported.go:12: undefined: ErrNotImplemented
pkg/system/xattrs_unsupported.go:5: imported and not used: "syscall"
pkg/system/xattrs_unsupported.go:8: undefined: ErrNotImplemented
pkg/system/xattrs_unsupported.go:12: undefined: ErrNotImplemented |
ping @alexlarsson |
This is a package for generic system calls etc that for some reason is not yet supported by "syscall", or where it is different enough for the different ports to need portability wrappers. Docker-DCO-1.1-Signed-off-by: Alexander Larsson <alexl@redhat.com> (github: alexlarsson)
New version passes make cross |
LGTM ping @creack @crosbymichael @vieux |
LGTM |
Create pkg/system and move stuff there from archive
This is a package for generic system calls etc that for some reason
is not yet supported by "syscall", or where it is different enough
for the different ports to need portability wrappers.
Docker-DCO-1.1-Signed-off-by: Alexander Larsson alexl@redhat.com (github: alexlarsson)
One reason for this other than pure cleanup is that the overlayfs backend also needs a bunch of these calls.