Closed
Bug 713134
Opened 13 years ago
Closed 13 years ago
Make packager.pm missing file warnings fatal on SeaMonkey
Categories
(SeaMonkey :: Build Config, enhancement, P1)
SeaMonkey
Build Config
Tracking
(seamonkey2.9 wontfix, seamonkey2.10 verified)
VERIFIED
FIXED
seamonkey2.11
People
(Reporter: rain1, Assigned: sgautherie)
References
Details
Attachments
(1 file)
616 bytes,
patch
|
Callek
:
review+
Callek
:
feedback+
Callek
:
approval-comm-aurora+
Callek
:
approval-comm-beta-
|
Details | Diff | Splinter Review |
To do this, we need to clean up suite/installer/package-manifest.in, then set MOZ_PKG_FATAL_WARNINGS = 1 before including packager.mk in suite/installer/Makefile.in.
Assignee | ||
Updated•13 years ago
|
Version: SeaMonkey 2.7 Branch → Trunk
Assignee | ||
Comment 1•13 years ago
|
||
Let's prepare for when blockers are fixed.
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #586715 -
Flags: review?(bugspam.Callek)
Assignee | ||
Updated•13 years ago
|
Severity: normal → enhancement
Flags: in-testsuite-
Whiteboard: [landing depends on fixing all blockers first]
Target Milestone: --- → Future
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Ian, maybe you can review this?
Attachment #586715 -
Flags: review?(iann_bugzilla)
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Sorry this is more KaiRo's territory than mine
Attachment #586715 -
Flags: review?(iann_bugzilla) → review?(kairo)
Assignee | ||
Updated•13 years ago
|
Priority: -- → P1
Comment 4•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Review of attachment 586715 [details] [diff] [review]:
-----------------------------------------------------------------
Looks fine, however I won't mark it r+ until after this _CAN_ land. So r- for now, re-request when you feel we're safe to land it.
Attachment #586715 -
Flags: review?(kairo)
Attachment #586715 -
Flags: review?(bugspam.Callek)
Attachment #586715 -
Flags: review-
Attachment #586715 -
Flags: feedback+
Assignee | ||
Comment 5•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
http://build.mozillamessaging.com/tinderboxpushlog/?tree=ThunderbirdTry&rev=e4cb299645a0
succeeds (so far), fwiw.
Attachment #586715 -
Flags: review- → review?(bugspam.Callek)
Assignee | ||
Updated•13 years ago
|
status-seamonkey2.9:
--- → affected
Whiteboard: [landing depends on fixing all blockers first]
Target Milestone: Future → seamonkey2.10
Assignee | ||
Updated•13 years ago
|
Attachment #586715 -
Flags: review?(iann_bugzilla)
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Probably more KaiRo's territory
Attachment #586715 -
Flags: review?(iann_bugzilla) → review?(kairo)
Assignee | ||
Comment 7•13 years ago
|
||
Ping for review.
![]() |
||
Comment 8•13 years ago
|
||
(In reply to Serge Gautherie (:sgautherie) from comment #7)
> Ping for review.
At least in my case, this is a waste of time, I'm marking such things TODO in my mailbox, but I can only work on things like this in my free time, which is pretty sparse since I started working full-time for something other than SeaMonkey. I will get to it, no worries. Also, people are reminded weekly by Bugzilla nowadays about their outstanding review requests anyhow, no need to send bugmail in addition.
Comment 9•13 years ago
|
||
As you know I have no say in this, but personally I feel if all dependencies have been resolved and everything looks OK (cf. comment 5), we should just flip the switch after the next uplift and see how it works. Since it's just a one line change, it's easy to back it out if it hurts.
Assignee | ||
Comment 10•13 years ago
|
||
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #9)
Makes sense (at this point): then my proposal would be to land asap on 2.10/central then wait for 2.9 to go to beta.
Pros: this has stalled for so long (as in already missed previous cycle), spare an approval+landing on aurora, have it active for next central cycle when it is likely to be most useful (I don't want to go chasing these warnings again!)...
![]() |
||
Comment 11•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
I won't mark this as reviewed, as Callek needs to deal with the fallout. I think this would be better to land on Wednesday, i.e. very early in 2.11a1, we're too late in the game for 2.10 and too busy with release stuff to reasonably do this right now, IMHO - but I'll leave that to Callek.
Attachment #586715 -
Flags: review?(kairo)
Comment 12•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
r+ but I want to ensure we're not chasing our own tail when this lands, so please land on wednesday. If all looks fine by friday I'll approve for aurora and we can chase down issues with it there.
Attachment #586715 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 13•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
http://hg.mozilla.org/comm-central/rev/d1ed30542ed0
Attachment #586715 -
Attachment description: (Av1) Make packager.pm missing file warnings fatal on SeaMonkey. → (Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13]
Attachment #586715 -
Flags: approval-comm-beta?
Attachment #586715 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
status-seamonkey2.10:
--- → affected
Resolution: --- → FIXED
Target Milestone: seamonkey2.10 → seamonkey2.11
Assignee | ||
Comment 14•13 years ago
|
||
V.Fixed: catched bug 735567 right away ;->
Status: RESOLVED → VERIFIED
Comment 15•13 years ago
|
||
(In reply to Serge Gautherie (:sgautherie) from comment #14)
> V.Fixed: catched bug 735567 right away ;->
Serge, allow me to say (on the record) that if you (or anyone else with ability to review *any* code in SeaMonkey) notice packaging errors that turn us red, feel free to land the fix to package-manifest.in with rs+=me
My only requests are that a bug be on file for the change, and if there is a removed-files.in change is necessary it be attached to said bug (separately).
I don't require but would love at least a f? for the package-manifest.in change for a post-landing peek.
-- Good that in this case (catching the bustage) you landed a fix anyway.
Comment 16•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Marking aurora approval while I'm here :-) (I'll decide on beta a bit later)
Attachment #586715 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee | ||
Comment 17•13 years ago
|
||
(In reply to Justin Wood (:Callek) from comment #15)
> [...] rs+=me [and] at least a f? [...]
Right, that's exactly the expected process ;-)
(In reply to Justin Wood (:Callek) from comment #16)
> Marking aurora approval while I'm here :-) (I'll decide on beta a bit later)
Thanks, I concur to complete this step by step, letting the dust settle.
![]() |
||
Comment 18•13 years ago
|
||
> Depends on: 736864
http://hg.mozilla.org/mozilla-central/rev/da5e8f6a44bd
Back out a0fa92215a36 (bug 713132) for breaking --disable-updater as used in RPM nightlies
Assignee | ||
Comment 19•13 years ago
|
||
(In reply to Philip Chee from comment #18)
I'm not sure what your point was:
> > Depends on: 736864
--disable-updater is not used by SeaMonkey boxes, afaik.
> http://hg.mozilla.org/mozilla-central/rev/da5e8f6a44bd
> Back out a0fa92215a36 (bug 713132) for breaking --disable-updater as used in
> RPM nightlies
FF part has been fixed, and bug 713132 relanded.
Comment 20•13 years ago
|
||
Comment on attachment 586715 [details] [diff] [review]
(Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Let have this ride in from aurora... I don't think its worth the trouble on beta.
Attachment #586715 -
Flags: approval-comm-beta? → approval-comm-beta-
Assignee | ||
Updated•13 years ago
|
Comment 21•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #586715 -
Attachment description: (Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13] → (Av1) Make packager.pm missing file warnings fatal on SeaMonkey
[Checked in: Comment 13 & 21]
Assignee | ||
Comment 22•13 years ago
|
||
http://build.mozillamessaging.com/tinderboxpushlog/?tree=SeaMonkey-Aurora&rev=73a1f3d5bda7
seamonkey2.10: verified.
![]() |
||
Updated•13 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•