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

killsnoop: s/failed opens/failed kill syscalls/ #611

Merged
merged 2 commits into from
Jul 14, 2016
Merged

killsnoop: s/failed opens/failed kill syscalls/ #611

merged 2 commits into from
Jul 14, 2016

Conversation

cdown
Copy link
Contributor

@cdown cdown commented Jul 13, 2016

I think this is a bad copy/paste from opensnoop, if I'm not mistaken.

@cdown
Copy link
Contributor Author

cdown commented Jul 13, 2016

Please hold off on merging until I've got approval from legal to contribute (as trivial as this may be).

@brendangregg
Copy link
Member

Yes, thanks. LGTM.

@brendangregg
Copy link
Member

If you have an update branch button, can you please click it? I can't merge ATM as it is out of date.

@cdown
Copy link
Contributor Author

cdown commented Jul 13, 2016

I'll get back once I've got the goahead from legal (yes, this is tiny and documentation only, but...)

@cdown cdown closed this Jul 13, 2016
@cdown cdown reopened this Jul 14, 2016
@cdown
Copy link
Contributor Author

cdown commented Jul 14, 2016

Ok, this is good to go. I've also rebased it so it should be mergeable now.

@brendangregg brendangregg merged commit a1333bc into iovisor:master Jul 14, 2016
@brendangregg
Copy link
Member

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.

2 participants