|Summary:||volnoti-0.1.ebuild (New package)|
|Product:||Gentoo Linux||Reporter:||David Brazdil <david.brazdil>|
|Component:||New packages||Assignee:||Default Assignee for New Packages <maintainer-wanted>|
|Package list:||Runtime testing required:||---|
volnoti-0.1.ebuild (New package)
Description David Brazdil 2011-12-21 20:06:38 UTC
Created attachment 296599 [details] volnoti-0.1.ebuild (New package) Hi guys, Here's an ebuild for my latest piece of code - volnoti. I suggest you put it into x11-misc/volnoti. As you can read in the README: Volnoti is a lightweight volume notification daemon for GNU/Linux and other POSIX operating systems. It is based on GTK+ and D-Bus and should work with any sensible window manager. The original aim was to create a volume notification daemon for lightweight window managers like LXDE or XMonad. It is known to work with a wide range of WMs, including GNOME, KDE, Xfce, LXDE, XMonad, i3 and many others. The source code is based on the GNOME notification-daemon. Cheers, Dave
Comment 1 Agostino Sarubbo 2011-12-21 21:20:42 UTC
Hi, you can improve your ebuild. First make it as EAPI4, no need src_unpack and src_install No need die after emake in src_compile no need as depend "make" and "gcc" github is a thirdy party the keywords should be ~
Comment 2 David Brazdil 2011-12-21 21:39:52 UTC
Created attachment 296607 [details] volnoti-0.1.ebuild Fixed according to Agostino Sarubbo's recommendations
Comment 3 David Brazdil 2011-12-21 21:40:54 UTC
Thanks Agostino... Is it a problem that it's hosted on GitHub? Or did you just mean that therefore the keywords should be ~?
Comment 4 Agostino Sarubbo 2011-12-21 21:54:49 UTC
(In reply to comment #3) > Thanks Agostino... Is it a problem that it's hosted on GitHub? Or did you just > mean that therefore the keywords should be ~? no, run repoman full and you will see it take a look at gentoopm-0.2.1.ebuild the syntax like mirror://
Comment 5 David Brazdil 2011-12-21 22:03:46 UTC
Created attachment 296609 [details] volnoti-0.1.ebuild GitHub refered to through mirror://
Comment 6 David Brazdil 2011-12-21 22:04:14 UTC
Fixed, thanks and sorry you had to babysit me...
Comment 7 Agostino Sarubbo 2011-12-21 22:43:41 UTC
(In reply to comment #6) > Fixed, thanks and sorry you had to babysit me... No problem ;) anyway missing IUSE and I guess is not needed to inherit multilib eclass.