use "/" instead of path.sep in owners glob check

use "/" instead of path.sep in owners glob check

R=maruel@chromium.org
BUG=157022
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=163198

[M-A Ruel, 2012-10-20 23:05:23]

https://codereview.chromium.org/11236018/diff/1/owners.py
File owners.py (right):

https://codereview.chromium.org/11236018/diff/1/owners.py#newcode227
owners.py:227: if '/' in glob_string:
personally, I'd also blacklist '\\' for safety.

[Dirk Pranke, 2012-10-20 23:09:28]

On 2012/10/20 23:05:23, Marc-Antoine Ruel wrote:
> https://codereview.chromium.org/11236018/diff/1/owners.py
> File owners.py (right):
> 
> https://codereview.chromium.org/11236018/diff/1/owners.py#newcode227
> owners.py:227: if '/' in glob_string:
> personally, I'd also blacklist '\\' for safety.

Yeah, I've waffled a bit on this since there are cases where '\\' might be
legit, but those can probably be worked around if there's really a demand.

[Dirk Pranke, 2012-10-20 23:09:43]

On 2012/10/20 23:09:28, Dirk Pranke wrote:
> On 2012/10/20 23:05:23, Marc-Antoine Ruel wrote:
> > https://codereview.chromium.org/11236018/diff/1/owners.py
> > File owners.py (right):
> > 
> > https://codereview.chromium.org/11236018/diff/1/owners.py#newcode227
> > owners.py:227: if '/' in glob_string:
> > personally, I'd also blacklist '\\' for safety.
> 
> Yeah, I've waffled a bit on this since there are cases where '\\' might be
> legit, but those can probably be worked around if there's really a demand.

anyway, done.

[M-A Ruel, 2012-10-20 23:23:42]

lgtm

[commit-bot: I haz the power, 2012-10-20 23:36:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/11236018/5001

[commit-bot: I haz the power, 2012-10-20 23:39:02]

Presubmit check for 11236018-5001 failed and returned exit status 1.

************* Module owners
C0301:229,0: Line too long (83/80)
Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit Messages **
--tbr was specified, skipping OWNERS check

** Presubmit ERRORS **
Fix pylint errors first.

Presubmit checks took 152.0s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[commit-bot: I haz the power, 2012-10-20 23:45:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/11236018/2002

[commit-bot: I haz the power, 2012-10-20 23:47:43]

Change committed as 163198