Update README

.gitignore

@@ -1 +1,2 @@
 variables.conf
+.disabled

README.md

@@ -11,6 +11,14 @@ Version 2 implementation works roughly as follows:
   * if changes are detected, it runs install script (if present).
 
 ### Files
-* `conf-sync.sh`
+* `conf-sync.sh` is the main script, which updates the files.
+* `variables.conf.template` contains examples variables definitions, and should be copied locally to `variables.conf`
+  (with any required modifications).
+* `systemd-templates/` contains Systemd units templates for running the script unattended.
 #### Installation
-* `install.sh`
+* `install.sh` script copies Systemd templates to their destination, and enables conf-sync timer.
+
+### TODO
+* Add install support for OpenBSD
+* Replace desktop/server timer choice in one line in install script
+* Add `$sudo` in install script

conf-sync.sh

@@ -1,13 +1,19 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
-source "$(dirname "$0")/variables.conf"
+run_directory=$(dirname $(readlink -f "$0"))
+source "$run_directory/variables.conf"
 
-for dir in $sync_directory; do
-  git -C $dir rev-parse 2>/dev/null
-  if [ "$?" -ne 0 ]; then # ie. this is not a git repo
+for dir in $sync_directory/*; do
+  git -C $dir rev-parse 2>/dev/null >/dev/null
+  if [ "$?" -ne 0 ]; then
+    echo "$dir: not a Git repo"
     continue
   fi
-  git -C $dir fetch
+  if [ -f $dir/.disabled ]; then
+    echo "$dir update is disabled."
+    continue
+  fi
+  git -C $dir fetch 2>/dev/null >/dev/null
   UPSTREAM=${1:-'@{u}'}
   LOCAL=$(git -C $dir rev-parse @)
   REMOTE=$(git -C $dir rev-parse "$UPSTREAM")
@@ -16,9 +22,9 @@ for dir in $sync_directory; do
       echo "$dir: up-to-date"
   elif [ $LOCAL = $BASE ]; then
       echo "pulling $dir"
-      git -C pull
-      if []; then
-        echo "Running $dir\'s install script..."
+      git -C $dir pull 2>/dev/null >/dev/null
+      if [ -e $dir/install.sh ]; then
+        echo "Running $dir's install script..."
         $dir/install.sh
       fi
   elif [ $REMOTE = $BASE ]; then

install.sh

@@ -1,23 +1,45 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
-source "$(dirname "$0")/variables.conf"
+run_directory=$(dirname $(readlink -f "$0"))
+user=$(whoami)
 
-# TODO handle non-Systemd systems
+source "$run_directory/variables.conf"
 
-echo "Copying Systemd units to system directory..."
-sudo cp systemd-templates/conf-sync.service /etc/systemd/system/
-sudo sed -i -e "s/PH_USER/$(whoami)/g" /etc/systemd/system/conf-sync.service
-sudo sed -i -e "s:PH_DIRECTORY:$(pwd):g" /etc/systemd/system/conf-sync.service
-if $client_type == "server"; then # TODO not finished
-  sudo cp systemd-templates/dns-update.timer /etc/systemd/system/conf-sync.timer
-else
-  sudo cp systemd-templates/dns-update.timer /etc/systemd/system/conf-sync.timer
+if [ -f $run_directory/.disabled ]; then
+  echo "Installation is disabled. Exiting..."
+  exit 0
 fi
 
-echo "Reloading Systemd..."
-sudo systemctl daemon-reload
-echo "Enabling timer (not starting it, either do it manually or reboot)..."
-sudo systemctl enable conf-sync.timer
+if [[ $user != 'root' ]]; then
+  sudo="sudo"
+else
+  sudo=""
+fi
+
+# NOTE: the handling of init/service manager comes from this discussion:
+# https://unix.stackexchange.com/questions/18209/detect-init-system-using-the-shell
+
+if [ -e /run/systemd/system ]; then # service manager is Systemd
+  echo "Copying Systemd units to system directory..."
+  $sudo cp $run_directory/systemd-templates/conf-sync.service /etc/systemd/system/
+  $sudo sed -i -e "s/PH_USER/$(whoami)/g" /etc/systemd/system/conf-sync.service
+  $sudo sed -i -e "s:PH_DIRECTORY:$run_directory:g" /etc/systemd/system/conf-sync.service
+  if [ "$client_type" = "server" ]; then # TODO could be replaced in one line: cp conf-sync-$client_type
+    $sudo cp $run_directory/systemd-templates/conf-sync-server.timer /etc/systemd/system/conf-sync.timer
+  else
+    $sudo cp $run_directory/systemd-templates/conf-sync-desktop.timer /etc/systemd/system/conf-sync.timer
+  fi
+  echo "Reloading Systemd..."
+  $sudo systemctl daemon-reload
+  echo "Enabling timer (not starting it, either do it manually or reboot)..."
+  $sudo systemctl enable conf-sync.timer
+elif [ "$(uname)" = "OpenBSD" ]; then
+  echo "ERROR: OpenBSD is not yet supported, but this is planned."
+  exit 0
+else
+  echo "ERROR: could not determine a supported system. Exiting."
+  exit 1
+fi
 
 echo "Finished install. Exiting..."
 exit 0

variables.conf.template

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 # Copy this file as variables.conf, with the correct values