Closed Bug 736346 Opened 8 years ago Closed 8 years ago

Bump Venkman compatibility for Firefox 14.0 / SeaMonkey 2.11 / Thunderbird 14.0 / (Gecko) Toolkit 14.0

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect, P2, major)

Tracking

(seamonkey2.11 verified)

VERIFIED FIXED
mozilla14
Tracking Status
seamonkey2.11 --- verified

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #730691 +++

(perma-orange on SeaMonkey.)
(In reply to Serge Gautherie (:sgautherie) from comment #0)
> +++ This bug was initially created as a clone of Bug #730691 +++
> 
> (perma-orange on SeaMonkey.)

rs=me for updating this.

But, fwiw, this sounds like we're fighting symptoms, and leaving the real cause (trees going perma-orange because of add-on compat) unfixed. That just shouldn't happen; as it is the add-ons will run because of 'compatible by default', right?
http://hg.mozilla.org/venkman/rev/b6a784fb2a54


(In reply to Gijs Kruitbosch from comment #1)
> But, fwiw, this sounds like we're fighting symptoms, and leaving the real
> cause (trees going perma-orange because of add-on compat) unfixed. That just
> shouldn't happen; as it is the add-ons will run because of 'compatible by
> default', right?

Yes, but see bug 732749 comment 7 about the (current) situation.
Assignee: nobody → sgautherie.bz
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Summary: Bump Venkman compatibility for Firefox 14.0 / SeaMonkey 2.11 / Thunderbird 14.0 → Bump Venkman compatibility for Firefox 14.0 / SeaMonkey 2.11 / Thunderbird 14.0 / (Gecko) Toolkit 14.0
While here, one step toward more file synchronization.
Attachment #606586 - Flags: review?(gijskruitbosch+bugs)
(In reply to Serge Gautherie (:sgautherie) from comment #3)
> Created attachment 606586 [details] [diff] [review]
> Replace tabs with spaces
> 
> While here, one step toward more file synchronization.

I'm confused, sorry. Synchronization with what? Why does the spacing matter?
(In reply to Gijs Kruitbosch from comment #4)
> I'm confused, sorry. Synchronization with what? Why does the spacing matter?

Sync' one file with the other. Just a nit, nothing special.
Helps compare/edit/... them, just being more consistent, fwiw.
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1331928910.1331934391.22023.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/03/16 13:15:10

V.Fixed
Status: RESOLVED → VERIFIED
Blocks: 732749
Comment on attachment 606586 [details] [diff] [review]
Replace tabs with spaces
[Checked in: Comment 8]

(In reply to Serge Gautherie (:sgautherie) from comment #5)
> (In reply to Gijs Kruitbosch from comment #4)
> > I'm confused, sorry. Synchronization with what? Why does the spacing matter?
> 
> Sync' one file with the other. Just a nit, nothing special.
> Helps compare/edit/... them, just being more consistent, fwiw.

Next time, please just do it as part of the fix you've just done, and attach a diff -w for that.

rs=me
Attachment #606586 - Flags: review?(gijskruitbosch+bugs) → review+
Comment on attachment 606586 [details] [diff] [review]
Replace tabs with spaces
[Checked in: Comment 8]

https://hg.mozilla.org/venkman/rev/036e6bf7194d


(In reply to Gijs Kruitbosch from comment #7)
> Next time, please just do it as part of the fix you've just done, and attach
> a diff -w for that.

I'll try to remember.
Attachment #606586 - Attachment description: Replace tabs with spaces → Replace tabs with spaces [Checked in: Comment 8]
Blocks: 738564
Blocks: 748625
No longer blocks: 738564
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.