Always use single quotes for printf format string

[config/dotfiles.git] / lib.sh

diff --git a/lib.sh b/lib.sh
index 70be68df0433531e27c5f7ab58e99bf8350a305e..6a619027db7e7ae1046a66be85f9b1102fc3ab3b 100644 (file)
--- a/lib.sh
+++ b/lib.sh
@@ -2,7 +2,7 @@
 #
 # Their setup.sh script sources this file.
 
-# Copyright (C) 2009-2014  Simon Ruderich
+# Copyright (C) 2009-2018  Simon Ruderich
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -68,7 +68,7 @@ cmd_i() {
 
 # Usage: sed_i ... <file>
 #
-# sed -i is not compatible due to different implementations. See cmd_i.
+# sed -i is not portable due to different implementations. See cmd_i.
 sed_i() {
     cmd_i sed "$@"
 }
@@ -111,7 +111,6 @@ simple_cpp() {
 #
 # - Debian (debian)
 # - Gentoo (gentoo)
-# - Mac OS X (darwin)
 # - Solaris/OpenSolaris (sun)
 # - FreeBSD (freebsd)
 #
@@ -121,8 +120,6 @@ os() {
         echo debian
     elif test -f /etc/gentoo-release; then
         echo gentoo
-    elif test x`uname` = xDarwin; then
-        echo darwin
     elif test x`uname` = xSunOS; then
         echo sun
     elif test x`uname` = xFreeBSD; then
@@ -154,19 +151,19 @@ link() {
     # Abort if the target file exists and is no symbolic link. Prevents
     # overwriting real files.
     if test -e "$target" && test ! -h "$target"; then
-        printf "link(): target '%s' exists already and is no symbolic link!\n" \
+        printf 'link(): target "%s" exists already and is no symbolic link!\n' \
                "$target" >&2
         exit 1
     fi
 
     # Make sure the source exists.
     if test ! -e "$source"; then
-        printf "link(): source '%s' doesn't exist!\n" "$source" >&2
+        printf 'link(): source "%s" does not exist!\n' "$source" >&2
         exit 1
     fi
 
     # Create the new symbolic link; remove the old one if necessary.
-    printf "link(): linking '%s' to '%s'\n" "$source" "$target"
+    printf 'link(): linking "%s" to "%s"\n' "$source" "$target"
     rm -f "$target"
     ln -s "$source" "$target"
 
@@ -196,14 +193,14 @@ generate() {
         file_tmp="$file.tmp"
     else
         # We only need this message if we generate a new file.
-        printf "%s: generating from '%s' (%s)\n" \
+        printf '%s: generating from "%s" (%s)\n' \
             "$file" "$file$extension" "$1"
 
         echo '###################################'  >"$file"
         echo '# WARNING! DO NOT EDIT THIS FILE! #' >>"$file"
         echo '###################################' >>"$file"
         echo >>"$file"
-        printf "# It was generated from '%s' on %s.\n" \
+        printf '# It was generated from "%s" on %s.\n' \
             "$file$extension" "`date`" >>"$file"
         echo >>"$file"