Bug 556602

Created attachment 385635 [details]
lesspipe.sh

I have added new *.xz into lesspipe.sh. Please install build http://koji.fedoraproject.org/koji/taskinfo?taskID=1933411 or you can download attachment and copy it into /usr/bin(there is lesspipe.sh)

(In reply to comment #1)

> I have added new *.xz into lesspipe.sh.

Not quite, I guess.  Instead of
  *.xz) tar Jtvvf "$1" ;;
like now it should be, IMO:
  *.tar.xz) tar Jtvvf "$1" ;;
  *.xz) xz -dc  -- "$1" ;;
similar to other compress programs.  Also it looks like that you forgot
the following change:
--- ./lesspipe.sh.orig  2010-01-20 02:30:01.000000000 -0700
+++ ./lesspipe.sh       2010-01-20 18:32:16.000000000 -0700
@@ -43,6 +43,7 @@
        case "$1" in
                *.gz)   DECOMPRESSOR="gunzip -c" ;;
                *.bz2)  DECOMPRESSOR="bunzip2 -c" ;;
+               *.xz)   DECOMPRESSOR="unxz -c" ;;
        esac
        if [ ! -z $DECOMPRESSOR ] ; then
                $DECOMPRESSOR -- "$1" ;


It is true that currently manpages are not compressed with xz but, once at this, I think that I would add on line 8 of lesspipe.sh all patterns which would be needed to handle such cases.  Just the same as all ".gz" patterns repeated with ".xz" instead.  It may come handy in the future and does not mess anything for now.

BTW - "*.tar.bz2" is processed with a pipe to tar instead of '-j' tar flag.  It really does not matter that much but maybe some consistency across different cases would be good?

Created attachment 385829 [details]
proposed patch to the lesspipe.sh from koji

Looking at patterns for manpage handling I only now noticed that one may have
.n suffix, and .n.bz2 but not .n.gz and I repeated that for .xz in my patch.
OTOH files from /usr/share/man/man?x directories are handled if they are gzip
compressed (and with xz with my patch) but not if bzip2 was used.  Still manpages from /usr/share/man[013]p/ directories are not covered by any pattern.  Weird.
I guess that this should be fixed too.

aaaahaaaa we are talking about manpages. I thought that you want to see file list with less not what is inside files. Yep that make sense.

(In reply to comment #5)
> aaaahaaaa we are talking about manpages.  

Not really.  Look closer.  manpages came up as an aside when I started checking what really happens in lesspipe.sh.  In attachment (id=385829) only the first chunk is for that, and as I wrote later not really complete, and the remaining chunks are "general".  I thought that comment #2 should be clear enough.  Just omit there a paragraph starting with "It is true ..." if you will.

Ok I'm fully in the picture. I will apply patch soon and put there new build and lesspipe.sh(or patch)

Created attachment 385994 [details]
lesspipe.sh

new lesspipe.sh and build http://koji.fedoraproject.org/koji/taskinfo?taskID=1936796

less-436-6.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/less-436-6.fc12

less-436-6.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update less'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-1342

less-436-6.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.