Skip to content

Use Metadata::is_dir #1189

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 2, 2018
Merged

Use Metadata::is_dir #1189

merged 1 commit into from
Jan 2, 2018

Conversation

nox
Copy link
Contributor

@nox nox commented Dec 11, 2017

No description provided.

@fitzgen
Copy link
Member

fitzgen commented Dec 12, 2017

@bors-servo r+

@bors-servo
Copy link

📌 Commit 13a78db has been approved by fitzgen

@bors-servo
Copy link

⌛ Testing commit 13a78db with merge 868627b...

bors-servo pushed a commit that referenced this pull request Dec 12, 2017
@nox
Copy link
Contributor Author

nox commented Dec 12, 2017

For the record, my use case for this was passing /dev/null as the input header, and passing multiple actual headers I care about with -- -include.

@fitzgen
Copy link
Member

fitzgen commented Dec 12, 2017

For the record, my use case for this was passing /dev/null as the input header, and passing multiple actual headers I care about with -- -include.

If you pass multiple headers they automatically get -included fwiw.

@nox
Copy link
Contributor Author

nox commented Dec 12, 2017

It didn't work from command line, but maybe I did it wrong, it was 2am after all. :)

@fitzgen
Copy link
Member

fitzgen commented Dec 12, 2017

It may only work with the builder right now.

@bors-servo
Copy link

💔 Test failed - status-travis

@emilio
Copy link
Contributor

emilio commented Jan 2, 2018

@bors-servo retry

@bors-servo
Copy link

⌛ Testing commit 13a78db with merge ac52323...

bors-servo pushed a commit that referenced this pull request Jan 2, 2018
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: fitzgen
Pushing ac52323 to master...

@bors-servo bors-servo merged commit 13a78db into rust-lang:master Jan 2, 2018
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.

5 participants