<p dir="ltr"><br>
On Dec 17, 2012 6:17 AM, "Zbigniew Jędrzejewski-Szmek" <<a href="mailto:zbyszek@in.waw.pl">zbyszek@in.waw.pl</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> eudev has made a project annoucement [1], and I thought it would be<br>
> worthwhile to go through their patches and cherry-pick. I've now done<br>
> that (until cc5c144a70fc37e 'Merge pull request #32') and the results<br>
> regarding patches which can be used directly are not very impressive:<br>
> one patch to fix clang warnings. There are quite a few<br>
> build/compiler-warning in their tree, but they just seem to be fixes<br>
> for problems introduced with their changes.</p>
<p dir="ltr">This has more or less been my takeaway. Its hard to tell what the concrete changes are with all the noise in the commit log.</p>
<p dir="ltr">A clean(er) build with clang would be nice, but I'm not convinced many of the -Wunused warnings are worth fixing.</p>
<p dir="ltr">><br>
> Potentially usefull would also be the patch c189ab 'Fix unused result<br>
> warnings', but it would be good if somebody familar with the code took<br>
> a look if it is enough to print warnings (or even if they should be<br>
> printed) or some more definite action should be taken.<br>
><br>
> Also potentially usefull are the changes to allow 'make distcheck'<br>
> without gtk-doc, but they are spread out over multiple commits in a<br>
> messy way, so could only be used as inspiration.</p>
<p dir="ltr">The distcheck target isn't something you should need to care about users running. I'm -1 on changing it since no one routinely running this target will chime in ;).</p>
<p dir="ltr">> There's also nice patch bfc850 'Add fallback path when accept4() is<br>
> not available.' Do we care about kernels < 2.6.28? (This version is<br>
> according to man accept4(2), patch text mentions different versions.)</p>
<p dir="ltr">I think their concern here is more geared towards ARM which doesn't have a usable accept4 until 2.6.38. Recent udev requires devtmps which puts a minimal requirement of 2.6.32(??) but I think Kay would chime in and not guarantee that everything Just Works™ without a newer kernel even on x86.</p>

<p dir="ltr">Dave</p>
<p dir="ltr">><br>
> Zbyszek<br>
><br>
> [1] <a href="http://thread.gmane.org/gmane.linux.gentoo.devel/81901">http://thread.gmane.org/gmane.linux.gentoo.devel/81901</a><br>
> _______________________________________________<br>
> systemd-devel mailing list<br>
> <a href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</p>