Deploy new tooltool with per-file setup support (Puppet)

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Details

Attachments

(1 attachment)

Created attachment 8641903 [details]
MozReview Request: Bug 1189948: update tooltool to 568945fb; r?markco

Bug 1189948: update tooltool to 568945fb; r?markco
Attachment #8641903 - Flags: review?(mcornmesser)
Comment on attachment 8641903 [details]
MozReview Request: Bug 1189948: update tooltool to 568945fb; r?markco

The only concern I have is at line 82 and 83 of the new file. Is that a line break between the two lines? And if it is, will it cause an issue? 

I would have made the comment in reviewboard, but currently I am able to login.
Attachment #8641903 - Flags: review?(mcornmesser) → review+
It is a line break, just wrapping the line so that it doesn't get too long.  It won't hurt.
remote:   https://hg.mozilla.org/build/puppet/rev/26d9131d9a32
remote:   https://hg.mozilla.org/build/puppet/rev/7181a047ebac
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Updated

3 years ago
Depends on: 1190860
This got backed out in Bug 1190860

https://hg.mozilla.org/build/puppet/rev/6e7de1c8f6a3
https://hg.mozilla.org/build/puppet/rev/483ee5bd5b8d
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Back in:

remote:   https://hg.mozilla.org/build/puppet/rev/7b60e1dcc9d8
remote:   https://hg.mozilla.org/build/puppet/rev/4d3daf1b854d
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.