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 633485 - Move nautilus-sendto extension to nautilus
Move nautilus-sendto extension to nautilus
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-10-29 17:16 UTC by Bastien Nocera
Modified: 2010-11-01 13:18 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Move nautilus-sendto extension to nautilus (10.39 KB, patch)
2010-10-29 17:17 UTC, Bastien Nocera 		reviewed Details | Review
Move nautilus-sendto extension to nautilus (11.62 KB, patch)
2010-11-01 12:24 UTC, Bastien Nocera 		committed Details | Review

Description Bastien Nocera 2010-10-29 17:16:58 UTC 
As discussed on IRC.

Rhythmbox and evolution already embed their own extensions.
Comment 1 Bastien Nocera 2010-10-29 17:17:00 UTC 
Created attachment 173510 [details] [review]
Move nautilus-sendto extension to nautilus

And add a check for the presence of nautilus-sendto to the
initialisation of the object.
Comment 2 Cosimo Cecchi 2010-11-01 12:02:51 UTC 
Review of attachment 173510 [details] [review]:

Looks good; I only would like to see a configure switch to disable building the extension.
Comment 3 Bastien Nocera 2010-11-01 12:24:49 UTC 
Created attachment 173629 [details] [review]
Move nautilus-sendto extension to nautilus

And add a check for the presence of nautilus-sendto to the
initialisation of the object.

Building the extension is optional, though on by default.
Comment 4 Cosimo Cecchi 2010-11-01 13:13:37 UTC 
Review of attachment 173629 [details] [review]:

Thanks, feel free to push to master.
Comment 5 Bastien Nocera 2010-11-01 13:18:39 UTC 
Attachment 173629 [details] pushed as b4aa3c0 - Move nautilus-sendto extension to nautilus