GNU bug report logs - #11557
cannot view nonempty *.x file using emacs (Image[imagemagick])

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 11557 in the body.
You can then email your comments to 11557 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: bug-emacs <at> gnu.org
Subject: cannot view nonempty *.x file using emacs (Image[imagemagick])
Date: Fri, 25 May 2012 17:56:36 +0200

I was surprised to find that emacs now tries to display
all of coreutils' man/*.x files as images.  It seems to
treat any nonempty *.x file that way.

Instead of seeing what I normally treat as a simple text file with
man file [section]/text snippets, all I see is a small square (maybe
20x20 pixels) in the upper left corner of the window.

Reproduce via e.g.,

    echo foo > a.x && emacs -q a.x

The mode line reports that it is in (Image[imagemagick]) mode.

What version?

  $ emacs --version
  GNU Emacs 24.1.50.1

I build snapshots from bzr nearly daily, and could quickly bisect down to
this 3-day interval:

  GOOD emacs-2012-04-13.06h37/bin/emacs
  BAD emacs-2012-04-16.08h09/bin/emacs

When it works as I expected (i.e., I can see actual text in the primary
buffer, and no outlined square), I see "LD-script" in the mode line.

Message #8 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Jim Meyering <jim <at> meyering.net>
Cc: 11557 <at> debbugs.gnu.org
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Fri, 25 May 2012 12:23:53 -0400

Jim Meyering wrote:

> I was surprised to find that emacs now tries to display
> all of coreutils' man/*.x files as images.  It seems to
> treat any nonempty *.x file that way.

Same issue as http://debbugs.gnu.org/11521

(memq 'X (imagemagick-types))  ->  (X X3F ...)

Message #11 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 11557 <at> debbugs.gnu.org
Cc: Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Fri, 25 May 2012 13:09:46 -0400

How about turning this ImageMagick thing around:

Instead of enabling image support for the 100s of extensions that
ImageMagick thinks are images, minus a few exceptions
(imagemagick-types-inhibit), have an explicit customizable list of the
types to accept, defaulting to things "everyone" agrees are really
images (bmp etc).

Message #14 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 11557 <at> debbugs.gnu.org
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Fri, 25 May 2012 19:18:21 +0200

Glenn Morris wrote:
> How about turning this ImageMagick thing around:
>
> Instead of enabling image support for the 100s of extensions that
> ImageMagick thinks are images, minus a few exceptions
> (imagemagick-types-inhibit), have an explicit customizable list of the
> types to accept, defaulting to things "everyone" agrees are really
> images (bmp etc).

That sounds right to me.

Message #17 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Jim Meyering <jim <at> meyering.net>
Cc: Glenn Morris <rgm <at> gnu.org>, 11557 <at> debbugs.gnu.org
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Sat, 26 May 2012 09:21:05 -0400

>> How about turning this ImageMagick thing around:
>> Instead of enabling image support for the 100s of extensions that
>> ImageMagick thinks are images, minus a few exceptions
>> (imagemagick-types-inhibit), have an explicit customizable list of the
>> types to accept, defaulting to things "everyone" agrees are really
>> images (bmp etc).
> That sounds right to me.

Sounds obvious enough, yes, please go ahead,


        Stefan

Message #20 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 11557 <at> debbugs.gnu.org, Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Thu, 31 May 2012 03:25:55 -0400

I've added an option imagemagick-type-enable with a conservative default.
I didn't put much thought into the default value.
I also wondered if for symmetry imagemagick-types-inhibit should maybe
be renamed to imagemagick-type-disable (or use -include and -exclude).

If anyone wants to change either of those it's fine by me.

Message #27 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 11557 <at> debbugs.gnu.org, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Thu, 31 May 2012 10:14:10 +0200

Glenn Morris wrote:
> I've added an option imagemagick-type-enable with a conservative default.
> I didn't put much thought into the default value.
> I also wondered if for symmetry imagemagick-types-inhibit should maybe
> be renamed to imagemagick-type-disable (or use -include and -exclude).
>
> If anyone wants to change either of those it's fine by me.

Thanks!  This fixes it.

FYI, I've just noticed that bison/yacc grammar files were also affected.
I.e., editing any *.y file, the default mode would display only that
small useless square in the upper left corner.

Message #30 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 11557 <at> debbugs.gnu.org, Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Thu, 31 May 2012 17:29:50 -0400

> I've added an option imagemagick-type-enable with a conservative default.
> I didn't put much thought into the default value.
> I also wondered if for symmetry imagemagick-types-inhibit should maybe
> be renamed to imagemagick-type-disable (or use -include and -exclude).

I don't think imagemagick-types-inhibit is important enough now to worry
about its name.  We could even mark is obsolete, couldn't we?


        Stefan

Message #33 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: Glenn Morris <rgm <at> gnu.org>, 11557 <at> debbugs.gnu.org,
	Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Sat, 02 Jun 2012 15:31:01 +0800

Stefan Monnier <monnier <at> IRO.UMontreal.CA> writes:

> I don't think imagemagick-types-inhibit is important enough now to worry
> about its name.  We could even mark is obsolete, couldn't we?

I don't think we should mark it as obsolete, since some users might want
to use a blacklist rather than a whitelist.  I changed its default to
nil, though, and added many more (hopefully uncontroversial) types to
the whitelist.

Message #36 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Chong Yidong <cyd <at> gnu.org>
Cc: 11557 <at> debbugs.gnu.org, Stefan Monnier <monnier <at> IRO.UMontreal.CA>,
	Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Sat, 02 Jun 2012 15:42:51 -0400

Chong Yidong wrote:

> I don't think we should mark it as obsolete, since some users might want
> to use a blacklist rather than a whitelist.  I changed its default to
> nil,

I don't see what the point of changing the inhibit default to nil was.
It makes it harder for someone to set imagemagick-enabled-types = t.

> , and added many more (hopefully uncontroversial) types to the
> whitelist.

CIN is controversial: http://debbugs.gnu.org/11521

Why are EPS/EPDF files treated differently to PS/PDF files?

Message #39 received at 11557 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> gnu.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 11557 <at> debbugs.gnu.org, Stefan Monnier <monnier <at> IRO.UMontreal.CA>,
	Jim Meyering <jim <at> meyering.net>
Subject: Re: bug#11557: cannot view nonempty *.x file using emacs
	(Image[imagemagick])
Date: Mon, 04 Jun 2012 21:02:51 +0800

Glenn Morris <rgm <at> gnu.org> writes:

> I don't see what the point of changing the inhibit default to nil was.
> It makes it harder for someone to set imagemagick-enabled-types = t.

I guess you're right.  Switched back.

> CIN is controversial: http://debbugs.gnu.org/11521

OK, removed.

> Why are EPS/EPDF files treated differently to PS/PDF files?

My original thought was that the "encapsulated" versions tend to be
figures rather than documents.  But on second thought, since EPS
etc. are displayed just fine by DocView, there is no need to have those
there.  Removed.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.