On Sun, Jun 1, 2014 at 8:15 AM, Samuli Suominen <ssuomi...@gentoo.org> wrote: > > On 01/06/14 10:04, Tom Gundersen wrote: >> >> >> On 1 Jun 2014 06:31, "Lennart Poettering" <lenn...@poettering.net >> <mailto:lenn...@poettering.net>> wrote: >> > >> > On Sat, 31.05.14 19:43, Samuli Suominen (ssuomi...@gentoo.org >> <mailto:ssuomi...@gentoo.org>) wrote: >> > >> > > >> > > >> > > On 31/05/14 17:10, Samuli Suominen wrote: >> > > > On 31/05/14 14:14, Samuli Suominen wrote: >> > > >> 1. "libsystemd_network_la_SOURCES =" in Makefile.am includes >> > > >> "src/libsystemd-network/network-internal.h" >> > > >> and there is no anykind of #ifdef -logic behind it >> > > >> >> > > >> 2. "src/libsystemd-network/network-internal.h" has #include >> <libkmod.h> >> > > >> and there is no anykind of #ifdef >> > > >> -logic behind it >> > > >> >> > > >> so kmod is hardcoded dependency in systemd-213: >> > > >> >> > > >> >> > > > This patch is a half-complete hack that only works with >> > > > --disable-networkd, as it only covers the files >> > > > that are outside of the #if NETWORKD in Makefile.am >> > > > As in, I've used this patch only for the package within Gentoo that >> > > > builds only udev, not rest of systemd, >> > > > that uses --disable-networkd >> > > > So consider this as proof of consept, not for inclusion >> > > > >> > > > - Samuli >> > > > >> > > > >> > > > _______________________________________________ >> > > > systemd-devel mailing list >> > > > systemd-devel@lists.freedesktop.org >> <mailto:systemd-devel@lists.freedesktop.org> >> > > > http://lists.freedesktop.org/mailman/listinfo/systemd-devel >> > > >> > > I just realized configure.ac <http://configure.ac> already has, >> > > >> > > AS_IF([test "x$have_networkd" = "xyes" -a "x$have_kmod" != "xyes"], >> > > [AC_MSG_ERROR([networkd requires kmod])]) >> > > >> > > So this patch should be considered for inclusion after all as-is. You >> > > should credit "Mike Gilbert <flop...@gentoo.org >> <mailto:flop...@gentoo.org>>" for >> > > the patch, not me. >> > >> > I'd prefer I we could fix this properly and allow networkd build without >> > kmod. I am very cool on linking against kmod from networkd anyway (in >> > particular since I want networkd to run without CAP_SYS_MODULE), so I >> > think this could be the least we should be doing... >> >> We should be able to drop the whole kmod thing from networkd any day >> now (a fix is in net-next). Merging it was probably a mistake in the >> first place. >> >> > > So, how about applying the patch from this thread to fix the immediate > problem? (This reply isn't aimed specifically to you, but to anyone with > such > deciding/commit powers really.)
I'll review and apply on Monday if no one beats me to it, I'm currently on holiday and don't have the access. -t _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel