Merge branch 'develop' into main

* Use regex to extract compiler arguments.
* Added --compiler.

.drone.yml

@@ -1,176 +1,147 @@
-pipeline:
-  gcc5:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=g++-5
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - apt-get update -q
-      - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
-      - echo "deb http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial main" >> /etc/apt/sources.list.d/ubuntu-toolchain-r.list
-      - apt-get install -qy gnupg
-      - gpg --keyserver hkp://keyserver.ubuntu.com --recv-keys 0x60c317803a41ba51845e371a1e9377a2ba9ef27f
-      - gpg --armor --export 0x60c317803a41ba51845e371a1e9377a2ba9ef27f | apt-key add -
-      - apt-get update -q
-      - apt-get install -qy -t xenial g++-5
-      - apt-get install -qy cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+kind: pipeline
+name: x86_64 build
 
-  gcc6:
-    image: debian:stretch-slim
-    pull: true
-    environment:
-      - LANG=C.utf8
-      - CXX=g++-6
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - apt-get update -q
-      - apt-get install -qy g++-6 cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - apt-get install -qy dpkg-dev d-shlibs rpm file wget
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-      - make install DESTDIR=install
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+volumes:
+- name: debian-package-cache
+  host:
+    path: /var/cache/debian-package-cache
 
-  gcc7:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=g++-7
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - apt-get update -q
-      - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
-      - echo "deb http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial main" >> /etc/apt/sources.list.d/ubuntu-toolchain-r.list
-      - apt-get install -qy gnupg
-      - gpg --keyserver hkp://keyserver.ubuntu.com --recv-keys 0x60c317803a41ba51845e371a1e9377a2ba9ef27f
-      - gpg --armor --export 0x60c317803a41ba51845e371a1e9377a2ba9ef27f | apt-key add -
-      - apt-get update -q
-      - apt-get install -qy -t xenial g++-7
-      - apt-get install -qy cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+trigger:
+  event:
+    exclude:
+    - tag
 
-  gcc8:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=g++-8
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - apt-get update -q
-      - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
-      - echo "deb http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial main" >> /etc/apt/sources.list.d/ubuntu-toolchain-r.list
-      - apt-get install -qy gnupg
-      - gpg --keyserver hkp://keyserver.ubuntu.com --recv-keys 0x60c317803a41ba51845e371a1e9377a2ba9ef27f
-      - gpg --armor --export 0x60c317803a41ba51845e371a1e9377a2ba9ef27f | apt-key add -
-      - apt-get update -q
-      - apt-get install -qy -t xenial g++-8
-      - apt-get install -qy cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+steps:
+- name: submodules
+  image: docker:git
+  commands:
+    - git submodule init
+    - git submodule update --recursive
 
-  clang3:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=clang++
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - apt-get update -q
-      - apt-get install -qy clang cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+- name: gcc6
+  image: debian:stretch-slim
+  pull: always
+  environment:
+    LANG: C.utf8
+    CXX: g++-6
+    CXXFLAGS: -pipe -O2
+  commands:
+  - rm /etc/apt/apt.conf.d/docker-clean
+  - apt-get update -q
+  - apt-get install -qy g++-6 cmake pkg-config
+  - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
+  - apt-get install -qy dpkg-dev d-shlibs rpm file wget
+  - rm -rf build && mkdir -p build && cd build
+  - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
+  - make VERBOSE=1
+  - make install DESTDIR=install
+  volumes:
+  - name: debian-package-cache
+    path: /var/cache/apt/archives
 
-  clang5:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=clang++-5.0
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
-      - echo "deb http://deb.debian.org/debian stretch-backports main" >> /etc/apt/sources.list.d/stretch.list
-      - apt-get update -q
-      - apt-get install -qy -t stretch-backports clang-5.0
-      - apt-get install -qy cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
 
