Closed
Bug 1271153
Opened 8 years ago
Closed 8 years ago
Adjust automation for new file mail/config/version_display.txt
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1175331 +++
Just like with Firefox, do the same with Thunderbird.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8750120 -
Flags: review?(rail)
Attachment #8750120 -
Flags: review?(jlund)
Assignee | ||
Updated•8 years ago
|
Comment 2•8 years ago
|
||
Comment on attachment 8750120 [details] [diff] [review]
0001-Bug-1271153-Adjust-automation-for-new-file-mail-conf.patch
Sounds like we need to land version_disaplay.txt first to https://hg.mozilla.org/releases/comm-beta/file/tip/mail/config and https://hg.mozilla.org/releases/comm-esr45/file/tip/mail/config
Comment 3•8 years ago
|
||
Comment on attachment 8750120 [details] [diff] [review]
0001-Bug-1271153-Adjust-automation-for-new-file-mail-conf.patch
clearing r? while details (https://bugzilla.mozilla.org/show_bug.cgi?id=1271153#c2) are discussed
Attachment #8750120 -
Flags: review?(jlund)
Comment 5•8 years ago
|
||
Comment on attachment 8750120 [details] [diff] [review]
0001-Bug-1271153-Adjust-automation-for-new-file-mail-conf.patch
Review of attachment 8750120 [details] [diff] [review]:
-----------------------------------------------------------------
The patch lgtm, but we cannot land it until we get the in-tree files checked in.
Attachment #8750120 -
Flags: review?(rail)
Assignee | ||
Comment 7•8 years ago
|
||
Comment on attachment 8750120 [details] [diff] [review]
0001-Bug-1271153-Adjust-automation-for-new-file-mail-conf.patch
The bug 1271151 landed!
(this can wait after the workweek)
Attachment #8750120 -
Flags: review?(rail)
Comment 8•8 years ago
|
||
Since bug 1271151 had a string addition, that's not possible to land as such on esr45. You'll need to adjust this patch, or land a no-strings version of bug 1271151 for esr.
Assignee | ||
Comment 9•8 years ago
|
||
Right, thanks!
Attachment #8750120 -
Attachment is obsolete: true
Attachment #8750120 -
Flags: review?(rail)
Attachment #8762404 -
Flags: review?(rail)
Comment 10•8 years ago
|
||
Comment on attachment 8762404 [details] [diff] [review]
0001-Bug-1271153-Adjust-automation-for-new-file-mail-conf.patch
The patch itself is correct, but automation will fail, because there is not version_display.txt in:
https://hg.mozilla.org/releases/comm-esr45/file/tip/mail/config
https://hg.mozilla.org/releases/comm-beta/file/tip/mail/config
We need the there before we land this patch.
Attachment #8762404 -
Flags: review?(rail)
Assignee | ||
Comment 11•8 years ago
|
||
It won't land in esr45.
What do you recommend?
* Wait until it lands in beta and apply the patch in both at this time
* Apply it on nightly now and do the same in beta when we can
Comment 12•8 years ago
|
||
(In reply to Sylvestre Ledru [:sylvestre] from comment #11)
> What do you recommend?
> * Wait until it lands in beta and apply the patch in both at this time
> * Apply it on nightly now and do the same in beta when we can
Can we uplift the patch in bug 1271151 to comm-beta? If not, we can just create version_display.txt now and land the buildbot patch - it won't hut having that file and not using it.
Assignee | ||
Comment 13•8 years ago
|
||
Probably. Magnus, would it be ok?
Flags: needinfo?(mkmelin+mozilla)
Assignee | ||
Comment 15•8 years ago
|
||
Rail, are you going to take care of that or do you want me to do it? Merci
Flags: needinfo?(rail)
Comment 16•8 years ago
|
||
The patch is assigned to you, this is why I didn't bothered myself to look at this ;)
I don't mind porting this, just throw it my way.
Flags: needinfo?(rail)
Comment 17•8 years ago
|
||
Can we move forward with this, as now version_display.txt is wrong after today's merges
Comment 18•8 years ago
|
||
(Oh, didn't realize it was just a few days ago you posted.)
Assignee | ||
Comment 20•8 years ago
|
||
https://hg.mozilla.org/build/buildbot-configs/rev/06d19bac0eff19ccf2cf930aab5d0e8f5f8b13d3
[PATCH] Bug 1271153 - Adjust automation for the version_display.txt new file on tb r=rail
Assignee | ||
Comment 21•8 years ago
|
||
Fixed by hand in this comment for comm-central:
https://hg.mozilla.org/comm-central/rev/60e152b2aa6c
> Can we uplift the patch in bug 1271151 to comm-beta? If not, we can just
> create version_display.txt now and land the buildbot patch - it won't hut
> having that file and not using it.
Rail, is it going to fail in beta if the file doesn't exist?
Flags: needinfo?(rail)
Assignee | ||
Comment 22•8 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/a08bdac74f35aef9b0b56e0f8d025be5da0cb449
Bug 1271153 - Update the version_display. Should be managed by automation in the future a=sylvestre DONTBUILD
Comment 23•8 years ago
|
||
(In reply to Sylvestre Ledru [:sylvestre] from comment #21)
> Fixed by hand in this comment for comm-central:
> https://hg.mozilla.org/comm-central/rev/60e152b2aa6c
>
>
> > Can we uplift the patch in bug 1271151 to comm-beta? If not, we can just
> > create version_display.txt now and land the buildbot patch - it won't hut
> > having that file and not using it.
> Rail, is it going to fail in beta if the file doesn't exist?
yup, we should add that file
Flags: needinfo?(rail)
Assignee | ||
Comment 24•8 years ago
|
||
https://hg.mozilla.org/releases/comm-beta/rev/f432b2d0ec76e40131940aa710954c8baa5801bd
Bug 1271153 -Add the file version_display.txt, not going to be used by the product but this will simplify the automation r=rail a=sylvestre
Assignee | ||
Comment 26•8 years ago
|
||
I think this is fixed now
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(sledru)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•