patch: add libnotify support about grive

[match065]

Thanks for the patch! inotify support is very much needed now.

Why do we need to link with gdk pixbuf?

[match065]

Also, please use tab for indentation. Please set tab width to 4.

See the wiki for the rules:
https://github.com/match065/grive/wiki

[match065]

Sorry that I have to revert the patch. I can't get it compile without libinotify.

[OpenNingia]

gdk pixbuf is needed because notifies uses it to render the icons. Of course you need libnotify to compile, I have no clue of how to make that an optional dependency.

[OpenNingia]

Howdy! here is my second attempt! In this patch I've

• pulled your last commits from origin
• changed spaces into tabs ( width 4 )
• made libnotify optional using a cmake switch
  to enable libnotify support run
  cmake -DWANT_NOTIFY=1 ..

[match065]

Oh... I mean I want inotify, not libnotify.

Inotify is a library to tell us which file is created/modified.
http://www.kernel.org/pub/linux/kernel/people/rml/inotify/README

I googled libnotify, and found that it is about desktop notification. That is also useful, but I wouldn't put it in libgrive. I would prefer to have a separate executable (maybe call it grive_notify?) to use both libgrive and libnotify to notify the desktop. libgrive should be independent of GUI and desktop environments.

[match065]

I think putting the libnotify code in the "grive" executable is also OK. No need to have a separate executable.

[OpenNingia]

ok but then libgrive should communicate the grive executable about events to notify

[match065]

I will close this one first. Let us discuss a good interface between libgrive and grive later.