ifupdown: Be consistent in error messages.
Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
This commit is contained in:
parent
8dc2295006
commit
a3d11ded43
1 changed files with 13 additions and 5 deletions
|
@ -103,15 +103,23 @@ skip_interface(struct lif_interface *iface, const char *ifname)
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (exec_opts.force)
|
|
||||||
return false;
|
|
||||||
|
|
||||||
if (iface->has_config_error)
|
if (iface->has_config_error)
|
||||||
{
|
{
|
||||||
fprintf(stderr, "%s: skipping interface %s due to config errors\n", argv0, ifname);
|
if (exec_opts.force)
|
||||||
return true;
|
{
|
||||||
|
fprintf(stderr, "%s: (de)configuring interface %s despite config errors\n", argv0, ifname);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
fprintf(stderr, "%s: skipping interface %s due to config errors\n", argv0, ifname);
|
||||||
|
return true;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (exec_opts.force)
|
||||||
|
return false;
|
||||||
|
|
||||||
if (up && iface->refcount > 0)
|
if (up && iface->refcount > 0)
|
||||||
{
|
{
|
||||||
if (exec_opts.verbose)
|
if (exec_opts.verbose)
|
||||||
|
|
Loading…
Reference in a new issue