[Bug-tar] PATCH: LZO patch (ping)

[Kevin Day]

So, some time again there was discussion on adding both LZO and LZMA.
LZMA recently made it in, but LZO has not yet merged.

LZO is very easy to add, see patch below.

So whats going on here?
Was it decided that LZMA would make it in and not LZO?

Is it because LZMA is more popular in the proprietary world?

Here is my lzop-tar patch against the latest:
diff -r -u tar-1.20.orig/src/buffer.c tar-1.20/src/buffer.c
--- tar-1.20.orig/src/buffer.c  2008-05-18 14:06:01 -0500
+++ tar-1.20/src/buffer.c       2008-05-18 14:10:07 -0500
@@ -202,6 +202,7 @@
   ct_compress,
   ct_gzip,
   ct_bzip2,
+  ct_lzop,
   ct_lzma
 };

@@ -219,6 +220,7 @@
   { ct_compress, 2, "\037\235", "compress", "-Z" },
   { ct_gzip,     2, "\037\213", "gzip", "-z"  },
   { ct_bzip2,    3, "BZh",      "bzip2", "-j" },
+  { ct_lzop,     4, "\211LZO",  "lzop", "-q" },
   { ct_lzma,     6, "\xFFLZMA", "lzma", "--lzma" }, /* FIXME: ???? */
 };

diff -r -u tar-1.20.orig/src/tar.c tar-1.20/src/tar.c
--- tar-1.20.orig/src/tar.c     2008-05-18 14:06:01 -0500
+++ tar-1.20/src/tar.c  2008-05-18 14:08:18 -0500
@@ -594,6 +594,8 @@
    N_("use archive suffix to determine the compression program"), GRID+1 },
   {"bzip2", 'j', 0, 0,
    N_("filter the archive through bzip2"), GRID+1 },
+  {"lzop", 'q', 0, 0,
+   N_("filter the archive through lzop"), GRID+1 },
   {"gzip", 'z', 0, 0,
    N_("filter the archive through gzip"), GRID+1 },
   {"gunzip", 0, 0, OPTION_ALIAS, NULL, GRID+1 },
@@ -1428,6 +1430,10 @@
       absolute_names_option = true;
       break;

+    case 'q':
+      set_use_compress_program_option ("lzop");
+      break;
+
     case 'r':
       set_subcommand_option (APPEND_SUBCOMMAND);
       break;

-- 
Kevin Day