Closed
Bug 715719
Opened 13 years ago
Closed 13 years ago
Venkman not registered in thunderbird tools menu
Categories
(Other Applications Graveyard :: Venkman JS Debugger, defect)
Other Applications Graveyard
Venkman JS Debugger
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla12
People
(Reporter: sfleiter, Assigned: sfleiter)
References
Details
Attachments
(1 file, 1 obsolete file)
In Thunderbird venkman from trunk does not get registered in the tools menu.
According to my investigations that is caused by the fact that content/venkman/venkman-overlay-tb.xul but not shipped in the jar file.
See attached patch.
Assignee | ||
Comment 1•13 years ago
|
||
Comment on attachment 586269 [details] [diff] [review]
Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu
I have no experience in the build system and how extensions are built but the attached patch fixes the problem for me and seems logical.
Bug found when trying to test fix of bug 418426.
Attachment #586269 -
Flags: review?(gijskruitbosch+bugs)
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → stefan.fleiter
Status: NEW → ASSIGNED
Comment 2•13 years ago
|
||
Comment on attachment 586269 [details] [diff] [review]
Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu
rs=me
Can you do a full mercurial export (hg commit with an appropriate message after setting your mercurial name + email)? That'll help me commit this with proper credits to you as the patch author. Thanks! :-)
(more info: https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F )
Attachment #586269 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 3•13 years ago
|
||
I hope this patch has the correct format.
Thanks a lot for your patience and the pointer to the mq documentation.
Attachment #586269 -
Attachment is obsolete: true
Attachment #586415 -
Flags: review?(gijskruitbosch+bugs)
Comment 4•13 years ago
|
||
Comment on attachment 586415 [details] [diff] [review]
Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu, mq patch [Checkin: Comment 6]
r=me
Yay for the bugzilla review reminder emails, and sorry this slipped a few weeks, it got lost...
In the future, note also that in cases like this where you post the exact same patch but with patch formatting details different (or maybe even if you exactly copied instructions regarding code formatting or other trivial changes) it is considered fine to 'carry over' review, that is, you wouldn't have needed to re-request review.
Thanks again for the patch, I'll try to check it in straight away. :-)
Attachment #586415 -
Flags: review?(gijskruitbosch+bugs) → review+
Comment 5•13 years ago
|
||
Scratch that, my HG account has been disabled due to inactivity. Re-requesting access, setting checkin-needed in the meantime, we'll see what happens first...
Keywords: checkin-needed
Comment 6•13 years ago
|
||
Comment on attachment 586415 [details] [diff] [review]
Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu, mq patch [Checkin: Comment 6]
http://hg.mozilla.org/venkman/rev/d8acf291d178
Attachment #586415 -
Attachment description: Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu, mq patch → Include venkman-overlay-tb.xul in jar to enable registering venkman in Thunderbird tools menu, mq patch [Checkin: Comment 6]
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
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
•