-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(rust): overrides should only check full segments #3668
Conversation
it would be awesome if someone could test this on windows, @davidkna perhaps? |
c83523f
to
252013b
Compare
252013b
to
9910cd8
Compare
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.
LGTM
The ninja type-change from variable shadowing. Haven't seriously thought about that one in a while. Thanks @syphar for the fix and for getting this moving! |
1 similar comment
The ninja type-change from variable shadowing. Haven't seriously thought about that one in a while. Thanks @syphar for the fix and for getting this moving! |
Description
Fix fixes a bug introduced in #3359, an override for
~/some/folder
was also used for~/some/folder_with_suffix
. Not the first time that the difference ofPath::starts_with
andstr::starts_with
bites someone (Path::starts_with
only matches full segments).How Has This Been Tested?
Checklist: