[meld] Use a MsgArea to report patch errors while trying to get a VC diff
- From: Kai Willadsen <kaiw src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [meld] Use a MsgArea to report patch errors while trying to get a VC diff
- Date: Sun, 28 Oct 2012 20:55:00 +0000 (UTC)
commit 0babfe25915e9b572c0522ed05dc296e43da6913
Author: Kai Willadsen <kai willadsen gmail com>
Date: Mon Dec 12 07:21:13 2011 +1000
Use a MsgArea to report patch errors while trying to get a VC diff
The old patch error dialog tended to result in mailing list spam with
insufficient details; this way at least the spam will go to bugzilla.
Also, this patch separates the cases of not having patch installed from
the case of the reverse patch not applying.
data/ui/vcview.ui | 8 ++++++
meld/vcview.py | 64 +++++++++++++++++++++++++++++------------------------
2 files changed, 43 insertions(+), 29 deletions(-)
---
diff --git a/data/ui/vcview.ui b/data/ui/vcview.ui
index acf00c3..48b6853 100644
--- a/data/ui/vcview.ui
+++ b/data/ui/vcview.ui
@@ -48,6 +48,14 @@
</packing>
</child>
<child>
+ <object class="MsgAreaController" id="msgarea_mgr">
+ <property name="visible">True</property>
+ </object>
+ <packing>
+ <property name="expand">False</property>
+ </packing>
+ </child>
+ <child>
<object class="GtkVPaned" id="vpaned">
<property name="visible">True</property>
<property name="can_focus">True</property>
diff --git a/meld/vcview.py b/meld/vcview.py
index deb044f..83525bf 100644
--- a/meld/vcview.py
+++ b/meld/vcview.py
@@ -561,6 +561,19 @@ class VcView(melddoc.MeldDoc, gnomeglade.Component):
self._open_files(self._get_selected_files())
def show_patch(self, prefix, patch, silent=False):
+ if vc._vc.call(["which", "patch"]):
+ primary = _("Patch tool not found")
+ secondary = _("Meld needs the <i>patch</i> tool to be installed "
+ "to perform comparisons in %s repositories. Please "
+ "install <i>patch</i> and try again.") % self.vc.NAME
+
+ msgarea = self.msgarea_mgr.new_from_text_and_icon(
+ gtk.STOCK_DIALOG_ERROR, primary, secondary)
+ msgarea.add_button(_("Hi_de"), gtk.RESPONSE_CLOSE)
+ msgarea.connect("response", lambda *args: self.msgarea_mgr.clear())
+ msgarea.show_all()
+ return False
+
tmpdir = tempfile.mkdtemp("-meld")
self.tempdirs.append(tmpdir)
@@ -590,35 +603,28 @@ class VcView(melddoc.MeldDoc, gnomeglade.Component):
self.emit("create-diff", d)
return True
elif not silent:
- import meldapp
- msg = _("""
- Invoking 'patch' failed.
-
- Maybe you don't have 'GNU patch' installed,
- or you use an untested version of %s.
-
- Please send email bug report to:
- meld-list gnome org
-
- Containing the following information:
-
- - meld version: '%s'
- - source control software type: '%s'
- - source control software version: 'X.Y.Z'
- - the output of '%s somefile.txt'
- - patch command: '%s'
- (no need to actually run it, just provide
- the command line)
-
- Replace 'X.Y.Z' by the actual version for the
- source control software you use.
- """) % (self.vc.NAME,
- meldapp.version,
- self.vc.NAME,
- " ".join(self.vc.diff_command()),
- " ".join(patchcmd))
- msg = '\n'.join([line.strip() for line in msg.split('\n')])
- misc.run_dialog(msg, parent=self)
+ primary = _("Error fetching original comparison file")
+ secondary = _("Meld couldn't obtain the original version of your "
+ "comparison file. If you are using the most recent "
+ "version of Meld, please report a bug, including as "
+ "many details as possible.")
+
+ msgarea = self.msgarea_mgr.new_from_text_and_icon(
+ gtk.STOCK_DIALOG_ERROR, primary, secondary)
+ msgarea.add_button(_("Hi_de"), gtk.RESPONSE_CLOSE)
+ msgarea.add_button(_("Report a bug"), gtk.RESPONSE_OK)
+
+ def patch_error_cb(msgarea, response):
+ if response == gtk.RESPONSE_OK:
+ bug_url = "https://bugzilla.gnome.org/enter_bug.cgi?" + \
+ "product=meld"
+ misc.open_uri(bug_url)
+ else:
+ self.msgarea_mgr.clear()
+
+ msgarea.connect("response", patch_error_cb)
+ msgarea.show_all()
+
return False
def refresh(self):
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]