Closed
Bug 736346
Opened 13 years ago
Closed 13 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)
Other Applications Graveyard
Venkman JS Debugger
Tracking
(seamonkey2.11 verified)
VERIFIED
FIXED
mozilla14
Tracking | Status | |
---|---|---|
seamonkey2.11 | --- | verified |
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #730691 +++
(perma-orange on SeaMonkey.)
Comment 1•13 years ago
|
||
(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?
Assignee | ||
Comment 2•13 years ago
|
||
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: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
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
Assignee | ||
Comment 3•13 years ago
|
||
While here, one step toward more file synchronization.
Attachment #606586 -
Flags: review?(gijskruitbosch+bugs)
Comment 4•13 years ago
|
||
(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?
Assignee | ||
Comment 5•13 years ago
|
||
(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.
Assignee | ||
Comment 6•13 years ago
|
||
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
Assignee | ||
Updated•13 years ago
|
status-seamonkey2.11:
--- → verified
Comment 7•13 years ago
|
||
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+
Assignee | ||
Comment 8•13 years ago
|
||
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]
Updated•6 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•