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

Fix be_mode, be_executable, be_owned_by and be_grouped_into matcher on Darwin (OS X) #103

Merged
merged 2 commits into from May 14, 2013

Conversation

Projects
None yet
2 participants
@matsuo
Contributor

matsuo commented May 14, 2013

stat command on OS X doesn't have -c option. This pull request works for me on OS X Mountain Lion v10.8.3.

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy May 14, 2013

Test of "should_not" is not needed."should" is sufficient.

I missed to point that in your previous patch for match_md5checksum matcher.Sorry.

mizzy commented on spec/darwin/commands_spec.rb in 2290ec1 May 14, 2013

Test of "should_not" is not needed."should" is sufficient.

I missed to point that in your previous patch for match_md5checksum matcher.Sorry.

@matsuo

This comment has been minimized.

Show comment
Hide comment
@matsuo

matsuo May 14, 2013

Contributor

Thanks. OK, I have fixed.

Contributor

matsuo commented May 14, 2013

Thanks. OK, I have fixed.

mizzy added a commit that referenced this pull request May 14, 2013

Merge pull request #103 from matsuo/darwin-support
Fix be_mode, be_executable, be_owned_by and be_grouped_into matcher on Darwin (OS X)

@mizzy mizzy merged commit 6bd1b2b into mizzy:master May 14, 2013

1 check passed

default The Travis CI build passed
Details
@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy May 14, 2013

Owner

Tanks a lot!

Owner

mizzy commented May 14, 2013

Tanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment