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

use platform dependent argument for 'find -perm' (fix #33) #37

Merged
merged 1 commit into from Oct 3, 2013

Conversation

Projects
None yet
3 participants
@dirk-thomas
Copy link
Member

commented Oct 3, 2013

@cottsay @wjwwood Please review

@wjwwood

This comment has been minimized.

Copy link
Member

commented Oct 3, 2013

+1

@wjwwood

This comment has been minimized.

Copy link
Member

commented Oct 3, 2013

I tried it locally.

@cottsay

This comment has been minimized.

Copy link
Member

commented Oct 3, 2013

This is great, thanks so much. Any chance this can make it into Groovy?

@dirk-thomas

This comment has been minimized.

Copy link
Member Author

commented Oct 3, 2013

Yes, will be backported after the merge.

@cottsay

This comment has been minimized.

Copy link
Member

commented Oct 3, 2013

Thank you!

dirk-thomas added a commit that referenced this pull request Oct 3, 2013

Merge pull request #37 from ros/find_perm
use platform dependent argument for 'find -perm' (fix #33)

@dirk-thomas dirk-thomas merged commit e4d91e1 into hydro-devel Oct 3, 2013

@dirk-thomas dirk-thomas deleted the find_perm branch Oct 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.