Closed
Bug 732745
(Venkman_0.9.89)
Opened 13 years ago
Closed 13 years ago
Release Venkman 0.9.89
Categories
(Other Applications Graveyard :: Venkman JS Debugger, defect)
Other Applications Graveyard
Venkman JS Debugger
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
0.9.88.2 was (not) tagged 11 months ago.
http://hg.mozilla.org/venkman/rev/5e7659a617fa
A lot has happened since then.
SeaMonkey builds off of
http://hg.mozilla.org/venkman/rev/a38583d7164a
and should switch to
http://hg.mozilla.org/venkman/rev/65ad515ebba6
Time to actually tag 0.9.88.3?
Assignee | ||
Comment 1•13 years ago
|
||
(In reply to Serge Gautherie (:sgautherie) from comment #0)
> Time to actually tag 0.9.88.3?
Or 0.9.89...
Assignee | ||
Updated•13 years ago
|
Alias: Venkman_0.9.88.3
Assignee | ||
Comment 2•13 years ago
|
||
I think
http://hg.mozilla.org/venkman/rev/148fe7bca62e
"Bug 418426 - JavaScript Debugger (Venkman) ignores breakpoints"
is worth a 0.9.89 release.
Last locale addition was
http://hg.mozilla.org/venkman/rev/289e582190e8
"Bug 576869 - Fix Venkman for the new component registration world"
which was released in 0.9.88.
(Ftr, there were 2-3 minor changes after that, but I assume they don't need to be required.)
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #602863 -
Flags: review?(gijskruitbosch+bugs)
Comment 3•13 years ago
|
||
Comment on attachment 602863 [details] [diff] [review]
(Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88
[Checked in: Comment 4]
r=me
I'll try to get a release uploaded to AMO when I've left work.
Attachment #602863 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 602863 [details] [diff] [review]
(Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88
[Checked in: Comment 4]
https://hg.mozilla.org/venkman/rev/d6a96f4eb38a
(In reply to Gijs Kruitbosch from comment #3)
> I'll try to get a release uploaded to AMO when I've left work.
I let you resolve this bug when you're done.
Attachment #602863 -
Attachment description: (Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88 → (Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88
[Checked in: Comment 4]
Assignee | ||
Comment 5•13 years ago
|
||
(In reply to Serge Gautherie (:sgautherie) from comment #4)
> I let you resolve this bug when you're done.
Please, tag(/branch) the release revision, so it's easier to pull from it.
Comment 6•13 years ago
|
||
So did I miss anything or is the summary actually wrong now? [Won't change it myself since I'm genuinely confused.]
Comment 7•13 years ago
|
||
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #6)
> So did I miss anything or is the summary actually wrong now? [Won't change
> it myself since I'm genuinely confused.]
Yes.
Alias: Venkman_0.9.88.3 → Venkman_0.9.89
Summary: Release Venkman 0.9.88.3 → Release Venkman 0.9.89
Comment 8•13 years ago
|
||
Submitted to AMO, tagged.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•13 years ago
|
||
Ftr,
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1332317963.1332323931.19075.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/03/21 01:19:23
{
TEST-PASS | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_checkAddonCompatibility.js | extension JavaScript Debugger 0.9.89 should be compatible
}
:-)
Comment 10•13 years ago
|
||
Comment on attachment 602863 [details] [diff] [review]
(Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88
[Checked in: Comment 4]
>+++ b/locales/en-US/chrome/venkman.properties
>+msg.locale.version = 0.9.88
>+++ b/resources/content/venkman-static.js
>+const __vnk_requiredLocale = "0.9.88";
>+++ b/version.txt
>+0.9.89
Shouldn't these all match?
Comment 11•13 years ago
|
||
(In reply to Ian Neal from comment #10)
> Comment on attachment 602863 [details] [diff] [review]
> (Av1) Release Venkman 0.9.89, Update (missed) locale to 0.9.88
> [Checked in: Comment 4]
>
> >+++ b/locales/en-US/chrome/venkman.properties
> >+msg.locale.version = 0.9.88
>
> >+++ b/resources/content/venkman-static.js
> >+const __vnk_requiredLocale = "0.9.88";
>
> >+++ b/version.txt
> >+0.9.89
>
> Shouldn't these all match?
No. version.txt has the XPI version. The other ones are the version of the locale required. That is only increased if there are string changes; this way localizers don't have to release dummy updates with no changes.
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
•