On Wed, Feb 13, 2013 at 04:27:59PM +0100, Lennart Poettering wrote:
> On Wed, 13.02.13 16:16, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote:
>
> >
> > On Wed, Feb 13, 2013 at 03:09:36PM +0100, har...@redhat.com wrote:
> > > k = inotify_add_watch(fd, "/run/systemd/seats/",
> > > IN_MOVED_TO|IN_DELETE);
> > > if (k < 0) {
> > > + log_error("Failed to watch /run/systemd/seats/:
> > > %m");
> > > close_nointr_nofail(fd);
> > Applied. I changed /Failed to watch/ to /Failed to add watch on/. I think it
> > makes the message slightly clearer.
>
> Sorry, I had to revert this commit. We should never log anything from
> library calls. This would be as if open() or write() of the glibc would
> log something to stderr as side-effect.
Fair enough.
> Logging is only only OK in "main programs" but never from library code,
> and this code in particular is compiled into an .so for normal
> consumers, so it should not log, but always only return error codes.
Right.
Zbyszek
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
