28736 - Include bug summary in "dependency changed" mail

Status: VERIFIED FIXED

(Bugzilla :: Email Notifications, enhancement, P1)

Description

[Daniel Veditz [:dveditz]]

I love the new mail format. It would help, though, for "dependency changed" 
mail to include the summary of the dependency bug.  Right now I know Bug A 
"blah blah blah" (in subject) had its dependent bug XXXXX change state, but no 
idea what bug XXXXX is.

While you're at it, a link to bug XXXXX in addition to the link to the 
unchanged bug A would be helpful.

Comment 1

[Matthew Tuck [:CodeMachine]]

*** Bug 30352 has been marked as a duplicate of this bug. ***

Comment 2

[Terry Weissman]

[e-mail] is the new owner of Bugzilla and Bonsai.  (For details,
see my posting in netscape.public.mozilla.webtools,
news://news.mozilla.org/38F5D90D.F40E8C1A%40geocast.com .)

Comment 3

[Matthew Tuck [:CodeMachine]]

*** Bug 37985 has been marked as a duplicate of this bug. ***

Comment 4

[Jesse Ruderman]

See also bug 61015, which would make CC/dependency changes more readable by 
showing each address/bug on a separate line instead of using wrapped, comma-
separated lists.  There's some debate as to whether all CCs/dependencies should 
be listed or just the ones that were changed.

Comment 5

[Peter Lairo]

Checking dependencies has become very difficult on long lists (meta bugs) ;)

Comment 6

[Matthew Tuck [:CodeMachine]]

This seriously annoys me.  I would like to delete most dependency changed mails
quickly, and I can't.

Comment 7

[Matthew Tuck [:CodeMachine]]

Attachment: This should do the trick.

Comment 8

[Dave Miller [:justdave]]

oooh, me wants :)

Comment 9

[Matthew Tuck [:CodeMachine]]

I think the first part of the patch above isn't meant to be - not sure how that
happened ...

Comment 10

[Dave Miller [:justdave]]

*** Bug 11820 has been marked as a duplicate of this bug. ***

Comment 11

[Matthew Tuck [:CodeMachine]]

Dunno what I was saying, both parts look needed. =)

Comment 12

[Jacob Steenhagen]

The patch conflicted (minorly) with the fix from bug 55161.  After some minor
massaging, I got it to apply.  Sample message:

==========
http://landfill.tequilarista.org/214shakedown/show_bug.cgi?id=12

This bug depends on bug 9, which changed state.
Bug 9 Summary: [bleep] censors won't let me [bleep] install new government

           What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
==========

Looks good to me.
r=jake
[will upload massaged version in a minute]

Comment 13

[Jacob Steenhagen]

Attachment: work around cvs conflict (changes in same area of code)

Comment 14

[Jacob Steenhagen]

Checked in.

Comment 15

[Daniel Veditz [:dveditz]]

Thank you, thank you!

Asa: any chance [e-mail] can pick this up in the near future?

Comment 16

[Matthew Tuck [:CodeMachine]]

This was checked in as a part of 2.14+, which is the informal name for
mozilla.org's next update within the next day or two.

Comment 17

[Jacob Steenhagen]

V. Fixed

Comment 18

[Mike Potter]

*** Bug 146476 has been marked as a duplicate of this bug. ***