-  clang6:
-    image: debian:stretch-slim
-    pull: true
-    when:
-      event: [push, pull_request]
-    environment:
-      - LANG=C.utf8
-      - CXX=clang++-6.0
-      - CXXFLAGS=-pipe -O2
-    commands:
-      - rm /etc/apt/apt.conf.d/docker-clean
-      - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
-      - echo "deb http://deb.debian.org/debian stretch-backports main" >> /etc/apt/sources.list.d/stretch.list
-      - apt-get update -q
-      - apt-get install -qy -t stretch-backports clang-6.0
-      - apt-get install -qy cmake pkg-config
-      - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
-      - rm -rf build && mkdir -p build && cd build
-      - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
-      - make VERBOSE=1
-    volumes:
-      - /var/cache/debian-package-cache:/var/cache/apt/archives
+- name: gcc5
+  image: debian:stretch-slim
+  pull: always
+  environment:
+    LANG: C.utf8
+    CXX: g++-5
+    CXXFLAGS: -pipe -O2
+  commands:
+  - rm /etc/apt/apt.conf.d/docker-clean
+  - apt-get update -q
+  - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
+  - echo "deb http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial main" >> /etc/apt/sources.list.d/ubuntu-toolchain-r.list
+  - apt-get install -qy gnupg
+  - gpg --keyserver hkp://keyserver.ubuntu.com --recv-keys 0x60c317803a41ba51845e371a1e9377a2ba9ef27f
+  - gpg --armor --export 0x60c317803a41ba51845e371a1e9377a2ba9ef27f | apt-key add -
+  - apt-get update -q
+  - apt-get install -qy -t xenial g++-5
+  - apt-get install -qy cmake pkg-config
+  - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
+  - rm -rf build && mkdir -p build && cd build
+  - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
+  - make VERBOSE=1
+  volumes:
+  - name: debian-package-cache
+    path: /var/cache/apt/archives
 
-  notify:
-    image: drillster/drone-email
-    pull: true
+- name: gcc9
+  image: debian:stretch-slim
+  pull: always
+  environment:
+    LANG: C.utf8
+    CXX: g++-9
+    CXXFLAGS: -pipe -O2
+  commands:
+  - rm /etc/apt/apt.conf.d/docker-clean
+  - apt-get update -q
+  - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
+  - echo "deb http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial main" >> /etc/apt/sources.list.d/ubuntu-toolchain-r.list
+  - apt-get install -qy gnupg
+  - gpg --keyserver hkp://keyserver.ubuntu.com --recv-keys 0x60c317803a41ba51845e371a1e9377a2ba9ef27f
+  - gpg --armor --export 0x60c317803a41ba51845e371a1e9377a2ba9ef27f | apt-key add -
+  - apt-get update -q
+  - apt-get install -qy -t xenial g++-9
+  - apt-get install -qy cmake pkg-config
+  - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
+  - rm -rf build && mkdir -p build && cd build
+  - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
+  - make VERBOSE=1
+  volumes:
+  - name: debian-package-cache
+    path: /var/cache/apt/archives
+
+- name: clang3
+  image: debian:stretch-slim
+  pull: always
+  environment:
+    LANG: C.utf8
+    CXX: clang++
+    CXXFLAGS: -pipe -O2
+  commands:
+  - rm /etc/apt/apt.conf.d/docker-clean
+  - apt-get update -q
+  - apt-get install -qy clang cmake pkg-config
+  - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
+  - rm -rf build && mkdir -p build && cd build
+  - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
+  - make VERBOSE=1
+  volumes:
+  - name: debian-package-cache
+    path: /var/cache/apt/archives
+
+- name: clang6
+  image: debian:stretch-slim
+  pull: always
+  environment:
+    LANG: C.utf8
+    CXX: clang++-6.0
+    CXXFLAGS: -pipe -O2
+  commands:
+  - rm /etc/apt/apt.conf.d/docker-clean
+  - echo "APT::Default-Release \"stretch\";" >> /etc/apt/apt.conf.d/00default_release
+  - echo "deb http://deb.debian.org/debian stretch-backports main" >> /etc/apt/sources.list.d/stretch.list
+  - apt-get update -q
+  - apt-get install -qy -t stretch-backports clang-6.0
+  - apt-get install -qy cmake pkg-config
+  - apt-get install -qy libconfig++-dev libxdg-basedir-dev asciidoc
+  - rm -rf build && mkdir -p build && cd build
+  - cmake -DCMAKE_INSTALL_PREFIX=/usr ..
+  - make VERBOSE=1
+  volumes:
+  - name: debian-package-cache
+    path: /var/cache/apt/archives
+
+- name: notify
+  image: drillster/drone-email
+  pull: always
+  settings:
     host: cryptoparty-celle.de
-    secrets: [ email_username, email_password ]
     from: drone@tzend.de
