Hi On Fri, Nov 28, 2014 at 3:54 PM, Martin Pitt <[email protected]> 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 [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
