Nebelhom wrote on 08/21/2013 05:19 PM:

>> -        regex = r'%(DATE)s-(\d+-)+%(BRANCH)s%(L10N)s$' % {
>> +        regex = r'%(DATE)s-(\d+-)+%(BRANCH)s(-l10n)?$' % {
> 
> Well, I guess, after all you are right, but to avoid such
> misinterpretation of the code as I have just done, I would suggest to
> put a comment there for future reference.
> 
> Not sure how to phrase this in one line though... Suggestions?

Then take two lines! ;) But for real just take what you think will be
right. I will have a look at in the next review.

-- 
Henrik Skupin
Software Engineer in Test
Mozilla Corporation