Created attachment 333234 [details, diff] Ebuild patch The patched ebuild changes the config path in the systemd service file and install it on the correct place.
Created attachment 335768 [details] collectd-5.2.0.ebuild Here's an updated ebuild that incorporates this change (and some other nitpicks). The merge commit can be found here: https://github.com/jbergstroem/gentoo-overlay/compare/collectd-systemd
+ 16 Jan 2013; Sergey Popov <pinkbyte@gentoo.org> +collectd-5.2.0-r1.ebuild: + Revision bump: add installation of systemd service file and other little + fixes, wrt bug #448480. Thanks to Johan Bergström One thing: i did not reorganize eclasses. Johan, are you sure that there are no side effects in this action?
(In reply to comment #2) > + 16 Jan 2013; Sergey Popov <pinkbyte@gentoo.org> +collectd-5.2.0-r1.ebuild: > + Revision bump: add installation of systemd service file and other little > + fixes, wrt bug #448480. Thanks to Johan Bergström > > One thing: i did not reorganize eclasses. Johan, are you sure that there are > no side effects in this action? If there are, I didn't encounter them. I'd love to read if there's any documentation regarding this, since I often come across people wanting me to sort variables, use flags and whatnot.
(In reply to comment #3) > If there are, I didn't encounter them. Good, then i will proceed with this > I'd love to read if there's any documentation regarding this, since I often > come across people wanting me to sort variables, use flags and whatnot. Usually sorting improves readability, but exported phase functions(e.g. base_src_prepare) is used from LAST inherited eclass(of course, if you did not override src_prepare() in ebuild itself). For example, games herd policy is to alwasy include games eclass LAST. Anyway, sometimes this matters, but it seems that it is not in this case.
+ 16 Jan 2013; Sergey Popov <pinkbyte@gentoo.org> collectd-5.2.0-r1.ebuild: + Sort eclasses inherit order, wrt bug #448480. Changes by Johan Bergström
