https://bugzilla.novell.com/show_bug.cgi?id=418333 Cyril Hrubis changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |adrian@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=418333 User skh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=418333#c2 --- Comment #2 from Sonja Krause-Harder 2008-09-16 05:21:13 MDT --- Is including the diff expensive in any way? Not including it seems to me like artificially withholding information which are available at the time the mail is sent anyway. The world will not end if I have to switch to another shell, find the project, find the package and call osc with the right incantation. It would just be much easier to have the diff right in the notification email. Your proposal on -buildservice is of course a major improvement to the current state in either case. :) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=418333 User froh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=418333#c4 --- Comment #4 from Susanne Oberhauser 2008-09-16 09:05:00 MDT --- (In reply to comment #2 from Sonja Krause-Harder)

Is including the diff expensive in any way? Not including it seems to me like artificially withholding information which are available at the time the mail is sent anyway.

alas, as abauer said, it's not readily 'available', and we're somewhat reluctant making one system depend on the other. you'd need to

The world will not end if I have to switch to another shell, find the project, find the package and call osc with the right incantation. It would just be much easier to have the diff right in the notification email.

I see. As you need to switch to a shell anyway to accept or decline the request, you can also just "osc submitreq show --diff $ID" it, and the mail shall contain the right command line, including $ID, in the message body, for cut&paste. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=418333 User froh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=418333#c6 Susanne Oberhauser changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #6 from Susanne Oberhauser 2008-10-15 05:11:47 MDT --- Is the 'osc submitreq show -d id#' sufficient that's in the notifications now? The diff can be very long and not helpfull. It should be configurable, if implemented at all. I dare to close this as fixed for now. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.