Hi On Fri, Nov 28, 2014 at 3:54 PM, Martin Pitt <martin.p...@ubuntu.com> wrote: > Hey David, > > David Herrmann [2014-11-28 15:49 +0100]: >> Why not fix all those other occurrences with one fix by changing >> hostnamectl_main() the last line from: >> return r < 0 ? EXIT_FAILURE : r; >> to >> return r < 0 ? EXIT_FAILURE : 0; >> >> Usually, >=0 means success, <0 failure, in systemd. We should not >> return !=0 from main() if the return value wasn't negative. > > Yeah, that's even more robust and guards against similar errors in the > future. Updated patch attached.
Looks good, pushed! Thanks David _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel