Change default patch level for tryserver to 1

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
10 years ago
5 years ago

People

(Reporter: johnath, Assigned: johnath)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 347118 [details] [diff] [review]
Is it this simple?

More than once since moving to Hg I have screwed up a tryserver build by attaching a patch and hitting submit, forgetting to change the default patch level from 0 to 1, to match the way mercurial does patches.

Given that this is likely to be the most common use case, it feels like it is appropriate to flip the default.
Attachment #347118 - Flags: review?(bhearsum)
Comment on attachment 347118 [details] [diff] [review]
Is it this simple?

Yep....this should do it. Poke me tomorrow and I'll update build.m.o with this change.
Attachment #347118 - Flags: review?(bhearsum) → review+
(Assignee)

Comment 2

10 years ago
Checking in sendchange-ui.pm;
/cvsroot/mozilla/webtools/buildbot-try/sendchange-ui.pm,v  <--  sendchange-ui.pm
new revision: 1.7; previous revision: 1.6
done
Status: NEW → ASSIGNED
(Assignee)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
This has been deployed.

Updated

9 years ago
Component: Try Server → Release Engineering
Product: Webtools → mozilla.org
Version: Trunk → other
QA Contact: try-server → release
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.