* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
@ 2009-07-17 18:51 Tassilo Horn
0 siblings, 0 replies; 3+ messages in thread
From: Tassilo Horn @ 2009-07-17 18:51 UTC (permalink / raw)
To: emacs-devel; +Cc: 3865
Hi all,
this patch fixes the inconveniences Reiner pointed out in #3865. Could
somebody please commit it for me on branch and head?
I'm to busy to read up how to merge between branches with CVS -- I'm
going to marry tomorrow! :-)
--8<---------------cut here---------------start------------->8---
2009-07-16 Tassilo Horn <tassilo@member.fsf.org>
* doc-view.el (doc-view-initiate-display): Add yes-or-no-p if
rendering of pngs is not possible instead of messaging a long
description.
--8<---------------cut here---------------end--------------->8---
--8<---------------cut here---------------start------------->8---
Index: lisp/doc-view.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/doc-view.el,v
retrieving revision 1.87
diff -u -r1.87 doc-view.el
--- lisp/doc-view.el 2 Apr 2009 21:12:56 -0000 1.87
+++ lisp/doc-view.el 17 Jul 2009 18:44:18 -0000
@@ -1078,15 +1078,12 @@
"editing or viewing the document."))))
(message
"%s"
- (substitute-command-keys
- (concat "No PNG support available or some conversion utility for "
- (file-name-extension doc-view-buffer-file-name)" files is missing. "
- "Type \\[doc-view-toggle-display] to switch to "
- (if (eq doc-view-doc-type 'ps)
- "ps-mode"
- "fundamental-mode")
- ", \\[doc-view-open-text] to show the doc as text in a separate buffer "
- " or \\[doc-view-kill-proc-and-buffer] to kill this buffer.")))))
+ (concat "No PNG support available or some conversion utility for "
+ (file-name-extension doc-view-buffer-file-name) " files is missing."))
+ (if (and (executable-find doc-view-pdftotext-program)
+ (yes-or-no-p "Cannot render file. View extracted text instead? "))
+ (doc-view-open-text)
+ (doc-view-toggle-display))))
(defvar bookmark-make-record-function)
--8<---------------cut here---------------end--------------->8---
Bye,
Tassilo
^ permalink raw reply [flat|nested] 3+ messages in thread
* Patch for #3865: Would somebody please commit on HEAD and branch?
@ 2009-07-17 18:51 Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
2009-07-19 5:34 ` Stefan Monnier
0 siblings, 2 replies; 3+ messages in thread
From: Tassilo Horn @ 2009-07-17 18:51 UTC (permalink / raw)
To: emacs-devel; +Cc: 3865
Hi all,
this patch fixes the inconveniences Reiner pointed out in #3865. Could
somebody please commit it for me on branch and head?
I'm to busy to read up how to merge between branches with CVS -- I'm
going to marry tomorrow! :-)
--8<---------------cut here---------------start------------->8---
2009-07-16 Tassilo Horn <tassilo@member.fsf.org>
* doc-view.el (doc-view-initiate-display): Add yes-or-no-p if
rendering of pngs is not possible instead of messaging a long
description.
--8<---------------cut here---------------end--------------->8---
--8<---------------cut here---------------start------------->8---
Index: lisp/doc-view.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/doc-view.el,v
retrieving revision 1.87
diff -u -r1.87 doc-view.el
--- lisp/doc-view.el 2 Apr 2009 21:12:56 -0000 1.87
+++ lisp/doc-view.el 17 Jul 2009 18:44:18 -0000
@@ -1078,15 +1078,12 @@
"editing or viewing the document."))))
(message
"%s"
- (substitute-command-keys
- (concat "No PNG support available or some conversion utility for "
- (file-name-extension doc-view-buffer-file-name)" files is missing. "
- "Type \\[doc-view-toggle-display] to switch to "
- (if (eq doc-view-doc-type 'ps)
- "ps-mode"
- "fundamental-mode")
- ", \\[doc-view-open-text] to show the doc as text in a separate buffer "
- " or \\[doc-view-kill-proc-and-buffer] to kill this buffer.")))))
+ (concat "No PNG support available or some conversion utility for "
+ (file-name-extension doc-view-buffer-file-name) " files is missing."))
+ (if (and (executable-find doc-view-pdftotext-program)
+ (yes-or-no-p "Cannot render file. View extracted text instead? "))
+ (doc-view-open-text)
+ (doc-view-toggle-display))))
(defvar bookmark-make-record-function)
--8<---------------cut here---------------end--------------->8---
Bye,
Tassilo
^ permalink raw reply [flat|nested] 3+ messages in thread
* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Tassilo Horn
@ 2009-07-17 19:17 ` Bastien
2009-07-19 5:34 ` Stefan Monnier
1 sibling, 0 replies; 3+ messages in thread
From: Bastien @ 2009-07-17 19:17 UTC (permalink / raw)
To: emacs-devel; +Cc: 3865
Tassilo Horn <tassilo@member.fsf.org> writes:
> this patch fixes the inconveniences Reiner pointed out in #3865. Could
> somebody please commit it for me on branch and head?
I'm willing to do this.
> I'm to busy to read up how to merge between branches with CVS -- I'm
> going to marry tomorrow! :-)
I also need to learn how to do this. Pointers anyone?
--
Bastien
^ permalink raw reply [flat|nested] 3+ messages in thread
* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
@ 2009-07-19 5:34 ` Stefan Monnier
1 sibling, 0 replies; 3+ messages in thread
From: Stefan Monnier @ 2009-07-19 5:34 UTC (permalink / raw)
To: emacs-devel; +Cc: 3865
> + (if (and (executable-find doc-view-pdftotext-program)
> + (yes-or-no-p "Cannot render file. View extracted text instead? "))
> + (doc-view-open-text)
> + (doc-view-toggle-display))))
Please avoid modal UIs wherever possible. E.g. rather than ask with
yes-or-no-p, just choose one of the two branches (maybe based on
a config var) and output a message explaining how to get the
other branch.
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-07-19 5:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-17 18:51 bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
-- strict thread matches above, loose matches on Subject: below --
2009-07-17 18:51 Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
2009-07-19 5:34 ` Stefan Monnier
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.