# HG changeset patch
# User Roy Marples <roy@marples.name>
# Date 1600863246 -3600
# Node ID 8d0d468b61441352a5e1922405885070594b1178
# Parent  42eb9ab2e771642d1c288e344cae19913dc71bf6
if: Always warn about ignored interfaces.

diff -r 42eb9ab2e771 -r 8d0d468b6144 src/if.c
--- a/src/if.c	Wed Sep 23 09:59:47 2020 +0100
+++ b/src/if.c	Wed Sep 23 13:14:06 2020 +0100
@@ -399,7 +399,7 @@
 		break;
 	case ARPHRD_LOOPBACK:
 	case ARPHRD_PPP:
-		if (if_noconf) {
+		if (if_noconf && active) {
 			logdebugx("%s: ignoring due to interface type and"
 			    " no config",
 			    ifp->name);
@@ -538,8 +538,11 @@
 		    !if_hasconf(ctx, spec.devname));
 
 		/* Don't allow some reserved interface names unless explicit. */
-		if (if_noconf && if_ignore(ctx, spec.devname))
+		if (if_noconf && if_ignore(ctx, spec.devname)) {
+			logdebugx("%s: ignoring due to interface type and"
+			    " no config", spec.devname);
 			active = IF_INACTIVE;
+		}
 
 		ifp = calloc(1, sizeof(*ifp));
 		if (ifp == NULL) {
@@ -584,7 +587,7 @@
 			case IFT_LOOP: /* FALLTHROUGH */
 			case IFT_PPP:
 				/* Don't allow unless explicit */
-				if (if_noconf) {
+				if (if_noconf && active) {
 					logdebugx("%s: ignoring due to"
 					    " interface type and"
 					    " no config",