Windows 7 Debug failing all tests | Exception: Sorry, cannot connect to jsbridge extension, port 24242

RESOLVED FIXED in Thunderbird 38.0

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rkent, Assigned: hiro)

Tracking

({intermittent-failure})

Trunk
Thunderbird 38.0
x86_64
Windows 8.1
intermittent-failure

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
jcranmer suspects this is associated with config change:

12:03:46 PM - jcranmer|away: oh, I think I know why
12:03:54 PM - jcranmer|away: and it's because autoconf is a $!@#%^&ing pain in the butt
12:04:22 PM - jcranmer|away: http://dxr.mozilla.org/comm-central/source/mail/configure.in#95
12:04:33 PM - jcranmer|away: the variable that's checking isn't defined until after that code is invoked
12:04:39 PM - jcranmer|away: so it's not properly triggering
Comment hidden (Treeherder Robot)
Said variable is declared here: <http://dxr.mozilla.org/comm-central/source/mozilla/configure.in#7257>

mail/configure.in is declared here: <http://dxr.mozilla.org/comm-central/source/mozilla/configure.in#4106>

Hence we're attempting to use a variable prior to its definition.
Keywords: intermittent-failure
Summary: Windows 7 Debug failing all tests → Windows 7 Debug failing all tests | Exception: Sorry, cannot connect to jsbridge extension, port 24242
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 50

4 years ago
FYI:
https://treeherder.mozilla.org/ui/#/jobs?repo=try-comm-central&revision=40af2c0566fd

Unfortunately adding --disable-debug-rtl option for ldap c-sdk did not solve this failure.
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 55

4 years ago
Created attachment 8544326 [details] [diff] [review]
Difference of xpcshell logs between opt and debug build

Just FYI.

I can't see any significant difference except debug symbol handling between them.
(Assignee)

Comment 56

4 years ago
Created attachment 8544341 [details] [diff] [review]
Fix

USE_DEBUG_RTL was used even if --disable-debug-rtl option is set.

This patch is equivalent to nspr change. 
https://hg.mozilla.org/mozilla-central/rev/8bc3daaf84cc
Assignee: nobody → hiikezoe
Attachment #8544326 - Attachment is obsolete: true
Attachment #8544341 - Flags: review?(Pidgeot18)
(Assignee)

Comment 57

4 years ago
(In reply to Hiroyuki Ikezoe (:hiro) from comment #56)
> Created attachment 8544341 [details] [diff] [review]
> Fix
> 
> USE_DEBUG_RTL was used even if --disable-debug-rtl option is set.

Oops! This patch is not a fix for this issue, I think. This patch does just fix the case of setting --disable-debug-rtl.
(Assignee)

Comment 59

4 years ago
Created attachment 8544485 [details] [diff] [review]
Moving application configure after exporting MOZ_NO_DEBUG_RTL

A try on comm-central:

https://treeherder.mozilla.org/ui/#/jobs?repo=try-comm-central&revision=4580ff11b093

and try on mozilla-central:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=175185d3eb70


This patch solved the failure on try server.
I am not sure this patch has no side-effect against seamonkey.
Attachment #8544485 - Flags: feedback?(philip.chee)
(Assignee)

Updated

4 years ago
Attachment #8544485 - Flags: feedback?(Pidgeot18)

Comment 60

4 years ago
Comment on attachment 8544485 [details] [diff] [review]
Moving application configure after exporting MOZ_NO_DEBUG_RTL

(In reply to Hiroyuki Ikezoe (:hiro) from comment #59)
> Created attachment 8544485 [details] [diff] [review]
> Moving application configure after exporting MOZ_NO_DEBUG_RTL
> 
> A try on comm-central:
> 
> https://treeherder.mozilla.org/ui/#/jobs?repo=try-comm-
> central&revision=4580ff11b093
> 
> and try on mozilla-central:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=175185d3eb70
> 
> This patch solved the failure on try server.
> I am not sure this patch has no side-effect against seamonkey.
I am sorry but build config is outside my area of competence.
I'll set the feedback? flag to :callek
Attachment #8544485 - Flags: feedback?(philip.chee) → feedback?(bugspam.Callek)
Comment on attachment 8544485 [details] [diff] [review]
Moving application configure after exporting MOZ_NO_DEBUG_RTL

Review of attachment 8544485 [details] [diff] [review]:
-----------------------------------------------------------------

I can't find anything that says this would hurt seamonkey at a glance. I can't test anytime soon though easily.

That said, http://mxr.mozilla.org/comm-central/source/mail/configure.in#96 will probably need to be ported for SeaMonkey (suite/configure.in) as well.
Attachment #8544485 - Flags: feedback?(bugspam.Callek) → feedback+
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Updated

4 years ago
Attachment #8544485 - Flags: feedback?(Pidgeot18) → review?(mh+mozilla)
Comment on attachment 8544485 [details] [diff] [review]
Moving application configure after exporting MOZ_NO_DEBUG_RTL

Review of attachment 8544485 [details] [diff] [review]:
-----------------------------------------------------------------

I'd rather move export MOZ_NO_DEBUG_RTL=1.
Attachment #8544485 - Flags: review?(mh+mozilla) → review-
(In reply to Mike Hommey [:glandium] from comment #67)
> Comment on attachment 8544485 [details] [diff] [review]
> Moving application configure after exporting MOZ_NO_DEBUG_RTL
> 
> Review of attachment 8544485 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I'd rather move export MOZ_NO_DEBUG_RTL=1.

That has no unknown side effects, while moving app subconfigure does.
(Assignee)

Comment 69

4 years ago
(In reply to Mike Hommey [:glandium] from comment #67)
> Comment on attachment 8544485 [details] [diff] [review]
> Moving application configure after exporting MOZ_NO_DEBUG_RTL
> 
> Review of attachment 8544485 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I'd rather move export MOZ_NO_DEBUG_RTL=1.

You mean also moving jemalloc check and something related codes?
ah, that's in a MOZ_MEMORY block... sigh, what a mess.
Comment on attachment 8544485 [details] [diff] [review]
Moving application configure after exporting MOZ_NO_DEBUG_RTL

Review of attachment 8544485 [details] [diff] [review]:
-----------------------------------------------------------------

Let's go with this, then, hoping app configures are not relying on anything above their new point of entry happening after them.
Attachment #8544485 - Flags: review- → review+
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment on attachment 8544341 [details] [diff] [review]
Fix

Review of attachment 8544341 [details] [diff] [review]:
-----------------------------------------------------------------

Is this needed, given https://bug1113863.bugzilla.mozilla.org/attachment.cgi?id=8544485 ?
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)