On Sun, Apr 13, 2014 at 2:11 PM, Umut Tezduyar Lindskog
<u...@tezduyar.com> wrote:
> On Sun, Apr 13, 2014 at 1:16 PM, Tom Gundersen <t...@jklm.no> wrote:
>> Hm, why? Are not error messages more useful?
>
> What is going to be the mapping for DHCP_EVENT_STOP?
>
> log_dhcp_client(client, "STOPPED: %s", strerror(-DHCP_EVENT_STOP));
>

Ah, now I get it. I guess we should special case this to something like:

if (r >= 0)
        log_dhcp_client(client, "STOPPED");
else
        log_dhcp_client(client, "STOPPED: %s", strerror(-r));

That way we get the error messages, but avoid things like "STOPPED:
Success." as well as the problem you pointed out.

What do you think?

-t
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to