[arch-projects] [netctl][PATCH 1/3] Minor code streamlining
Jouke Witteveen
j.witteveen at gmail.com
Fri Mar 3 18:23:54 UTC 2017
---
src/ifplugd.action | 2 +-
src/lib/connections/mobile_ppp | 4 ++--
src/lib/connections/pppoe | 4 ++--
src/lib/dhcp/dhclient | 4 ++--
src/lib/dhcp/dhcpcd | 4 ++--
src/lib/wpa | 25 +++++++++++++------------
src/netctl-auto | 9 ++++++---
src/netctl.in | 2 +-
8 files changed, 29 insertions(+), 25 deletions(-)
diff --git a/src/ifplugd.action b/src/ifplugd.action
index 42d1c5a..a1c8dee 100755
--- a/src/ifplugd.action
+++ b/src/ifplugd.action
@@ -4,7 +4,7 @@
. /usr/lib/network/globals
-PROFILE_FILE="$STATE_DIR/ifplugd_$1.profile"
+PROFILE_FILE="$STATE_DIR/ifplugd-$1.profile"
case "$2" in
up)
diff --git a/src/lib/connections/mobile_ppp b/src/lib/connections/mobile_ppp
index 0890373..11cb41e 100644
--- a/src/lib/connections/mobile_ppp
+++ b/src/lib/connections/mobile_ppp
@@ -11,7 +11,7 @@ quote_word() {
}
mobile_ppp_up() {
- local options_dir="$STATE_DIR/mobile_ppp.$Interface.$Profile"
+ local options_dir="$STATE_DIR/mobile_ppp-$Interface-$Profile"
network_ready
@@ -83,7 +83,7 @@ EOF
mobile_ppp_down() {
local options_dir pidfile pid
- options_dir="$STATE_DIR/mobile_ppp.$Interface.$Profile"
+ options_dir="$STATE_DIR/mobile_ppp-$Interface-$Profile"
pidfile="/var/run/ppp-$Profile.pid"
if [[ -r $pidfile ]]; then
diff --git a/src/lib/connections/pppoe b/src/lib/connections/pppoe
index 15e4db5..96cda80 100644
--- a/src/lib/connections/pppoe
+++ b/src/lib/connections/pppoe
@@ -8,7 +8,7 @@ quote_word() {
}
pppoe_up() {
- local options="$STATE_DIR/pppoe.$Interface.$Profile/options"
+ local options="$STATE_DIR/pppoe-$Interface-$Profile/options"
if ! is_interface "$Interface"; then
report_error "Interface '$Interface' does not exist"
@@ -61,7 +61,7 @@ EOF
pppoe_down() {
local options pidfile pid
- options="$STATE_DIR/pppoe.$Interface.$Profile/options"
+ options="$STATE_DIR/pppoe-$Interface-$Profile/options"
pidfile="/var/run/ppp-$Profile.pid"
if [[ -r $pidfile ]]; then
diff --git a/src/lib/dhcp/dhclient b/src/lib/dhcp/dhclient
index 87e9a92..397adda 100644
--- a/src/lib/dhcp/dhclient
+++ b/src/lib/dhcp/dhclient
@@ -8,7 +8,7 @@ dhclient_start() {
*) return 1;;
esac
[[ $2 == "noaddr" ]] && options+=" -S"
- if ! do_debug do_readable dhclient -$1 -q -e "TIMEOUT=${TimeoutDHCP:-30}" -pf "$pidfile" $options "$Interface"; then
+ if ! do_debug do_readable dhclient "-$1" -q -e "TIMEOUT=${TimeoutDHCP:-30}" -pf "$pidfile" $options "$Interface"; then
report_error "DHCP IPv$1 lease attempt failed on interface '$Interface'"
return 1
fi
@@ -18,7 +18,7 @@ dhclient_stop() {
local stop="-x" pidfile="/run/dhclient-$Interface-$1.pid"
if [[ -f $pidfile ]]; then
is_yes "${DHCPReleaseOnStop:-no}" && stop="-r"
- do_debug dhclient -$1 -q $stop "$Interface" -pf "$pidfile" > /dev/null
+ do_debug dhclient "-$1" -q $stop "$Interface" -pf "$pidfile" > /dev/null
fi
}
diff --git a/src/lib/dhcp/dhcpcd b/src/lib/dhcp/dhcpcd
index c88047f..ef1245d 100644
--- a/src/lib/dhcp/dhcpcd
+++ b/src/lib/dhcp/dhcpcd
@@ -13,7 +13,7 @@ dhcpcd_start() {
fi
# If using own dns, tell dhcpcd to NOT replace resolv.conf
[[ $DNS ]] && options+=" -C resolv.conf"
- if ! do_debug do_readable dhcpcd -$1 -q -t "${TimeoutDHCP:-30}" $options "$Interface"; then
+ if ! do_debug do_readable dhcpcd "-$1" -q -t "${TimeoutDHCP:-30}" $options "$Interface"; then
report_error "DHCP IPv$1 lease attempt failed on interface '$Interface'"
return 1
fi
@@ -23,7 +23,7 @@ dhcpcd_stop() {
local stop="-x"
if [[ -f "/run/dhcpcd-$Interface-$1.pid" ]]; then
is_yes "${DHCPReleaseOnStop:-no}" && stop="-k"
- do_debug dhcpcd -$1 -q $stop "$Interface" > /dev/null
+ do_debug dhcpcd "-$1" -q $stop "$Interface" > /dev/null
fi
}
diff --git a/src/lib/wpa b/src/lib/wpa
index b61f93b..e7d873d 100644
--- a/src/lib/wpa
+++ b/src/lib/wpa
@@ -64,29 +64,29 @@ wpa_wait_while_state() {
# $2: interface driver(s)
# $3: configuration file
wpa_start() {
- local interface=$1 driver=$2 configuration=$3
- local pidfile="/run/wpa_supplicant_$interface.pid"
+ local interface=$1 driver=$2 config_file=$3
+ local pidfile="/run/wpa_supplicant-$interface.pid"
if ! type wpa_supplicant &> /dev/null; then
report_error "You need to install 'wpa_supplicant'"
return 127
fi
- if [[ $configuration ]]; then
- configuration="-c$configuration"
+ if [[ $config_file ]]; then
+ config_file="-c$config_file"
else
WPA_CTRL_DIR="/run/wpa_supplicant"
- configuration="-C$WPA_CTRL_DIR"
+ config_file="-C$WPA_CTRL_DIR"
fi
do_debug wpa_supplicant -q -B -P "$pidfile" -i "$interface" -D "$driver" \
- "$configuration" $WPAOptions >&2
+ "$config_file" $WPAOptions >&2
# Wait up to one second for the pid file to appear
if ! timeout_wait 1 '[[ -f $pidfile ]]'; then
# Remove the configuration file if it was generated
- configuration="$STATE_DIR/wpa_supplicant_$interface.conf"
- [[ -f $configuration && -O $configuration ]] && rm -f "$configuration"
+ config_file="$STATE_DIR/wpa_supplicant-$interface.conf"
+ [[ -f $config_file && -O $config_file ]] && rm -f "$config_file"
return 1
fi
}
@@ -94,7 +94,8 @@ wpa_start() {
## Stop an instance of the wpa supplicant
# $1: interface name
wpa_stop() {
- local interface=$1 config_file="$STATE_DIR/wpa_supplicant_$1.conf"
+ local interface=$1 config_file="$STATE_DIR/wpa_supplicant-$1.conf"
+ local pidfile="/run/wpa_supplicant-$interface.pid"
# We need this as long as wpa_cli has a different default than netctl
if [[ -z $WPA_CTRL_DIR && -z $WPAConfigFile ]]; then
WPA_CTRL_DIR="/run/wpa_supplicant"
@@ -107,8 +108,8 @@ wpa_stop() {
[[ -f $config_file && -O $config_file ]] && rm -f "$config_file"
# Wait up to one second for the pid file to be removed
- if ! timeout_wait 1 '[[ ! -f "/run/wpa_supplicant_$interface.pid" ]]'; then
- kill "$(< "/run/wpa_supplicant_$interface.pid")" &> /dev/null
+ if ! timeout_wait 1 '[[ ! -f $pidfile ]]'; then
+ kill "$(< "$pidfile")" &> /dev/null
fi
}
@@ -192,7 +193,7 @@ wpa_unquote() {
## Create a configuration file for wpa_supplicant without any network blocks
# $1: interface name
wpa_make_config_file() {
- local config_file="$STATE_DIR/wpa_supplicant_$1.conf"
+ local config_file="$STATE_DIR/wpa_supplicant-$1.conf"
if [[ -e $config_file ]]; then
report_debug "The anticipated filename '$config_file' is not available"
diff --git a/src/netctl-auto b/src/netctl-auto
index b25d258..f78efe3 100755
--- a/src/netctl-auto
+++ b/src/netctl-auto
@@ -8,6 +8,7 @@
: ${ACTIOND:=wpa_actiond -p /run/wpa_supplicant}
: ${ACTION_SCRIPT:=$SUBR_DIR/auto.action}
+
usage() {
cat << END
Usage: netctl-auto {COMMAND} ...
@@ -81,7 +82,7 @@ profile_enable_disable() {
local action="$1" profile="$2"
local id interfaces wpa_cmd
- if [[ -n "$profile" ]]; then
+ if [[ $profile ]]; then
read -r interfaces id < <(get_wpa_network_id "$profile") || return 1
else
interfaces=$(list_netctl_auto_interfaces)
@@ -197,7 +198,7 @@ list() {
# $1: interface
start() {
local interface="$1"
- local pidfile="$STATE_DIR/wpa_actiond_$1.pid"
+ local pidfile="$STATE_DIR/wpa_actiond-$interface.pid"
if wpa_is_active "$interface"; then
exit_error "The interface ($interface) is already in use"
@@ -247,7 +248,7 @@ start() {
# $1: interface
stop() {
local interface="$1"
- local pidfile="$STATE_DIR/wpa_actiond_$1.pid"
+ local pidfile="$STATE_DIR/wpa_actiond-$interface.pid"
[[ -e "$pidfile" ]] && kill "$(< "$pidfile")"
if [[ -x "$PROFILE_DIR/interfaces/$interface" ]]; then
@@ -259,6 +260,7 @@ stop() {
return 0
}
+
case $# in
1)
case $1 in
@@ -292,4 +294,5 @@ case $# in
exit_error "$(usage)";;
esac
+
# vim: ft=sh ts=4 et sw=4:
diff --git a/src/netctl.in b/src/netctl.in
index ea1c9aa..37c095c 100644
--- a/src/netctl.in
+++ b/src/netctl.in
@@ -97,7 +97,7 @@ unit_enable() {
do_readable touch "$unit"
echo ".include @systemdsystemunitdir@/netctl at .service" > "$unit"
echo -e "\n[Unit]" >> "$unit"
- [[ -n $Description ]] && echo "Description=$Description" >> "$unit"
+ [[ $Description ]] && echo "Description=$Description" >> "$unit"
declare -p BindsToInterfaces &> /dev/null || BindsToInterfaces=$Interface
if (( ${#BindsToInterfaces[@]} )); then
: ${InterfaceRoot=sys/subsystem/net/devices/}
--
2.12.0
More information about the arch-projects
mailing list