Rain [:rain1] (inactive) Reporter
	Description • 13 years ago

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.

	Serge Gautherie (:sgautherie) Assignee
	Comment 1 • 13 years ago

Let's prepare for when blockers are fixed.

	Serge Gautherie (:sgautherie) 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?

	Ian Neal
	Comment 3 • 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]

Sorry this is more KaiRo's territory than mine

	Justin Wood (:Callek)
	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.

	Serge Gautherie (:sgautherie) 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.

	Ian Neal
	Comment 6 • 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]

Probably more KaiRo's territory

	Serge Gautherie (:sgautherie) Assignee
	Comment 7 • 13 years ago

Ping for review.

	Robert Kaiser
	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.

	Jens Hatlak (:InvisibleSmiley)
	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.

	Serge Gautherie (:sgautherie) 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!)...

	Robert Kaiser
	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.

	Justin Wood (:Callek)
	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.

	Serge Gautherie (:sgautherie) 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

	Serge Gautherie (:sgautherie) Assignee
	Comment 14 • 13 years ago

V.Fixed: catched bug 735567 right away ;->

	Justin Wood (:Callek)
	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.

	Justin Wood (:Callek)
	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)

	Serge Gautherie (:sgautherie) 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.

	Philip Chee
	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

	Serge Gautherie (:sgautherie) 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.

	Justin Wood (:Callek)
	Comment 20 • 12 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.

	Justin Wood (:Callek)
	Comment 21 • 12 years ago

http://hg.mozilla.org/releases/comm-aurora/rev/6e46db9f7b18

	Serge Gautherie (:sgautherie) Assignee
	Comment 22 • 12 years ago

http://build.mozillamessaging.com/tinderboxpushlog/?tree=SeaMonkey-Aurora&rev=73a1f3d5bda7

seamonkey2.10: verified.