Closed Bug 748610 Opened 8 years ago Closed 8 years ago

[SeaMonkey, 2.10+] "Error: package error or possible missing or unnecessary file: bin/distribution/extensions/debugQA@mozilla.org.xpi"

Categories

(SeaMonkey :: Build Config, defect, P1, blocker)

defect

Tracking

(seamonkey2.9 wontfix, seamonkey2.10 verified, seamonkey2.11 verified)

VERIFIED FIXED
seamonkey2.12
Tracking Status
seamonkey2.9 --- wontfix
seamonkey2.10 --- verified
seamonkey2.11 --- verified

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(1 file, 1 obsolete file)

SeaMonkey Beta-2.10:
{
Error: package error or possible missing or unnecessary file: bin/distribution/extensions/debugQA@mozilla.org.xpi (package-manifest, 546).
}
Flags: in-testsuite-
(Untested.)

[Approval Request Comment]
"No" risk, buildtime only.
Attachment #618112 - Flags: review?(bugspam.Callek)
Attachment #618112 - Flags: approval-comm-beta?
Attachment #618112 - Flags: approval-comm-aurora?
Comment on attachment 618112 [details] [diff] [review]
(Av1) Package SeaMonkey debugQA extension on Alpha builds only

>diff --git a/suite/installer/Makefile.in b/suite/installer/Makefile.in
>+ifdef IS_ALPHA
>+DEFINES += -DMOZ_IS_ALPHA=1
>+endif

I don't feel comfortable pushing this logic up to configure, and would rather just use this define with

ifneq (,$(findstring a,$(MOZ_APP_VERSION:pre=a)))
Attachment #618112 - Flags: review?(bugspam.Callek)
Attachment #618112 - Flags: review-
Attachment #618112 - Flags: approval-comm-beta?
Attachment #618112 - Flags: approval-comm-aurora?
Ftr, the extension should not be removed(-files.in), per

https://wiki.mozilla.org/SeaMonkey:Debug_And_QA_UI
"For release builds you can install extension from AMO."

*****

(In reply to Justin Wood (:Callek) from comment #2)

> I don't feel comfortable pushing this logic up to configure,

Why? It would seem logical to reuse a unique definition.

> and would rather just use this define with
> 
> ifneq (,$(findstring a,$(MOZ_APP_VERSION:pre=a)))

In that case, shouldn't we drop 'pre' handling, which is obsolete (isn't it)?
Handling MOZ_APP_VERSION in two different ways seems odd (currently).
Attachment #618112 - Flags: feedback?(bugspam.Callek)
(In reply to Serge Gautherie (:sgautherie) from comment #3)
> Ftr, the extension should not be removed(-files.in), per
> 
> https://wiki.mozilla.org/SeaMonkey:Debug_And_QA_UI
> "For release builds you can install extension from AMO."
> 
> *****
> 
> (In reply to Justin Wood (:Callek) from comment #2)
> 
> > I don't feel comfortable pushing this logic up to configure,
> 
> Why? It would seem logical to reuse a unique definition.

To me, custom logic based on version for SeaMonkey only belongs under suite/ at the least. And because of that I don't think we want/need a configure piece of magic here just for this.

> > and would rather just use this define with
> > 
> > ifneq (,$(findstring a,$(MOZ_APP_VERSION:pre=a)))
> 
> In that case, shouldn't we drop 'pre' handling, which is obsolete (isn't it)?
> Handling MOZ_APP_VERSION in two different ways seems odd (currently).

IMO, using pre *might* still have legit reasons in the future... so I'd rather keep that working since it doesn't hurt anything
Av1, with comment 4 suggestion(s).

I am not fully convinced, but that doesn't matter much ftb.

[Approval Request Comment]
Blocker bustage fix.
Attachment #618112 - Attachment is obsolete: true
Attachment #618112 - Flags: feedback?(bugspam.Callek)
Attachment #619954 - Flags: review?(bugspam.Callek)
Attachment #619954 - Flags: approval-mozilla-beta?
Attachment #619954 - Flags: approval-mozilla-aurora?
Attachment #619954 - Flags: approval-mozilla-beta?
Attachment #619954 - Flags: approval-mozilla-aurora?
Attachment #619954 - Flags: approval-comm-beta?
Attachment #619954 - Flags: approval-comm-aurora?
Attachment #619954 - Flags: review?(bugspam.Callek)
Attachment #619954 - Flags: review+
Attachment #619954 - Flags: approval-comm-beta?
Attachment #619954 - Flags: approval-comm-beta+
Attachment #619954 - Flags: approval-comm-aurora?
Attachment #619954 - Flags: approval-comm-aurora+
Comment on attachment 619954 [details] [diff] [review]
(Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only
[Checked in: Comments 6 and 8]

http://hg.mozilla.org/comm-central/rev/4df6a3ad8bc6
Attachment #619954 - Attachment description: (Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only → (Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only [Checked in: Comment 6]
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1335917741.1335928891.23436.gz&fulltext=1
WINNT 5.2 comm-central-trunk build on 2012/05/01 17:15:41
{
-DMOZ_PACKAGE_DEBUGQA=1

[debugqa]
bin/distribution/extensions/debugQA@mozilla.org.xpi
}
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: 4df6a3ad8bc6 to c-a and c-b]
Comment on attachment 619954 [details] [diff] [review]
(Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only
[Checked in: Comments 6 and 8]

http://hg.mozilla.org/releases/comm-aurora/rev/2c595fb63995
http://hg.mozilla.org/releases/comm-beta/rev/6c022d9e7c65
Attachment #619954 - Attachment description: (Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only [Checked in: Comment 6] → (Av2) Package SeaMonkey debugQA extension on Alpha (or Nightly) builds only [Checked in: Comments 6 and 8]
Keywords: checkin-needed
Whiteboard: [c-n: 4df6a3ad8bc6 to c-a and c-b]
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Beta/1336009132.1336013775.25771.gz&fulltext=1
WINNT 5.2 comm-beta build on 2012/05/02 18:38:52
{
[debugqa]
/bin/sh {...}
}

V.Fixed.
seamonkey2.11 and seamonkey2.10: verified.
Status: RESOLVED → VERIFIED
Summary: [SeaMonkey, Beta+] "Error: package error or possible missing or unnecessary file: bin/distribution/extensions/debugQA@mozilla.org.xpi" → [SeaMonkey, 2.10+] "Error: package error or possible missing or unnecessary file: bin/distribution/extensions/debugQA@mozilla.org.xpi"
You need to log in before you can comment on or make changes to this bug.