GNOME Bugzilla – Bug 346797
Help -> Submit Bug Report menu item does not work
Last modified: 2013-09-13 00:49:02 UTC
Selecting "Submit Bug Report" from the Help menu brings up an error dialog saying "Both the --pid and --appname arguments are required." Looks like Evolution just needs to update the command-line arguments used to invoke bug-buddy. The offending code is in the command_submit_bug() function in shell/e-shell-window-commands.c.
hmm... are you running the bug-buddy version from cvs HEAD?
bug-buddy 2.15.0, and evolution 2.7.3
Created attachment 68570 [details] [review] Proposed patch This patch supplies the missing bug-buddy arguments and also "GLib-ifies" the command_submit_bug() function a little more.
targetting to 2.7; asking harish for patch review, so people can you use bugbuddy again.
IIRC bug-buddy 2.15.0 can currently only submit crashers. Adding a pid argument makes no sense for regular bug reports (will be treated probably as a crasher). Suggest to wait on the outcome of bug 346901.
(In reply to comment #5) I can confirm that it _is_ treated as a crasher in Bug Buddy. Perhaps instead of opening Bug Buddy, the "Submit Bug Report" menu item should open a web browser to the configuration string PACKAGE_BUGREPORT (http://bugzilla.gnome.org/enter_bug.cgi?product=Evolution)?
bkor is right, bug-buddy in HEAD no longer supports an interface to file non-crasher bugs.
I fixed bug-buddy so with the --packege command line argument it is in non-crash mode and works fine with current evolution.
Please still consider the patch in comment #3. It cleans up the code a little bit and uses g_spawn_command_line_async() instead of gnome_execute_async(). Some of the options (e.g. --pid) can be removed for use with bug-buddy 2.15.90.
harish, can we please get this patch reviewed and in asap? i'm interested in getting evolution bug reports by bugbuddy. thanks. ;-)
matthew, can you give a updated patch with the those options removed. Otherwise it looks fine IMO.
Created attachment 70410 [details] [review] Proposed patch #2 Here's a new, simplified patch for use with bug-buddy 2.15.90. However, there seems to be some issues with bug-buddy itself. It does not seem to recognize "evolution" as a valid package. If I select Help -> Submit Bug Report with this patch applied, I get an error dialog saying: Bug Buddy doesn't know how to send a suggestion for the application evolution. But it recognizes "Evolution" (uppercase 'E'). Unfortunately PACKAGE and PACKAGE_NAME are both defined as "evolution" (lowercase 'E'), so I had to hard-code the package name in the bug-buddy invocation command. I also observed that bug-buddy crashes with a "corrupted double-linked list" from glibc if it's invoked without the --sm-disable option. But that's a separate bug for bug-buddy. I just mention it FYI.
Allow me to clarify the second paragraph because I changed the patch half-way through writing the comment and didn't proofread very well. Where I say, "... with this patch applied, I get an error dialog ...", I was speaking from before I changed the patch to pass --package=Evolution instead of --package=evolution to bug-buddy. The attached patch is the newer version (--package=Evolution) and seems to work correctly, although I would prefer it if bug-buddy recognized "evolution" with a lowercase 'E' so that we could use the PACKAGE constant.
fixed to HEAD.