-    when:
-      status: [ changed, failure ]
+    username:
+      from_secret: email_username
+    password:
+      from_secret: email_password
+  when:
+    status: [ changed, failure ]

CMakeLists.txt

@@ -1,50 +1,47 @@
-cmake_minimum_required (VERSION 3.2)
-project(compilescript
-        VERSION 0.4.0
-        LANGUAGES CXX
-)
+# Support version 3.6 and above, but use policy settings up to 3.14.
+# 3.6 is needed because of IMPORTED_TARGET in pkg_check_modules().
+cmake_minimum_required(VERSION 3.6...3.14)
+# Ranges are supported from 3.12, set policy to current for < 3.12.
+if(${CMAKE_VERSION} VERSION_LESS 3.12)
+  cmake_policy(VERSION ${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION})
+endif()
 
-include(GNUInstallDirs)
-find_package(PkgConfig REQUIRED)
-pkg_check_modules(LIBXDG_BASEDIR REQUIRED libxdg-basedir)
-pkg_check_modules(LIBCONFIG REQUIRED libconfig++)
+project(compilescript
+  VERSION 0.4.1
+  LANGUAGES CXX)
+# DESCRIPTION was introduced in version 3.9.
+if(NOT (${CMAKE_VERSION} VERSION_LESS 3.9))
+  set(PROJECT_DESCRIPTION
+    "Execute source files from compiled languages as scripts.")
+endif()
+
+option(WITH_MAN "Compile and install manpage." YES)
 
 set(CMAKE_CXX_STANDARD 14)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
