After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 356258 - make thunderbird plugin work
make thunderbird plugin work
Status: RESOLVED FIXED
Product: nautilus-sendto
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: nautilus-sendto-maint
nautilus-sendto-maint
: 510182 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-09-16 13:13 UTC by Matthias Clasen
Modified: 2008-03-27 18:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (1.40 KB, patch)
2006-09-16 13:14 UTC, Matthias Clasen
none Details | Review
my approach to get thunderbird working (2.26 KB, patch)
2006-11-03 14:54 UTC, Whoopie
none Details | Review
updated patch for nautilus-sendto 0.10 (2.05 KB, patch)
2007-03-12 13:57 UTC, Whoopie
none Details | Review
thunderbird support (2.04 KB, patch)
2007-06-08 14:28 UTC, Whoopie
none Details | Review
updated patch for nautilus-sendto 0.12 (2.38 KB, patch)
2007-09-03 12:54 UTC, Whoopie
none Details | Review
fix thunderbird name and attachment escaping (1.43 KB, patch)
2007-09-09 09:36 UTC, Whoopie
needs-work Details | Review

Description Matthias Clasen 2006-09-16 13:13:39 UTC
I have put the following patch in the Fedora package, to make 
the thunderbird plugin work.
Comment 1 Matthias Clasen 2006-09-16 13:14:21 UTC
Created attachment 72896 [details] [review]
patch
Comment 2 Whoopie 2006-11-03 14:54:43 UTC
Created attachment 75934 [details] [review]
my approach to get thunderbird working

Hi,

I couldn't get it to work with your patch. I looked at http://www.interveritas.net/thunderbird/ how it was implemented before and used this method.
I also added a "check" if it's an URI or not, as file-roller gives the file back as "/tmp/whatever/file.tar.gz" and not as "file:///tmp/whatever/file.tar.gz".

Best regards,
Whoopie
Comment 3 Whoopie 2007-03-12 13:57:53 UTC
Created attachment 84428 [details] [review]
updated patch for nautilus-sendto 0.10

updated patch for nautilus-sendto 0.10
Comment 4 Whoopie 2007-06-08 14:28:39 UTC
Created attachment 89610 [details] [review]
thunderbird support

only difference is
- t_cmd = g_find_program_in_path ("mozilla-thunderbird");
+ t_cmd = g_find_program_in_path ("thunderbird");
so that it works with thunderbird 2.0 in Ubuntu Gutsy.
Comment 5 Whoopie 2007-09-03 12:54:10 UTC
Created attachment 94861 [details] [review]
updated patch for nautilus-sendto 0.12

This patch is an update to work with nautilus-sendto 0.12 and Thunderbird 2.0.
Comment 6 Whoopie 2007-09-09 09:36:45 UTC
Created attachment 95204 [details] [review]
fix thunderbird name and attachment escaping

This patch fixes the thunderbird name and the escaping for the attachment parameter.
Comment 7 Bastien Nocera 2007-09-09 10:32:31 UTC
Comment on attachment 95204 [details] [review]
fix thunderbird name and attachment escaping

>--- nautilus-sendto-0.12-orig/src/plugins/thunderbird.c	2007-09-09 10:28:11.000000000 +0200
>+++ nautilus-sendto-0.12/src/plugins/thunderbird.c	2007-09-09 11:15:35.000000000 +0200
<snip>
>-	g_string_append_printf (mailto," attachment='\"%s\"'",file_list->data);
>+	g_string_append_printf (mailto,"\"attachment='%s",file_list->data);

Please follow the coding style used. Also, why not use g_shell_quote instead of putting the quotes by hand?
Comment 8 Bastien Nocera 2008-01-20 13:44:47 UTC
*** Bug 510182 has been marked as a duplicate of this bug. ***
Comment 9 Bastien Nocera 2008-03-27 18:29:24 UTC
2008-03-27  Bastien Nocera  <hadess@hadess.net>

        * configure.in: Allow distributors to override the
        default thunderbird binary name with
        --with-thunderbird-name=XXX, enable the thunderbird
        plugin by default

        * src/nautilus-sendto-command.c (send_button_cb):
        Fix a memory leak

        * src/plugins/thunderbird.c (init), (send_files):
        Attachments are passed as URIs, so quote them as
        expected

        (Closes: #356258)