-
Notifications
You must be signed in to change notification settings - Fork 12.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
libglob: only return dirs for globs ending in /
`foo.txt/` should not return `foo.txt` if `foo.txt` is in fact a text file and not a directory.
- Loading branch information
Showing
2 changed files
with
14 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1700f35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from brson
at ben0x539@1700f35
1700f35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging ben0x539/rust/glob-dots = 1700f35 into auto
1700f35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ben0x539/rust/glob-dots = 1700f35 merged ok, testing candidate = e2c84a7
1700f35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/5158
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/5158
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/4250
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/4264
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/5258
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/4345
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/4353
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/5260
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/4345
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/4350
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/4416
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2145
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/5257
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/4354
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/4366
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/5022
1700f35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = e2c84a7