Skip to content

Open files with CLOEXEC#273

Merged
nathany merged 3 commits intofsnotify:masterfrom
linxiulei:cloexec
Oct 5, 2019
Merged

Open files with CLOEXEC#273
nathany merged 3 commits intofsnotify:masterfrom
linxiulei:cloexec

Conversation

@linxiulei
Copy link
Copy Markdown
Contributor

@linxiulei linxiulei commented Oct 25, 2018

What does this pull request do?

Fix #272

Where should the reviewer start?

How should this be manually tested?

No

Copy link
Copy Markdown
Contributor

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@cpuguy83
Copy link
Copy Markdown
Contributor

Looks like darwin got support in 10.7, and (free) BSD in 8.2... seems ok to add this in implicitly.

@nathany
Copy link
Copy Markdown
Contributor

nathany commented Oct 5, 2019

Thanks

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.

Race with golang exec

3 participants