From 0129d4078b03e9f0919ad24368c4e88189b9702c Mon Sep 17 00:00:00 2001
From: Michael Prokop <mika@grml.org>
Date: Fri, 25 May 2018 12:06:52 +0200
Subject: [PATCH] netcardconfig: default VLAN option to no and ensure 8021q
 module is loaded

Configuring VLAN is optional and usually not required, so default
to no.

Without the 8021q module being loaded it spills a warning
message, even though it works fine. This might be confusing
for the user though, so let's loaded ahead.
---
 sbin/netcardconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sbin/netcardconfig b/sbin/netcardconfig
index e3d0d11..3a80366 100755
--- a/sbin/netcardconfig
+++ b/sbin/netcardconfig
@@ -420,10 +420,11 @@ configiface() {
   fi
 
   # Configure VLAN on this interface?
-  if $DIALOG --yesno "$MESSAGE16" 8 45; then
+  if $DIALOG --defaultno --yesno "$MESSAGE16" 8 45; then
     $DIALOG --inputbox "$MESSAGE17 $DV" 10 45 2>"$TMP" || bailout 1
     read -r VLAN <"$TMP" ; rm -f "$TMP"
     if [ -n "$VLAN" ]; then
+      modprobe 8021q # avoid warning that VLAN support isn't present yet
       PDV=$DV
       DV="vlan$VLAN"
     fi
-- 
2.1.4