Re: [Nautilus-list] [PATCH] for bug 43302

[Alex Larsson <alexl redhat com>]

On Tue, 23 Apr 2002, Gaute Lindkvist wrote:

> That is.. add themes from .tar.gz-files.
> 
> This patch let's you add themes from .tar-files, .tar.gz and .tar.bz2.
> Most of it is just ripped from the .tar.gz-handler in the
> gtk-theme-capplet and adjusted for Nautilus.
> I added a FIXME since there should probably be a better way of finding out
> if the file is bz2 or not, but it should work fine for now.

You should be using gnome_vfs_get_file_mime_type() and look at the 
mimetypes.

Don't use a fixed buffer for the command, use g_strdup_printf().

If the file is not the correct type it should return
NAUTILUS_THEME_INSTALL_NOT_A_THEME_FILE, and a correct error message for 
it should be added to src/nautilus-theme-selector.c


-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's an unconventional soccer-playing stage actor with nothing left to lose. 
She's a time-travelling hip-hop queen of the dead married to the Mob. They 
fight crime!