I think I have a patch available but I am having trouble building tdeaddons. Hopefully I can resolve it today.

Calvin

Please, can you expose your patch?
I can test it in my build environment...

Created attachment 811 [details]
a patch

my patch just adds the right mimetype. hope it works!b

I tested this patch in R14 GIT. With tar.xz or txz files I did not have an "Extract" or "Extract to" popup menu option. With tar.gz and tgz I do.

I remember when we added lzma and xz support long ago. Several patches were needed. Possibly then more than just tdeaddons needs patching to provide the Extract menu option.

The original lzma/xz commits:

f7a66a3f  [tdeutils]  Added missing mimetypes to ark_part desktop file for lzma and xz support
08f47b25  [tdeaddons] Enable xz and lzma compress menu support by default
29f1d0b4  [tdeaddons] Added xz and lzma support to the compress menu
2de1f0b1  [tdelibs]   Added new mimetypes to Makefile
faf41e2e  [tdelibs]   More xz/lzma fixes
a4043f1f  [tdeutils]  Preliminary xzip/LZMA support
ffcb34f4  [tdelibs]   Preliminary xzip/LZMA support

Darrell,

Thanks for the pointers. I was mostly hoping that the correct mimetype was not registering, but it seems to be more to it than that.

I'll keep hunting.

Created attachment 813 [details]
Patch to add Extract popup menu item

The attached patch works for me. Somebody else needs to test before pushing to GIT or backporting to 3.5.13.1.

Testing it out now, seems my patch was simply in the wrong correction. I am going to add both of those mimetypes (the tgz and the x-compressed... one) and test it.

Created attachment 814 [details]
a final patch with both mimetypes included

Here is a final patch! tested with 3.5.13.SRU works well!

please push it :)

Calvin

What is x-xz-compressed-tar and why needed?

Should brackets be used? Otherwise the second command is executed all the time rather than as part of the "if ( havexz )" condition.

diff -urN tdeaddons/konq-plugins/arkplugin/arkplugin.cpp tdeaddons.new/konq-plugins/arkplugin/arkplugin.cpp
--- tdeaddons/konq-plugins/arkplugin/arkplugin.cpp  2012-05-27 18:04:24.000000000 -0500
+++ tdeaddons.new/konq-plugins/arkplugin/arkplugin.cpp  2012-09-12 16:16:07.000000000 -0500
@@ -404,7 +404,10 @@
         if ( havelzma )
             m_extractMimeTypes << "application/x-lzma";
         if ( havexz )
-            m_extractMimeTypes << "application/x-xz";
+        {
+            m_extractMimeTypes << "application/x-txz";
+            m_archiveMimeTypes << "x-xz-compressed-tar";
+        }
     }

     if ( !KStandardDirs::findExe( "lha" ).isNull() )

Yes :) sorry I was typing from my cell phone, thankfully you caught it

Regarding x-xz-compressed-tar:

The new mimetype should be added everywhere else too, as well as x-lzma-compressed-tar. That is, the new mimetypes do not (yet) appear anywhere in tdelibs, tdebase, tdeutils, tdeaddons. If we add new mimetypes then add the support correctly.

Seems for this bug report we should use the second patch as-is and open a new bug report to add support for the new mimetypes.

Browsing /usr/share/mime on my system reveals the following mimetypes do not seem fully supported in Trinity:

x-7z-compressed
x-cpio-compressed
x-zip-compressed
x-compressed-tar
x-bzip-compressed-tar
x-xz-compressed-tar
x-lzma-compressed-tar
x-lzop-compressed-tar

There probably are additional "x-*-compressed*" mimetypes.

I don't know how the full mimetype scheme works in Trinity, but I suspect we need to update the following:

ark.desktop
ark_part.desktop
tdeutils
tdeaddons
tdelibs

Done correctly, icon associations would be needed too for each new mimetype.

(In reply to comment #12)
> Browsing /usr/share/mime on my system reveals the following mimetypes do not
> seem fully supported in Trinity:
> 
> x-7z-compressed
> x-cpio-compressed
> x-zip-compressed
> x-compressed-tar
> x-bzip-compressed-tar
> x-xz-compressed-tar
> x-lzma-compressed-tar
> x-lzop-compressed-tar
> 
> There probably are additional "x-*-compressed*" mimetypes.
> 
> I don't know how the full mimetype scheme works in Trinity, but I suspect we
> need to update the following:
> 
> ark.desktop
> ark_part.desktop
> tdeutils
> tdeaddons
> tdelibs
> 
> Done correctly, icon associations would be needed too for each new mimetype.

Okay a separate report would be good. You can assign it to me.

Bug report 1213 created to address the missing mometypes.

The patch in attachment 813 [details] was pushed to GIT in commit 6f0ebd61.

Calvin/Slavek: The patch needs to be backported to 3.5.13.1.

Pushed also into v3.5.13-sru.