Description of problem: Version-Release number of selected component (if applicable): GNU Emacs 22.1.1 (i386-redhat-linux-gnu, GTK+ Version 2.10.8) of 2007-06-26 on xenbuilder4.fedora.phx.redhat.com How reproducible: Steps to Reproduce: 1. Download and open in Emacs the compressed Emacs lisp file specified by the URL. Actual results: Error while executing "uncompress -c < /home/aaronh/Desktop/unique.el.Z" /bin/sh: uncompress: command not found Expected results: File should be uncompressed with `gzip -d -c -q', since `compress' and `uncompress' are not installed by default or as dependencies. Additional info: Glad to see the Emacs icon in the Emacs-22.1 package! It didn't show up with the Emacs-22.0.99 package.
Created attachment 177701 [details] Patch against jka-cmp-hook.el.diff
Created attachment 190081 [details] Revised patch to jka-cmp-hook.el My first patch was a bit ignorant. Gzip can uncompress compressed files (.Z), but I didn't actually think about (or test) whether gzip can create a compressed file -- compatible with the compress command. I now know that was an oversight. The new patch now uses gzip as an alternative to uncompress, but *not* for compress. So, saving a .Z file in Emacs will now generate the error "/bin/sh: compress: command not found". So still a bit buggy, but I argue that we should enable users to open compress-ed files in Emacs, rather than not at all. If they have trouble writing compressed files, then they should install the ncompress package. More than likely, they should be compressing the file to the gzip or bzip2 format -- or compression may not be paramount, anyway. However, we can generally be assured gzip is installed, so let's use it here! If you'd rather see this upstream, I'm willing to help.
Yeah, IMO sending this upstream would be best. Simplest is M-x report-emacs-bug, then include the patch.
Done. We'll see if it's accepted. http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00056.html
Resolved upstream. Commited to trunk: http://lists.gnu.org/archive/html/emacs-diffs/2007-09/msg00852.html