Port |Bug 739132 - --disable-necko-wifi causes "Error: package error or possible missing or unnecessary file: bin/components/necko_wifi.xpt"| to SeaMonkey

RESOLVED FIXED in seamonkey2.13

Status

SeaMonkey
Build Config
P3
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

Trunk
seamonkey2.13
Dependency tree / graph
Bug Flags:
in-testsuite -

SeaMonkey Tracking Flags

(seamonkey2.8 wontfix, seamonkey2.9 wontfix, seamonkey2.10 wontfix, seamonkey2.11 fixed, seamonkey2.12 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
SM 2.10 and 2.9:
I won't bother fixing that, unless someone actually complains about this bug.
Flags: in-testsuite-
(Assignee)

Comment 1

6 years ago
Created attachment 615522 [details] [diff] [review]
(Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging
[Checked in: See comment 2+5+6 & 7 & 8+9]
Attachment #615522 - Flags: review?(bugspam.Callek)

Updated

6 years ago
Attachment #615522 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 2

6 years ago
Comment on attachment 615522 [details] [diff] [review]
(Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging
[Checked in: See comment 2+5+6 & 7 & 8+9]

http://hg.mozilla.org/comm-central/rev/769ffdc41a52


[Approval Request Comment]
No risk.
Attachment #615522 - Attachment description: (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging → (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: Comment 2]
Attachment #615522 - Flags: approval-comm-aurora?
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
status-seamonkey2.11: --- → affected
status-seamonkey2.9: affected → wontfix
Resolution: --- → FIXED
Target Milestone: seamonkey2.11 → seamonkey2.12
Backed this out due to complete all-platform build bustage, please remember to watch all trees you can affect especially for the short time after pushing for changes like this.

https://hg.mozilla.org/comm-central/rev/28dc4ceee019

checking for sys/int_types.h... (cached) no
/builds/slave/tb-comm-cen-lnx/build/configure: line 17818: syntax error near unexpected token `iwlib.h'
/builds/slave/tb-comm-cen-lnx/build/configure: line 17818: `  MOZ_CHECK_HEADER(iwlib.h)'
*** Fix above errors and then restart with               "make -f client.mk build"
make[2]: *** [configure] Error 1
make[2]: Leaving directory `/builds/slave/tb-comm-cen-lnx/build'
make[1]: *** [objdir-tb/config.status] Error 2
make[1]: Leaving directory `/builds/slave/tb-comm-cen-lnx/build'
make: *** [build] Error 2
program finished with exit code 2
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 4

6 years ago
MOZ_CHECK_HEADER(iwlib.h)'

Removing/commenting out this line makes build successful.
(Assignee)

Comment 5

6 years ago
Comment on attachment 615522 [details] [diff] [review]
(Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging
[Checked in: See comment 2+5+6 & 7 & 8+9]

http://hg.mozilla.org/comm-central/rev/207aa287be44
(Bv1) s/MOZ_CHECK_HEADER/AC_CHECK_HEADER/.
+
http://hg.mozilla.org/comm-central/rev/6dcf542bcc36
Merge of bug 745998.

Succeeded as
https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=11204cc7189b


(In reply to Mark Banner (:standard8) from comment #3)
> Backed this out due to complete all-platform build bustage, please remember
> to watch all trees you can affect especially for the short time after
> pushing for changes like this.

Sorry about that (and forgetting to Try it in the first place). I usually do.
Attachment #615522 - Attachment description: (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: Comment 2] → (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: See comment 2+5]
(Assignee)

Updated

6 years ago
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Attachment #615522 - Flags: approval-comm-aurora? → approval-comm-beta?
(Assignee)

Updated

6 years ago
status-seamonkey2.10: affected → wontfix
(Assignee)

Comment 6

6 years ago
Comment on attachment 615522 [details] [diff] [review]
(Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging
[Checked in: See comment 2+5+6 & 7 & 8+9]

http://hg.mozilla.org/comm-central/rev/530a1afbf7f4
(Cv1) Unbackout the 2 other files too.
Attachment #615522 - Attachment description: (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: See comment 2+5] → (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: See comment 2+5+6]
(Assignee)

Updated

6 years ago
status-seamonkey2.12: --- → affected
Keywords: checkin-needed
Whiteboard: [c-n: 530a1afbf7f4 to c-a]
Target Milestone: seamonkey2.12 → seamonkey2.13
(Assignee)

Updated

6 years ago
Depends on: 769655
https://hg.mozilla.org/releases/comm-aurora/rev/f74e6f31fc6e
status-seamonkey2.12: affected → fixed
Keywords: checkin-needed
Whiteboard: [c-n: 530a1afbf7f4 to c-a]

Updated

5 years ago
Attachment #615522 - Flags: approval-comm-beta? → approval-comm-beta+
http://hg.mozilla.org/releases/comm-beta/rev/13a71e741a05
status-seamonkey2.11: affected → fixed
I just had to push a bustage fix for this:

https://hg.mozilla.org/releases/comm-beta/rev/c26790d2d108

This was the transplant of the previous bustage fix.

When requesting approval, please ensure your patch is the combined non-busted version and also be explicit about what changesets peoples should transplant if they are transplanting.
(Assignee)

Comment 10

5 years ago
Comment on attachment 615522 [details] [diff] [review]
(Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging
[Checked in: See comment 2+5+6 & 7 & 8+9]

(In reply to Mark Banner (:standard8) from comment #9)
> be explicit about what changesets peoples should transplant if they are transplanting.

I think I am! In this case, I had not (yet) requested c-n to beta...
Attachment #615522 - Attachment description: (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: See comment 2+5+6] → (Av1) Add configure support for NECKO_WIFI, Use it for SeaMonkey packaging [Checked in: See comment 2+5+6 & 7 & 8+9]
(In reply to Serge Gautherie (:sgautherie) from comment #10)
> I think I am! In this case, I had not (yet) requested c-n to beta...

The main issue was the patch you requested approval on was not correct for beta. Neither the aurora or beta approval requests mentioned about the fact the patch wasn't correct.
(Assignee)

Comment 12

5 years ago
(In reply to Mark Banner (:standard8) from comment #11)
> The main issue was the patch you requested approval on was not correct for
> beta.

My point of view is that the "main" issue is that patches I want to land are stuck on approval requests while patches I haven't asked to land get pushed (out of order) :-<
I understand what you write but I can't accept blame for what other people (= Callek in this case) do on their own.
You need to log in before you can comment on or make changes to this bug.