bin/qemush

@@ -1,19 +1,19 @@
 #!/usr/bin/env bash
-# version=0.8.0
+# version=0.7.0
 
 # Re-exec the script as qemu via sudo (only if needed)
 [ "$(whoami)" != qemu ] && exec sudo -E -H -u qemu -- "$0" "$@"
 
 # Environment
 PATH="${HOME}/launchers:${HOME}/bin:${PATH}"
+EDITOR="${EDITOR:-nvim}"
+export QEMUSH_NAME
 
 # Aliases
-alias ls='ls --color=auto'
+ls='ls --color=auto'
-alias exec='exec '
-shopt -s expand_aliases
 
 # Set a restrictive umask to make sure qemu user files are private
-umask 7027
+umask 027
 
 # Function to print a colored error
 perror() {
@@ -22,6 +22,7 @@ perror() {
 
 # Function to show the usage
 public_help() {
+    local name
     name=$(basename "$0")
 
     exec cat << EOF
@@ -45,21 +46,19 @@ EOF
 
 # Function to throw an invalid usage error (skill issue)
 error_usage() {
-    perror "invalid usage"
+    perror "Invalid usage"
     >&2 public_help
-
     return 1
 }
 
 # Function to start a virtual machine
 public_start() {
-    export QEMUSH_NAME="$1"
+    QEMUSH_NAME="$1"
 
     set -- "$@" \
         -name "$QEMUSH_NAME" \
         -monitor "unix:$(pathof socket),server,nowait" \
         -daemonize
-
     if ! "$@"; then
         perror "error launching virtual machine \"${QEMUSH_NAME}\""
         return 2
@@ -69,7 +68,8 @@ public_start() {
 # Attach to a running virtual machine output, the latest opened if no
 # argument is provided
 public_attach() {
-    export QEMUSH_NAME="$1"
+    QEMUSH_NAME="$1"
+    shift
 
     exec socat -,rawer,escape=15 "UNIX-CONNECT:$(pathof socket)"
 }
@@ -77,31 +77,30 @@ public_attach() {
 # List running virtual machines
 public_running() {
     cd || return
-
     echo "Running machines:"
-    exec ls -t sockets/monitors
+    set -- $ls -t sockets/monitors "$@"
+    exec "$@"
 }
 
 # List available virtual machines entrypoints
 public_ls() {
     cd || return
-
     echo "Available machines:"
-    exec ls launchers
+    set -- $ls launchers "$@"
+    exec "$@"
 }
 
 # Create a copy-on-write disk for a virtual machine
 public_diskadd() {
-    export QEMUSH_NAME="$1"
+    QEMUSH_NAME="$1"
-
+    shift
-    exec qemu-img create -f qcow2 "$(pathof disk)" "$2"
+    exec qemu-img create -f qcow2 "$(pathof disk)" "$1"
 }
 
 # Delete a disk
 public_diskrm() {
     for disk in "$@"; do
-        export QEMUSH_NAME="$disk"
+        QEMUSH_NAME="$disk"
-
         rm -vi -- "$(pathof disk)"
     done
 }
@@ -109,66 +108,55 @@ public_diskrm() {
 # List available disks
 public_diskls() {
     cd || return
-
     echo "Available disks:"
-    exec ls disks
+    set -- $ls disks "$@"
+    exec "$@"
 }
 
 # Edit a virtual machine entrypoint with a text editor
 public_edit() {
-    file="launchers/${1}"
+    cd || return
-    # I don't even know why shellcheck gives me this warning
+    local file="launchers/${1}"
-    # shellcheck disable=2209
-    [ -z "$EDITOR" ] && EDITOR=vi
 
-    set -e
+    "$EDITOR" "$file"
-    cd
+    [ -f "$file" ] && exec chmod u+x "$file"
-    touch -- "$file"
-    chmod u+x -- "$file"
-    exec "$EDITOR" -- "$file"
 }
 
 # Delete a virtual machine entrypoint
 public_rm() {
     cd ~/launchers || return
-
     exec rm -vi -- "$@"
 }
 
 # Invoke bash as qemu user in its home directory
 public_shell() {
     cd || return
-
+    set -- bash -i "$@"
-    exec bash -i
+    exec "$@"
 }
 
 # Output the content of an entrypoint, with coloration if on a virtual
 # terminal
 public_cat() {
     cd ~/launchers || return
-
+    cat -- "$@"
-    exec cat -- "$@"
 }
 
 # Copy a file in entrypoints folder
 public_add() {
-    if [ -z "$1" ]; then
-        perror "specify the path of a launching script you want to add"
-        return 1
-    fi
-
-    if [ -n "$2" ]; then
-        destination="$2"
-    else
-        destination=$(basename "$1")
-    fi
-    destination="${HOME}/launchers/${destination}"
-
     trap return EXIT
     set -e
 
-    cp -vi -- "$1" "$destination"
+    local name
-    chmod 0740 -- "$destination"
+    if [ -n "$2" ]; then
+        name="$2"
+    else
+        name=$(basename "$1")
+    fi
+    name="${HOME}/launchers/${name}"
+
+    cp -vi -- "$1" "$name"
+    chmod 740 "$name"
 
     set +e
     trap - EXIT
@@ -181,8 +169,7 @@ public_do() {
 
 # Expose SPICE via TCP
 public_spice() {
-    export QEMUSH_NAME="$1"
+    QEMUSH_NAME="$1"
-
     if [ -n "$2" ]; then
         port=$2
     else
@@ -193,22 +180,15 @@ public_spice() {
         socat TCP-LISTEN:"${port},reuseaddr,fork" UNIX-CLIENT:"$(pathof spice)"
 }
 
-function_exists() {
+# Retrieve user requested function
-    declare -F \
+function="$1"
-        | cut -d \  -f 3- \
+shift
-        | grep '^public_' \
-        | sed 's/^public_//' \
-        | grep -q "^${1}\$"
-}
 
 # Defauts to `active` if no function is supplied; else checks for a public
 # function named after the argument; else fails
-if [ -z "$1" ]; then
+if [ -z "$function" ]; then
     public_running
-elif function_exists "$1"; then
+elif declare -F | cut -d \  -f 3- | grep '^public_' | sed 's/^public_//' | grep -q "^${function}$"; then
-    function=$1
-    shift
-
     "public_${function}" "$@"
 else
     error_usage