handle components with slashes in their names

RESOLVED FIXED

Status

Developer Services
Mercurial: bzexport
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: sfink, Assigned: sfink)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Components can legitimately contain the '/' character. Don't interpret --component "foo/bar" as product=foo component=bar if it is an exact match.
(Assignee)

Comment 1

6 years ago
Created attachment 602113 [details] [diff] [review]
Don't interpret --component "foo/bar" as product=foo component=bar if it is an exact match
Attachment #602113 - Flags: review?(josh)

Comment 2

6 years ago
Comment on attachment 602113 [details] [diff] [review]
Don't interpret --component "foo/bar" as product=foo component=bar if it is an exact match

>+    if component is not None and product is None:

Is there a reason this can't be |if component and not product|?
Attachment #602113 - Flags: review?(josh) → review+
(Assignee)

Comment 3

6 years ago
(In reply to Josh Matthews [:jdm] from comment #2)
> Comment on attachment 602113 [details] [diff] [review]
> Don't interpret --component "foo/bar" as product=foo component=bar if it is
> an exact match
> 
> >+    if component is not None and product is None:
> 
> Is there a reason this can't be |if component and not product|?

Silly paranoia. What if you had a component named '0'? Think of the children!

Oh, wait. This is Python, not Perl. Never mind; fixed.

http://hg.mozilla.org/users/tmielczarek_mozilla.com/bzexport/rev/9ee8ce05c83a
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: Other Applications → Developer Services
You need to log in before you can comment on or make changes to this bug.