Skip to content

Tag all files as being for Windows only.#132

Closed
ijc wants to merge 1 commit intomicrosoft:masterfrom
ijc:master
Closed

Tag all files as being for Windows only.#132
ijc wants to merge 1 commit intomicrosoft:masterfrom
ijc:master

Conversation

@ijc
Copy link

@ijc ijc commented Jul 11, 2017

This prevents e.g. go guru when running on a different platform tripping over
these files even when the build tags are correctly set. In my case this was the
case on the github.com/containerd/containerd code base due to hcsshim being
vendored there.

I must confess to a small naughtiness which is that since I was unable to run
mksyscall_windows on my Linux machine I have hand edited both the generator and
the generated file.

This prevents e.g. go guru when running on a different platform tripping over
these files even when the build tags are correctly set. In my case this was the
case on the github.com/containerd/containerd code base due to hcsshim being
vendored there.

I must confess to a small naughtiness which is that since I was unable to run
mksyscall_windows on my Linux machine I have hand edited both the generator and
the generated file.
@msftclas
Copy link

@ijc,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@ijc
Copy link
Author

ijc commented Jul 12, 2017

Thank's @friism for the carry in #133.

@ijc ijc closed this Jul 12, 2017
dcantah pushed a commit to dcantah/hcsshim that referenced this pull request Mar 17, 2021
Fixes an issue on cleanup with mapped directories
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.

2 participants