-set(CMAKE_CXX_FLAGS_DEBUG
-    "${CMAKE_CXX_FLAGS_DEBUG} -Wall -Wextra -Wpedantic -ftrapv -fsanitize=undefined -g -Og -fno-omit-frame-pointer")
-
-include_directories(${PROJECT_BINARY_DIR})
-include_directories(${LIBXDG_BASEDIR_INCLUDE_DIRS})
-include_directories(${LIBCONFIG_INCLUDE_DIRS})
-
-link_directories(${LIBXDG_BASEDIR_LIBRARY_DIRS})
-link_directories(${LIBCONFIG_LIBRARY_DIRS})
-
-# Write version in header
-configure_file(
-    "${PROJECT_SOURCE_DIR}/src/version.hpp.in"
-    "${PROJECT_BINARY_DIR}/version.hpp"
-)
-
-file(GLOB sources src/*.cpp)
-add_executable(${CMAKE_PROJECT_NAME} "${sources}")
-target_link_libraries(${CMAKE_PROJECT_NAME}
-                      "${LIBXDG_BASEDIR_LDFLAGS} ${LIBCONFIG_LDFLAGS}"
-                      "-lstdc++fs")
-install(TARGETS ${CMAKE_PROJECT_NAME} DESTINATION ${CMAKE_INSTALL_BINDIR})
-
-set(WITH_MAN "YES" CACHE STRING "WITH_MAN defaults to \"YES\"")
-if (WITH_MAN)
-    add_custom_command(OUTPUT "${PROJECT_BINARY_DIR}/${CMAKE_PROJECT_NAME}.1"
-                       WORKING_DIRECTORY "${PROJECT_BINARY_DIR}"
-                       DEPENDS "${CMAKE_SOURCE_DIR}/${CMAKE_PROJECT_NAME}.1.adoc"
-                       COMMAND ${CMAKE_SOURCE_DIR}/build_manpage.sh
-                       ARGS ${PROJECT_VERSION})
-    add_custom_target(run ALL
-                      DEPENDS "${PROJECT_BINARY_DIR}/${CMAKE_PROJECT_NAME}.1")
-    install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_PROJECT_NAME}.1
-            DESTINATION ${CMAKE_INSTALL_MANDIR}/man1)
+set(DEBUG_CXXFLAGS
+  "-Wall"
+  "-Wextra"
+  "-Wpedantic"
+  "-ftrapv"
+  "-fsanitize=undefined"
+  "-g"
+  "-Og"
+  "-fno-omit-frame-pointer")
+set(DEBUG_LDFLAGS
+  "-fsanitize=undefined")
+add_compile_options("$<$<CONFIG:Debug>:${DEBUG_CXXFLAGS}>")
+# add_link_options was introduced in version 3.13.
+if(${CMAKE_VERSION} VERSION_LESS 3.13)
+  set(CMAKE_SHARED_LINKER_FLAGS_DEBUG "${DEBUG_LDFLAGS}")
+else()
+  add_link_options("$<$<CONFIG:Debug>:${DEBUG_LDFLAGS}>")
+endif()
+
+add_subdirectory(src)
+
+if (WITH_MAN)
+  add_subdirectory(man)
 endif()

README.adoc

@@ -1,7 +1,7 @@
 = compilescript
 
-**compilescript** allows you to execute files from compiled languages as
-scripts. By default it uses `g++`.
+*compilescript* allows you to execute files from compiled languages like scripts.
+By default it uses `g++`.
 
 It compiles the source file, stores the binary in
 `${XDG_CACHE_HOME}/compilescript/` and executes it. If the binary in cache is
@@ -9,7 +9,7 @@ newer than the source file, the compilation is skipped.
 
 == Usage
 
-Have a look at the https://schlomp.space/tastytea/compilescript/src/branch/master/compilescript.1.adoc[manpage].
+Have a look at the https://schlomp.space/tastytea/compilescript/src/branch/main/man/compilescript.1.adoc[manpage].
 
 == Install
 
@@ -22,8 +22,8 @@ https://schlomp.space/tastytea/overlay[repository].
 
 ==== Dependencies
 
-* C++ compiler (tested: https://gcc.gnu.org/[gcc] 5/6/7/8,
-  https://llvm.org/[clang] 3/5/6)
+* C++ compiler (tested: https://gcc.gnu.org/[gcc] 5/6/8/9,
+  https://llvm.org/[clang] 3/6)
 * https://cmake.org/[cmake] (at least 3.2)
 * http://repo.or.cz/w/libxdg-basedir.git[libxdg-basedir] (tested: 1.2)
 * https://github.com/hyperrealm/libconfig[libconfig++] (tested: 1.5)

man/CMakeLists.txt

@@ -0,0 +1,14 @@
+include(GNUInstallDirs)
+
+add_custom_command(
+  OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}.1"
+  WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}"
+  DEPENDS "${PROJECT_NAME}.1.adoc"
+  COMMAND "${PROJECT_SOURCE_DIR}/man/build_manpage.sh"
+  ARGS "${PROJECT_VERSION}")
+
+add_custom_target(man ALL
+  DEPENDS "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}.1")
+
+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}.1
+  DESTINATION ${CMAKE_INSTALL_MANDIR}/man1)

man/build_manpage.sh

@@ -3,9 +3,10 @@
 name="compilescript"
 
 if [ -n "${1}" ]; then
-    dir="$(dirname ${0})"
+    dir=$(dirname "${0}")
+    version=${1}
     cp -vf "${dir}/${name}.1.adoc" .
-    sed -Ei "s/(Revision: +)[0-9]+\.[0-9]\.[0-9]/\1${1}/" "${name}.1.adoc"
+    sed -Ei "s/(Revision: +)[0-9]+\.[0-9]\.[0-9]/\1${version}/" "${name}.1.adoc"
     a2x --doctype manpage --format manpage --no-xmllint "${name}.1.adoc"
 else
     echo "usage: ${0} VERSION" >&2

man/compilescript.1.adoc

@@ -1,7 +1,8 @@
 = compilescript(1)
+:doctype:       manpage
 :Author:        tastytea
 :Email:         tastytea@tastytea.de
-:Date:          2019-01-25
+:Date:          2019-10-10
 :Revision:      0.0.0
 :man source:    compilescript
 :man version:   {revision}
@@ -9,7 +10,7 @@
 
 == NAME
 
-compilescript - allows you to execute files from compiled languages as scripts.
+compilescript - Execute source files from compiled languages as scripts.
 
 == SYNOPSIS
 
@@ -35,11 +36,15 @@ The compiler invocation looks like this: <configured compiler> <source file>
 
 == OPTIONS
 
-[frame="none",grid="none"]
-|===
-|*--cleanup* | Delete old cache files (older than +clean_after_hours+).
-|*--version* | Print version, copyright and license.
-|===
+*--cleanup*::
+    Delete old cache files. You can configure the timespan after which a file is
+    considered old with the config option _clean_after_hours_.
+
+*--compiler* _command_::
+    Override compiler command. Note that *--compiler*=_command_ does NOT work.
+
+*--version*::
+    Print version, copyright and license.
 
 == EXAMPLES
 
@@ -72,11 +77,8 @@ cache_dir = "/home/user/.cache/compilescript";
 
 == FILES
 
-[format="csv",frame="none",grid="none",cols=">,<"]
-|======
-Configuration file:,`${XDG_CONFIG_HOME}/compilescript.cfg`
-Cache:,`${XDG_CACHE_HOME}/compilescript/`
-|======
+- *Configuration file*: `${XDG_CONFIG_HOME}/compilescript.cfg`
+- *Cache*: `${XDG_CACHE_HOME}/compilescript/`
 
 `${XDG_CONFIG_HOME}` is usually `~/.config` and `${XDG_CACHE_HOME}` is usually
 `~/.cache`.

src/CMakeLists.txt

@@ -0,0 +1,22 @@
+include(GNUInstallDirs)
+
+find_package(PkgConfig REQUIRED)
+pkg_check_modules(libxdg-basedir REQUIRED IMPORTED_TARGET libxdg-basedir)
+pkg_check_modules(libconfig++ REQUIRED IMPORTED_TARGET libconfig++)
+
+# Write version in header
+configure_file("version.hpp.in" "${CMAKE_CURRENT_BINARY_DIR}/version.hpp")
+
+set(sources "main.cpp" "../xdgcfg/src/xdgcfg.cpp")
+
+add_executable(${CMAKE_PROJECT_NAME} "${sources}")
+
+target_include_directories(${PROJECT_NAME}
+  PRIVATE
+  "${PROJECT_SOURCE_DIR}/xdgcfg/include"
+  "${CMAKE_CURRENT_BINARY_DIR}")
+
+target_link_libraries(${CMAKE_PROJECT_NAME}
+  PRIVATE PkgConfig::libxdg-basedir PkgConfig::libconfig++ stdc++fs)
+
+install(TARGETS ${CMAKE_PROJECT_NAME} DESTINATION "${CMAKE_INSTALL_BINDIR}")

src/main.cpp

@@ -14,40 +14,63 @@
  *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#if __cplusplus >= 201703L
-    #include <filesystem>
-#else
-    #include <experimental/filesystem>
-#endif
-#include <iostream>
-#include <string>
-#include <fstream>
-#include <chrono>
-#include <system_error>
-#include <cstdlib>
-#include <unistd.h>
+#include "version.hpp"
+#include "xdgcfg.hpp"
 #include <libconfig.h++>
 #include <basedir.h>
-#include "xdgcfg.hpp"
-#include "version.hpp"
+#include <unistd.h>
+#include <chrono>
+#include <cstdlib>
+#include <exception>
+#include <experimental/filesystem>
+#include <fstream>
+#include <iostream>
+#include <regex>
+#include <string>
+#include <system_error>
+#include <vector>
 
-#if __cplusplus >= 201703L
-    namespace fs = std::filesystem;
-#else
-    namespace fs = std::experimental::filesystem;
-#endif
+namespace fs = std::experimental::filesystem;
 using std::cout;
 using std::cerr;
 using std::endl;
 using std::chrono::system_clock;
 using std::chrono::hours;
 using std::chrono::duration_cast;
+using std::string;
+using std::vector;
 
-string compiler = "g++ -x c++";
-fs::path cache_dir;
-int clean_after_hours = 30 * 24;
+class Compilescript
+{
+private:
+    string _compiler;
+    fs::path _cache_dir;
+    int _clean_after_hours;
 
-bool read_settings()
+public:
+    Compilescript();
+
+    //! Read settings and set member variables.
+    void read_settings();
+    //! Remove files older than `_clean_after_hours` from the cache.
+    void cleanup();
+    //! Compile `filename` if necessary and return filename of binary.
+    string compile(const string &filename);
+    //! Run binary. Pass original `argv`, the 1st entry is ignored.
+    void run(const string &filename, char *argv[]);
+    //! Print version, copyright and license.
+    void print_version();
+    //! Set compiler command.
+    void set_compiler(const string &command);
+};
+
+Compilescript::Compilescript()
+    : _compiler("g++ -x c++")
+    , _clean_after_hours(30 * 24)
+{}
+
+
+void Compilescript::read_settings()
 {
     bool need_save = false;
     xdgcfg config("compilescript.cfg");
@@ -56,105 +79,79 @@ bool read_settings()
 
     if (cfg.exists("compiler"))
     {
-        compiler = cfg["compiler"].c_str();
+        _compiler = cfg["compiler"].c_str();
     }
     else
     {
-        cfg.add("compiler", libconfig::Setting::TypeString) = compiler;
+        cfg.add("compiler", libconfig::Setting::TypeString) = _compiler;
         need_save = true;
     }
 
     if (cfg.exists("clean_after_hours"))
     {
-        cfg.lookupValue("clean_after_hours", clean_after_hours);
+        cfg.lookupValue("clean_after_hours", _clean_after_hours);
     }
     else
     {
         cfg.add("clean_after_hours",
-                libconfig::Setting::TypeInt) = clean_after_hours;
+                libconfig::Setting::TypeInt) = _clean_after_hours;
         need_save = true;
     }
 
     if (cfg.exists("cache_dir"))
     {
-        cache_dir = cfg["cache_dir"].c_str();
+        _cache_dir = cfg["cache_dir"].c_str();
     }
     else
     {
         xdgHandle xdg;
         xdgInitHandle(&xdg);
-        cache_dir = xdgCacheHome(&xdg);
-        cache_dir /= "compilescript";
+        _cache_dir = xdgCacheHome(&xdg);
+        _cache_dir /= "compilescript";
         xdgWipeHandle(&xdg);
     }
-    if (!fs::is_directory(cache_dir))
+    if (!fs::is_directory(_cache_dir))
     {
-        fs::create_directories(cache_dir);
+        fs::create_directories(_cache_dir);
     }
 
     if (need_save)
     {
         config.write();
     }
-
-    return true;
 }
 
-void cleanup()
+void Compilescript::cleanup()
 {
-    for (const fs::directory_entry &entry
-         : fs::recursive_directory_iterator(cache_dir))
+    for (const auto &entry : fs::recursive_directory_iterator(_cache_dir))
     {
-        if (fs::is_regular_file(entry))
+        if (!fs::is_regular_file(entry))
         {
-            auto diff = system_clock::now() - fs::last_write_time(entry);
-            if (duration_cast<hours>(diff).count() > clean_after_hours)
-            {
-                fs::path current_path = entry.path();
-                std::error_code e;
+            continue;
+        }
 
-                while (fs::remove(current_path, e))
+        const auto diff = system_clock::now() - fs::last_write_time(entry);
+        if (duration_cast<hours>(diff).count() > _clean_after_hours)
+        {
+            fs::path current_path = entry.path();
+            std::error_code e;
+
+            while (fs::remove(current_path, e))
+            {
+                current_path = current_path.parent_path();
+                if (current_path == _cache_dir)
                 {
-                    current_path = current_path.parent_path();
-                    if (current_path == cache_dir)
-                    {
-                        break;
-                    }
+                    break;
                 }
             }
         }
     }
 }
 
-int main(int argc, char *argv[])
+string Compilescript::compile(const string &filename)
 {
-    if (!read_settings())
-    {
-        return 1;
-    }
-
-    if (argc <= 1)
-    {
-        cerr << "usage: " << argv[0] << " [file|--cleanup|--version] [arguments]\n";
-        return 1;
-    }
-    if (string(argv[1]) == "--cleanup")
-    {
-        cleanup();
-        return 0;
-    }
-    if (string(argv[1]) == "--version")
-    {
-        cout << "compilescript " << global::version << endl <<
-            "Copyright (C) 2018, 2019 tastytea <tastytea@tastytea.de>\n"
-            "License GPLv3: GNU GPL version 3 <https://www.gnu.org/licenses/gpl-3.0.html>.\n"
-            "This is free software: you are free to change and redistribute it.\n"
-            "There is NO WARRANTY, to the extent permitted by law.\n";
-        return 0;
-    }
-
-    const fs::path original = fs::canonical(argv[1]);
-    const fs::path source = cache_dir / original;
+    const fs::path original = fs::canonical(filename);
+    const fs::path source = _cache_dir / original;
     const fs::path binary = (source.string() + ".bin");
     string compiler_arguments;
     fs::create_directories(binary.parent_path());
@@ -177,14 +174,11 @@ int main(int argc, char *argv[])
                 }
 
                 std::getline(in, buf);
-                if (buf.substr(0, 16) == "//compilescript:")
+                const std::regex re("^(//|#|;) ?compilescript:");
+                std::smatch match;
+                if (std::regex_search(buf, match, re))
                 {
-                    compiler_arguments = buf.substr(16);
-                }
-                else if ((buf.substr(0, 15) == "#compilescript:") ||
-                         (buf.substr(0, 15) == ";compilescript:"))
-                {
-                    compiler_arguments = buf.substr(15);
+                    compiler_arguments = match.suffix();
                 }
                 else
                 {
@@ -202,26 +196,99 @@ int main(int argc, char *argv[])
             }
             else
             {
-                cerr << "ERROR: Could not open file: " << source << endl;
-                return 1;
+                throw std::runtime_error("Could not open file: "
+                                         + source.string());
             }
         }
         else
         {
-            cerr << "ERROR: Could not open file: " << original << endl;
-            return 1;
+            throw std::runtime_error("Could not open file: "
+                                     + original.string());
         }
 
-        int ret = std::system((compiler + " " + source.string() + " "
-                      + compiler_arguments + " -o " + binary.string()).c_str());
+        const string command = _compiler + " " + source.string() + " "
+            + compiler_arguments + " -o " + binary.string();
+        int ret = std::system(command.c_str()); // NOLINT(cert-env33-c)
         if (ret != 0)
         {
-            cerr << "ERROR: Compilation failed.\n";
-            return 1;
+            throw std::runtime_error("Compilation failed.");
         }
     }
 
-    execv(binary.c_str(), &argv[1]);
+    return binary.string();
+}
+
+void Compilescript::run(const string &filename, char *argv[])
+{
+    execvp(filename.c_str(), argv);
+}
+
+void Compilescript::print_version()
+{
+    cout << "compilescript " << global::version << '\n' <<
+        "Copyright (C) 2018, 2019 tastytea <tastytea@tastytea.de>\n"
+        "License GPLv3: GNU GPL version 3 "
+        "<https://www.gnu.org/licenses/gpl-3.0.html>.\n"
+        "This program comes with ABSOLUTELY NO WARRANTY. This is free software,"
+        "\nand you are welcome to redistribute it under certain conditions.\n";
+}
+
+void Compilescript::set_compiler(const string &command)
+{
+    _compiler = command;
+}
+
+int main(int argc, char *argv[])
+{
+    const vector<string> args(argv, argv + argc);
+
+    try
+    {
+        Compilescript App;
+        App.read_settings();
+
+        if (args.size() <= 1)
+        {
+            cerr << "usage: " << args[0]
+                 << " [file|--cleanup|--version|--compiler command] "
+                 << "[arguments]\n";
+            return 1;
+        }
+        if (args[1] == "--cleanup")
+        {
+            App.cleanup();
+            return 0;
+        }
+        if (args[1] == "--version")
+        {
+            App.print_version();
+            return 0;
+        }
+        if (args[1] == "--compiler")
+        {
+            if (args.size() <= 3)
+            {
+                cerr << "Error: You need to specify a command.\n";
+                return 1;
+            }
+            App.set_compiler(args[2]);
+            const string binary = App.compile(args[3]);
+            // We know that argv[3] exists.
+            // NOLINTNEXTLINE(cppcoreguidelines-pro-bounds-pointer-arithmetic)
+            App.run(binary, &argv[3]);
+            return 0;
+        }
+
+        const string binary = App.compile(args[1]);
+        // We know that argv[1] exists.
+        // NOLINTNEXTLINE(cppcoreguidelines-pro-bounds-pointer-arithmetic)
+        App.run(binary, &argv[1]);
+    }
+    catch (const std::exception &e)
+    {
+        cerr << "Error: " << e.what() << endl;
+        return 1;
+    }
 
     return 0;
 }

src/version.hpp.in

@@ -3,7 +3,7 @@
 
 namespace global
 {
-    static constexpr char version[] = "@PROJECT_VERSION@";
+static constexpr char version[] = "@PROJECT_VERSION@";
 }
 
 #endif // VERSION_HPP

src/xdgcfg.cpp

@@ -1 +0,0 @@
-../xdgcfg/src/xdgcfg.cpp

src/xdgcfg.hpp

@@ -1 +0,0 @@
-../xdgcfg/src/xdgcfg.hpp

xdgcfg

@@ -1 +1 @@
-Subproject commit e22f82fc6f1c40cda3d3ce5e671299f26f622528
+Subproject commit 9337964266ae55415beaa7f517cc4240a259289a