Closed Bug 1209568 Opened 9 years ago Closed 7 years ago

Try syntax doesn't cope with the double quotes added by mozreview when using bind-autoland

Categories

(Release Engineering :: General, defect)

Product:
Release Engineering
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INCOMPLETE

People

(Reporter: Gijs, Unassigned)

Details

This is essentially bug 1171729 but it's clearly just not fixed yet, see https://bugzilla.mozilla.org/show_bug.cgi?id=1209561 :

(In reply to :Gijs Kruitbosch from comment #0)
> Examples:
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=11970dcb67a1
> 
> has no tests (despite -u all)
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=102c5636cbc4
> 
> has no talos runs (despite -t all)
> 
> etc. etc.
> 
> 
> The quotes should just not be there.


... but while they're there, it would be quite nice if the try syntax Just Worked. I don't care too much whether both of these bugs get fixed, but at least one of them really really really needs to get fixed. This is wasting my time and machine time.
Chris, do you know someone who could look at this?
Flags: needinfo?(cmanchester)
Bug 1209561 has a patch. The buildbot try parser will be retired with our transition to taskcluster, so I don't think polish there should be a priority.
Flags: needinfo?(cmanchester)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.