exchange usage of inlined libraries with git submodules

Integrate the git submodules as subproject in
the meson.build as static library and pass down
default values for specific meson options to the
subprojects. In addition, the other tooling like
clang-tidy script needed to be adjusted to
include the libraries.

Signed-off-by: Michael Engel <[email protected]>

.gitconfig

@@ -0,0 +1,2 @@
+[safe]
+	directory = /root

.markdownlint-cli2.yaml

@@ -8,3 +8,4 @@ globs:
 
 ignores:
   - "doc/man/*.md"
+  - "subprojects/*/*.md"

Makefile

@@ -4,7 +4,7 @@ DESTDIR ?=
 BUILDDIR=builddir
 
 CODESPELL_PARAMS=\
-	-S Makefile,imgtype,copy,AUTHORS,bin,.git,CHANGELOG.md,changelog.txt,.cirrus.yml,"*.xz,*.gz,*.tar,*.tgz,*ico,*.png,*.1,*.5,*.orig,*.rej,*.xml,*xsl",build.ninja,intro-targets.json,./tests/tests/tier0/proxy-service-fails-on-typo-in-file/systemd/simple.service,tags,./builddir\
+	-S Makefile,imgtype,copy,AUTHORS,bin,.git,CHANGELOG.md,changelog.txt,.cirrus.yml,"*.xz,*.gz,*.tar,*.tgz,*ico,*.png,*.1,*.5,*.orig,*.rej,*.xml,*xsl",build.ninja,intro-targets.json,./tests/tests/tier0/proxy-service-fails-on-typo-in-file/systemd/simple.service,tags,./builddir,./subprojects,\
 	-L keypair,flate,uint,iff,od,ERRO
 
 build:

build-scripts/build-srpm.sh

@@ -25,8 +25,13 @@ sed \
 
 # Prepare source archive
 [[ -d rpmbuild/SOURCES ]] || mkdir -p rpmbuild/SOURCES
-git archive --format=tar --prefix=hirte-$VERSION/ --add-file=hirte.spec HEAD \
-    | gzip -9 > rpmbuild/SOURCES/hirte-$VERSION.tar.gz
+
+git archive --format=tar --prefix=hirte-$VERSION/ --add-file=hirte.spec -o hirte-$VERSION.tar HEAD
+git submodule foreach --recursive \
+    "git archive --prefix=hirte-$VERSION/\$path/ --output=\$sha1.tar HEAD && \
+     tar --concatenate --file=$(pwd)/hirte-$VERSION.tar \$sha1.tar && rm \$sha1.tar"
+gzip hirte-$VERSION.tar
+mv hirte-$VERSION.tar.gz rpmbuild/SOURCES/
 
 # Build source package
 rpmbuild \

build-scripts/clang-tidy-all.sh

@@ -6,6 +6,7 @@ additional_opts=$1
 SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
 SRC_DIR=$SCRIPT_DIR/../src
 BUILD_DIR=$SCRIPT_DIR/../builddir
+SUBPROJECTS_DIR=$SCRIPT_DIR/../subprojects
 
 ALL_SRC_FILES=$(find $SRC_DIR -name "*.[ch]" \
         ! -path "$SRC_DIR/libhirte/ini/ini.[ch]" \
@@ -18,5 +19,10 @@ do
     echo "Linting $src_file..."
 
     # Header config.h is autogenerated during the build, so it needs to be added manually here as a parameter
-    clang-tidy $additional_opts --quiet $src_file -- -I $SRC_DIR -I $BUILD_DIR -include config.h
+    clang-tidy $additional_opts --quiet $src_file -- \
+        -I $SRC_DIR \
+        -I $BUILD_DIR \
+        -I $SUBPROJECTS_DIR/hashmap.c \
+        -I $SUBPROJECTS_DIR/inih \
+        -include config.h
 done

meson.build

@@ -27,14 +27,6 @@ if api_bus == 'user'
    common_cflags += '-DUSE_USER_API_BUS'
 endif
 
-# inih library options
-# Set option of maximum line length
-# 500 characters for actual value
-# 20 characters as buffer (key, library internal usage, etc.)
-common_cflags += '-DINI_MAX_LINE=520'
-# Set option to abort parsing on first error
-common_cflags += '-DINI_STOP_ON_FIRST_ERROR=1'
-
 # Build time configuration header file
 conf = configuration_data()
 
@@ -55,6 +47,22 @@ config_h = configure_file(
 add_project_arguments('-include', 'config.h', language : 'c')
 
 
+# (External) subprojects used by hirte
+inih = subproject('inih', default_options: [
+  'default_library=static',           # include it as static library
+  '-Dinih:distro_install=false',      # disable distro install so other options are used
+  '-Dinih:max_line_length=520',       # set maximum line length so that ~500 characters can be used as value
+  '-Dinih:stop_on_first_error=true',  # abort after first error when parsing file
+])
+inih_dep = inih.get_variable('inih_dep')
+
+hashmapc = subproject('hashmap.c', default_options: [
+  'default_library=static', # include it as static library
+  'werror=false'            # don't treat warnings as errors since it
+])
+hashmapc_dep = hashmapc.get_variable('hashmapc_dep')
+
+
 # Subdirectory for the shared library.
 subdir('src/libhirte')
 

src/agent/agent.h

@@ -5,9 +5,11 @@
 #include <stdbool.h>
 #include <sys/socket.h>
 
+// NOLINTNEXTLINE
+#include <hashmap.h>
+
 #include "libhirte/common/cfg.h"
 #include "libhirte/common/common.h"
-#include "libhirte/hashmap/hashmap.h"
 
 #include "types.h"
 

src/agent/meson.build

@@ -13,6 +13,8 @@ executable(
   node_src,
   dependencies: [
     systemd_dep,
+    inih_dep,
+    hashmapc_dep,
   ],
   link_with: [
     hirte_lib,

src/libhirte/common/cfg.c

@@ -8,10 +8,11 @@
 #include <string.h>
 #include <unistd.h>
 
+#include <hashmap.h>
+#include <ini.h>
+
 #include "libhirte/common/list.h"
 #include "libhirte/common/network.h"
-#include "libhirte/hashmap/hashmap.h"
-#include "libhirte/ini/ini.h"
 
 #include "cfg.h"
 #include "common.h"