ipse666 / cueplayer

Аудио/Видео проигрыватель, конвертор.
0 stars 0 forks source link

libnotify support. #6

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hey, what do you think about libnotify notifications? I made a provisional
support for libnotify. Files in the attach. It requires libnotify-dev.

Original issue reported on code.google.com by michal.m...@gmail.com on 15 May 2010 at 9:18

Attachments:

GoogleCodeExporter commented 9 years ago
Make out please all in the form of a patch
diff -u original_cueplayer_directory/ changed_cueplayer_directory/ 
>libnotify.patch
Also send me already these patches that I could look changes.
Specify in the message svn revision number cueplayer (better use the last,  the
newest) over which changes were made.

Besides, to me have offered today more universal path of the notifications
http://unixforum.org/index.php?s=&showtopic=99937&view=findpost&p=1063800
I would like to depend as less as far as possible on indirect libraries, 
therefore I
offer:
1. To include libnotify pays attention on optional at assemblage, by default
separated.
2. Possibility of a choice of type of the notifications or its disabled.

If can realise, I wait a patch.

Original comment by ipse666@gmail.com on 17 May 2010 at 12:12

GoogleCodeExporter commented 9 years ago
Here you are. But notice it is a 'makeshift' version.

Original comment by michal.m...@gmail.com on 17 May 2010 at 12:30

Attachments:

GoogleCodeExporter commented 9 years ago
Has included optional, for inclusion at to configurate so:
qmake "DEFINES + = LIBNOTIFY"

Original comment by ipse666@gmail.com on 17 May 2010 at 5:33

GoogleCodeExporter commented 9 years ago
It's work! Thanks.

Original comment by michal.m...@gmail.com on 17 May 2010 at 8:19

GoogleCodeExporter commented 9 years ago

Original comment by ipse666@gmail.com on 18 May 2010 at 9:34