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 734717 - geary should support elementary's contractor
geary should support elementary's contractor
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: build
0.7.x
Other Linux
: Normal normal
: 0.8.2
Assigned To: Geary Maintainers
Geary Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-08-13 12:05 UTC by steven
Modified: 2014-11-01 00:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add elementary contractor support (1.75 KB, patch)
2014-08-16 01:27 UTC, Viko Adi Rahmawan
none Details | Review
Previous patch cant open file wiht spaces (2.69 KB, patch)
2014-08-17 15:31 UTC, Viko Adi Rahmawan
none Details | Review
Add contractor support (3.71 KB, patch)
2014-10-09 08:43 UTC, Viko Adi Rahmawan
none Details | Review

Description steven 2014-08-13 12:05:55 UTC
Geary should support elementary's contractor at least as a plugin. 
That way pantheon-files and other applications can make use of it and send files per e-mail.
Comment 1 steven 2014-08-13 15:16:38 UTC
According to Viko Adi Rahmawan a command parameter to attach a file on the new message window would be enough to easily create a contract file.
Comment 2 Jim Nelson 2014-08-13 19:26:30 UTC
Geary already supports attachments on the command-line via a mailto: argument.  The mailto: schema supports an "attach" or "attachment" parameter.  This is how Geary interoperates with xdg-open, nautilus-sendto, and LibreOffice.

Is that the only issue?  If so, is there anything else Geary needs to do here?
Comment 3 Jim Nelson 2014-08-14 01:54:27 UTC
A patch to fix this should include all the necessary files (I suspect it's only a geary.contract file) and the necessary CMake code to install it in the right place.

It would be best if the installation code detected Contactor's presence before installing the file.  Otherwise, there should be a ./configure switch to enable/disable it.
Comment 4 Viko Adi Rahmawan 2014-08-15 17:07:40 UTC
Sorry but i still dont get the contractor detection part.
Do you mean that it need to install contract file only if we found contractor package installed or it only detect contractor presence when compiling?
I dont know if CMake can do package detection on package installation (Im new to CMake).
Thanks
Comment 5 Jim Nelson 2014-08-15 17:52:19 UTC
CMake can do package detection, although I'm less sure about it discovering if an application is installed versus a developer package (i.e. headers, .pc file, etc.).

I thought about this last night and realized I was being a bit strict.  If we're only talking about one file being installed in a well-known location, then we don't need to go overboard.  The patch should merely install the file to the location and remove it during uninstall.  This would be very similar to how the .desktop file is treated (or, more simply, how the files in sql/ are installed/uninstalled).
Comment 6 Viko Adi Rahmawan 2014-08-16 01:27:00 UTC
Created attachment 283575 [details] [review]
Add elementary contractor support
Comment 7 Viko Adi Rahmawan 2014-08-16 01:27:31 UTC
this patch will add contractor support in geary.
it have a --disable-contract flag to disable contract installation when compiling

I have to install two files one .contract file in share/contractor and one bash file in bin to support multiple files attachment
Comment 8 steven 2014-08-16 07:58:32 UTC
With your patch will pantheon-files be able to send files via Geary ?
Comment 9 Viko Adi Rahmawan 2014-08-16 08:56:41 UTC
files, calendar, scratch and in the future photos will be able to attach to email. that's the beauty of contractor.
if the patch can make it into geary 0.8 and be backported by elementary you can have it in freya
Comment 10 steven 2014-08-17 08:34:21 UTC
I absolutely want that if you want you can claim the bug on bountysource.
Comment 11 Viko Adi Rahmawan 2014-08-17 15:31:41 UTC
Created attachment 283655 [details] [review]
Previous patch cant open file wiht spaces
Comment 12 Jim Nelson 2014-08-19 19:42:23 UTC
Viko, these patches have the contractor files but I don't see the changes to the CMake files.
Comment 13 Viko Adi Rahmawan 2014-10-09 08:43:00 UTC
Created attachment 288099 [details] [review]
Add contractor support
Comment 14 Jim Nelson 2014-10-10 21:52:05 UTC
Thanks!

Pushed to master, commit 66dc46