Skip to content

Use same find -perm syntax as rustc #1168

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

Merged
merged 1 commit into from
Jan 14, 2015
Merged

Conversation

alexcrichton
Copy link
Member

Closes #822

@rust-highfive
Copy link

r? @huonw

(rust_highfive has picked a reviewer for you, use r? to override)

@huonw
Copy link
Member

huonw commented Jan 14, 2015

It seems this is a step backwards? rust-lang/rust#19981

@alexcrichton
Copy link
Member Author

I guess it may be a little better than +111, but rust-lang/rust#19983 didn't land b/c it didn't work on windows. I'm not actually sure how to do this cross platform-wise...

@huonw
Copy link
Member

huonw commented Jan 14, 2015

Oh. 😢

@bors: r+ fd781a1

@bors
Copy link
Contributor

bors commented Jan 14, 2015

⌛ Testing commit fd781a1 with merge 686fe90...

@bors
Copy link
Contributor

bors commented Jan 14, 2015

💔 Test failed - cargo-win-64

@alexcrichton
Copy link
Member Author

@bors: retry

@bors
Copy link
Contributor

bors commented Jan 14, 2015

⌛ Testing commit fd781a1 with merge 6c22ca1...

bors added a commit that referenced this pull request Jan 14, 2015
@bors
Copy link
Contributor

bors commented Jan 14, 2015

☀️ Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64

@bors bors merged commit fd781a1 into rust-lang:master Jan 14, 2015
@alexcrichton alexcrichton deleted the issue-822 branch January 20, 2015 06:42
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.

Invalid find -perm test in the Makefile
4 participants