* bug#3865: 23.0.95; doc-view-mode with PDF without PNG support
@ 2009-07-16 9:01 ` Reiner Steib
2009-07-17 19:50 ` bug#3865: marked as done (23.0.95; doc-view-mode with PDF without PNG support) Emacs bug Tracking System
0 siblings, 1 reply; 12+ messages in thread
From: Reiner Steib @ 2009-07-16 9:01 UTC (permalink / raw)
To: emacs-pretest-bug
Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:
To reproduce:
M-x toggle-debug-on-error RET
M-x toggle-debug-on-quit RET
C-x C-f some-pdf-file.pdf
I get the message:
No PNG support available or some conversion utility for pdf files is
missing. Type C-c C-c to switch to fundamental-mode, C-c C-t to
show the doc as text in a separate buffer or k to kill this buffer.
I don't have the png-DLL installed:
(image-type-available-p 'png) -> nil
Maybe also some of the conversion programs are not installed. But I
just wanted to view/edit the PDF file.
Problems:
- The message is too long to be read in the echo area.
- When pressing some key (e.g. `down'), I get "image-next-line: Invalid
image specification". So the first message disappears as soon as the
user does a movement.
This hit me several times. I didn't realize the message so I killed the
buffer and used `find-file-literally'.
I think the right behavior would be either...
(1) Switch to fundamental-mode automatically if doc-view-mode cannot do
anything useful and just display a message.
(2) Instead of telling about `C-c C-c', `C-c C-t' and `k', use a
(y-or-n-p) dialog.
In GNU Emacs 23.0.95.1 (i386-mingw-nt6.0.6001)
of 2009-06-20 on SOFT-MJASON
Windowing system distributor `Microsoft Corp.', version 6.0.6001
configured using `configure --with-gcc (3.4)'
Important settings:
value of $LC_ALL: nil
value of $LC_COLLATE: nil
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: DEU
value of $XMODIFIERS: nil
locale-coding-system: cp1252
default-enable-multibyte-characters: t
Major mode: Debugger
Minor modes in effect:
show-paren-mode: t
savehist-mode: t
recentf-mode: t
msb-mode: t
iswitchb-mode: t
desktop-save-mode: t
tooltip-mode: t
tool-bar-mode: t
mouse-wheel-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
column-number-mode: t
line-number-mode: t
transient-mark-mode: t
Recent input:
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <tool-bar> <server-start>
<help-echo> <help-echo> <down-mouse-1> <mouse-1> C-x
C-f <up> <up> <up> <up> C-g M-x t o g g <tab> d e <tab>
e <tab> <return> M-x <up> <end> <backspace> <backspace>
<backspace> <backspace> <backspace> q <tab> <return>
C-x C-f <up> <up> <up> <return> <down> M-x r e p o
r t - e m <tab> <return>
Recent messages:
Cleaning up the recentf list...done (0 removed)
No desktop file.
For information about GNU Emacs and the GNU system, type C-h C-a.
Quit
Debug on Error enabled globally
Debug on Quit enabled globally
No PNG support available or some conversion utility for pdf files is
missing. Type C-c C-c to switch to fundamental-mode, C-c C-t to show
the doc as text in a separate buffer or k to kill this buffer.
Error during redisplay: (wrong-type-argument numberp nil)
Entering debugger...
Error during redisplay: (wrong-type-argument numberp nil) [4 times]
*Backtrace*:
Debugger entered--Lisp error: (error "Invalid image specification")
image-size(nil)
image-next-line(1)
call-interactively(image-next-line nil nil)
Bye, Reiner
^ permalink raw reply [flat|nested] 12+ 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
` (4 more replies)
0 siblings, 5 replies; 12+ 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] 12+ messages in thread
* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
@ 2009-07-17 18:51 Tassilo Horn
0 siblings, 0 replies; 12+ 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] 12+ messages in thread
* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
@ 2009-07-17 19:17 ` Bastien
2009-07-17 19:17 ` Bastien
` (3 subsequent siblings)
4 siblings, 0 replies; 12+ 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] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
@ 2009-07-17 19:17 ` Bastien
2009-07-17 19:34 ` Lennart Borgman
2009-07-17 19:47 ` Tassilo Horn
2009-07-17 19:44 ` Chong Yidong
` (2 subsequent siblings)
4 siblings, 2 replies; 12+ 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] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 19:17 ` Bastien
@ 2009-07-17 19:34 ` Lennart Borgman
2009-07-17 19:47 ` Tassilo Horn
1 sibling, 0 replies; 12+ messages in thread
From: Lennart Borgman @ 2009-07-17 19:34 UTC (permalink / raw)
To: Bastien; +Cc: Emacs-Devel devel
On Fri, Jul 17, 2009 at 9:17 PM, Bastien<bastienguerry@googlemail.com> wrote:
> Tassilo Horn <tassilo@member.fsf.org> writes:
>
>> 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?
Go out, meet some people have fun... ;-)
Good luck, Tassilo!
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
2009-07-17 19:17 ` Bastien
@ 2009-07-17 19:44 ` Chong Yidong
2009-07-16 9:01 ` bug#3865: 23.0.95; doc-view-mode with PDF without PNG support Reiner Steib
2009-07-19 5:34 ` bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch? Stefan Monnier
2009-07-19 5:34 ` Stefan Monnier
4 siblings, 1 reply; 12+ messages in thread
From: Chong Yidong @ 2009-07-17 19:44 UTC (permalink / raw)
To: emacs-devel; +Cc: 3865-done
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've checked it into the trunk. I don't think this is serious enough to
commit to the branch.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 19:17 ` Bastien
2009-07-17 19:34 ` Lennart Borgman
@ 2009-07-17 19:47 ` Tassilo Horn
2009-07-18 5:51 ` Bastien
1 sibling, 1 reply; 12+ messages in thread
From: Tassilo Horn @ 2009-07-17 19:47 UTC (permalink / raw)
To: Bastien; +Cc: emacs-devel
Bastien <bastienguerry@googlemail.com> writes:
Hi Bastien,
>> 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.
Thanks.
>> 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?
Well, in general you first have to look for a woman... :-P
Bye,
Tassilo
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#3865: marked as done (23.0.95; doc-view-mode with PDF without PNG support)
2009-07-16 9:01 ` bug#3865: 23.0.95; doc-view-mode with PDF without PNG support Reiner Steib
@ 2009-07-17 19:50 ` Emacs bug Tracking System
0 siblings, 0 replies; 12+ messages in thread
From: Emacs bug Tracking System @ 2009-07-17 19:50 UTC (permalink / raw)
To: Chong Yidong
[-- Attachment #1: Type: text/plain, Size: 908 bytes --]
Your message dated Fri, 17 Jul 2009 15:44:39 -0400
with message-id <87fxcvnlg8.fsf@stupidchicken.com>
and subject line Re: Patch for #3865: Would somebody please commit on HEAD and branch?
has caused the Emacs bug report #3865,
regarding 23.0.95; doc-view-mode with PDF without PNG support
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)
--
3865: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=3865
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems
[-- Attachment #2: Type: message/rfc822, Size: 5896 bytes --]
From: Reiner Steib <reinersteib+gmane@imap.cc>
To: emacs-pretest-bug@gnu.org
Subject: 23.0.95; doc-view-mode with PDF without PNG support
Date: Thu, 16 Jul 2009 11:01:43 +0200
Message-ID: <20090716110143.d83eb84miog84csg@imap.uni-ulm.de>
Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:
To reproduce:
M-x toggle-debug-on-error RET
M-x toggle-debug-on-quit RET
C-x C-f some-pdf-file.pdf
I get the message:
No PNG support available or some conversion utility for pdf files is
missing. Type C-c C-c to switch to fundamental-mode, C-c C-t to
show the doc as text in a separate buffer or k to kill this buffer.
I don't have the png-DLL installed:
(image-type-available-p 'png) -> nil
Maybe also some of the conversion programs are not installed. But I
just wanted to view/edit the PDF file.
Problems:
- The message is too long to be read in the echo area.
- When pressing some key (e.g. `down'), I get "image-next-line: Invalid
image specification". So the first message disappears as soon as the
user does a movement.
This hit me several times. I didn't realize the message so I killed the
buffer and used `find-file-literally'.
I think the right behavior would be either...
(1) Switch to fundamental-mode automatically if doc-view-mode cannot do
anything useful and just display a message.
(2) Instead of telling about `C-c C-c', `C-c C-t' and `k', use a
(y-or-n-p) dialog.
In GNU Emacs 23.0.95.1 (i386-mingw-nt6.0.6001)
of 2009-06-20 on SOFT-MJASON
Windowing system distributor `Microsoft Corp.', version 6.0.6001
configured using `configure --with-gcc (3.4)'
Important settings:
value of $LC_ALL: nil
value of $LC_COLLATE: nil
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: DEU
value of $XMODIFIERS: nil
locale-coding-system: cp1252
default-enable-multibyte-characters: t
Major mode: Debugger
Minor modes in effect:
show-paren-mode: t
savehist-mode: t
recentf-mode: t
msb-mode: t
iswitchb-mode: t
desktop-save-mode: t
tooltip-mode: t
tool-bar-mode: t
mouse-wheel-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
column-number-mode: t
line-number-mode: t
transient-mark-mode: t
Recent input:
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <tool-bar> <server-start>
<help-echo> <help-echo> <down-mouse-1> <mouse-1> C-x
C-f <up> <up> <up> <up> C-g M-x t o g g <tab> d e <tab>
e <tab> <return> M-x <up> <end> <backspace> <backspace>
<backspace> <backspace> <backspace> q <tab> <return>
C-x C-f <up> <up> <up> <return> <down> M-x r e p o
r t - e m <tab> <return>
Recent messages:
Cleaning up the recentf list...done (0 removed)
No desktop file.
For information about GNU Emacs and the GNU system, type C-h C-a.
Quit
Debug on Error enabled globally
Debug on Quit enabled globally
No PNG support available or some conversion utility for pdf files is
missing. Type C-c C-c to switch to fundamental-mode, C-c C-t to show
the doc as text in a separate buffer or k to kill this buffer.
Error during redisplay: (wrong-type-argument numberp nil)
Entering debugger...
Error during redisplay: (wrong-type-argument numberp nil) [4 times]
*Backtrace*:
Debugger entered--Lisp error: (error "Invalid image specification")
image-size(nil)
image-next-line(1)
call-interactively(image-next-line nil nil)
Bye, Reiner
[-- Attachment #3: Type: message/rfc822, Size: 1948 bytes --]
From: Chong Yidong <cyd@stupidchicken.com>
To: emacs-devel@gnu.org
Cc: 3865-done@emacsbugs.donarmstrong.com
Subject: Re: Patch for #3865: Would somebody please commit on HEAD and branch?
Date: Fri, 17 Jul 2009 15:44:39 -0400
Message-ID: <87fxcvnlg8.fsf@stupidchicken.com>
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've checked it into the trunk. I don't think this is serious enough to
commit to the branch.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 19:47 ` Tassilo Horn
@ 2009-07-18 5:51 ` Bastien
0 siblings, 0 replies; 12+ messages in thread
From: Bastien @ 2009-07-18 5:51 UTC (permalink / raw)
To: emacs-devel
Tassilo Horn <tassilo@member.fsf.org> writes:
>> I also need to learn how to do this. Pointers anyone?
>
> Well, in general you first have to look for a woman... :-P
:)
--
Bastien
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
` (2 preceding siblings ...)
2009-07-17 19:44 ` Chong Yidong
@ 2009-07-19 5:34 ` Stefan Monnier
2009-07-19 5:34 ` Stefan Monnier
4 siblings, 0 replies; 12+ 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] 12+ messages in thread
* Re: Patch for #3865: Would somebody please commit on HEAD and branch?
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
` (3 preceding siblings ...)
2009-07-19 5:34 ` bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch? Stefan Monnier
@ 2009-07-19 5:34 ` Stefan Monnier
4 siblings, 0 replies; 12+ 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] 12+ messages in thread
end of thread, other threads:[~2009-07-19 5:34 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-17 18:51 Patch for #3865: Would somebody please commit on HEAD and branch? Tassilo Horn
2009-07-17 19:17 ` bug#3865: " Bastien
2009-07-17 19:17 ` Bastien
2009-07-17 19:34 ` Lennart Borgman
2009-07-17 19:47 ` Tassilo Horn
2009-07-18 5:51 ` Bastien
2009-07-17 19:44 ` Chong Yidong
2009-07-16 9:01 ` bug#3865: 23.0.95; doc-view-mode with PDF without PNG support Reiner Steib
2009-07-17 19:50 ` bug#3865: marked as done (23.0.95; doc-view-mode with PDF without PNG support) Emacs bug Tracking System
2009-07-19 5:34 ` bug#3865: Patch for #3865: Would somebody please commit on HEAD and branch? Stefan Monnier
2009-07-19 5:34 ` Stefan Monnier
-- strict thread matches above, loose matches on Subject: below --
2009-07-17 18:51 bug#3865: " Tassilo Horn
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.