Implemented NM_UNINSTALL_CONFIGDIR build option (closes #34) (#37)

This option allows NickelMenu to be uninstalled by deleting the config dir.

.drone.yml

@@ -11,8 +11,18 @@ steps:
   when:
     event: [push, pull_request, tag]
   commands:
+  - make clean
   - make all koboroot
   - mkdir out && mv KoboRoot.tgz src/libnm.so out/
+- name: build-NM_UNINSTALL_CONFIGDIR
+  image: docker.io/geek1011/nickeltc:1.0
+  when:
+    event: [push, pull_request, tag]
+  commands:
+  - make clean
+  - make all koboroot NM_UNINSTALL_CONFIGDIR=1
+  - mkdir out/with-NM_UNINSTALL_CONFIGDIR && mv KoboRoot.tgz src/libnm.so out/with-NM_UNINSTALL_CONFIGDIR/
+  depends_on: [build]
 - name: test-syms
   image: golang:1.14
   when:
@@ -30,9 +40,9 @@ steps:
     access_key: nickelmenu
     secret_key: {from_secret: S3_SECRET_NICKELMENU}
     target: artifacts/build/${DRONE_BUILD_NUMBER}
-    source: out/*
+    source: out/**/*
     strip_prefix: out/
-  depends_on: [build]
+  depends_on: [build, build-NM_UNINSTALL_CONFIGDIR]
 - name: upload-tag
   image: plugins/s3
   when:
@@ -42,11 +52,9 @@ steps:
     access_key: nickelmenu
     secret_key: {from_secret: S3_SECRET_NICKELMENU}
     target: artifacts/tag/${DRONE_TAG}
-    source: out/*
+    source: out/**/*
     strip_prefix: out/
-    when:
+  depends_on: [build, build-NM_UNINSTALL_CONFIGDIR]
-      event: [tag]
-  depends_on: [build]
 - name: upload-commit
   image: plugins/s3
   when:
@@ -57,6 +65,6 @@ steps:
     access_key: nickelmenu
     secret_key: {from_secret: S3_SECRET_NICKELMENU}
     target: artifacts/commit/${DRONE_COMMIT}
-    source: out/*
+    source: out/**/*
     strip_prefix: out/
-  depends_on: [build]
+  depends_on: [build, build-NM_UNINSTALL_CONFIGDIR]

Makefile

@@ -45,6 +45,13 @@ NM_VERSION := $(shell git describe --tags --always --dirty)
 ifdef NM_VERSION
  override CPPFLAGS += -DNM_VERSION='"$(NM_VERSION)"'
 endif
+
+ifeq ($(NM_UNINSTALL_CONFIGDIR),1)
+ $(info -- NM_UNINSTALL_CONFIGDIR... enabled)
+ override CPPFLAGS += -DNM_UNINSTALL_CONFIGDIR
+else
+ $(info -- NM_UNINSTALL_CONFIGDIR... disabled)
+endif
 endif
 
 define GITIGNORE_HEAD

src/config.c

@@ -15,14 +15,6 @@
 #include "menu.h"
 #include "util.h"
 
-#ifndef NM_CONFIG_DIR
-#define NM_CONFIG_DIR "/mnt/onboard/.adds/nm"
-#endif
-
-#ifndef NM_CONFIG_MAX_MENU_ITEMS_PER_MENU
-#define NM_CONFIG_MAX_MENU_ITEMS_PER_MENU 50
-#endif
-
 typedef enum {
     NM_CONFIG_TYPE_MENU_ITEM = 1,
     NM_CONFIG_TYPE_GENERATOR = 2,

src/config.h

@@ -7,6 +7,14 @@ extern "C" {
 #include <stdbool.h>
 #include "menu.h"
 
+#ifndef NM_CONFIG_DIR
+#define NM_CONFIG_DIR "/mnt/onboard/.adds/nm"
+#endif
+
+#ifndef NM_CONFIG_MAX_MENU_ITEMS_PER_MENU
+#define NM_CONFIG_MAX_MENU_ITEMS_PER_MENU 50
+#endif
+
 typedef struct nm_config_t nm_config_t;
 
 // nm_config_parse parses the configuration files in /mnt/onboard/.adds/nm.

src/init.c

@@ -22,6 +22,13 @@ __attribute__((constructor)) void nm_init() {
     char *err;
 
     NM_LOG("version: " NM_VERSION);
+    #ifdef NM_UNINSTALL_CONFIGDIR
+    NM_LOG("feature: NM_UNINSTALL_CONFIGDIR: true");
+    #else
+    NM_LOG("feature: NM_UNINSTALL_CONFIGDIR: false");
+    #endif
+    NM_LOG("config dir: %s", NM_CONFIG_DIR);
+
     NM_LOG("init: creating failsafe");
     nm_failsafe_t *fs;
     if (!(fs = nm_failsafe_create(&err)) && err) {
@@ -31,13 +38,22 @@ __attribute__((constructor)) void nm_init() {
     }
 
     NM_LOG("init: checking for uninstall flag");
-    if (!access("/mnt/onboard/.adds/nm/uninstall", F_OK)) {
+    if (!access(NM_CONFIG_DIR "/uninstall", F_OK)) {
         NM_LOG("init: flag found, uninstalling");
         nm_failsafe_uninstall(fs);
-        unlink("/mnt/onboard/.adds/nm/uninstall");
+        unlink(NM_CONFIG_DIR "/uninstall");
         goto stop;
     }
 
+    #ifdef NM_UNINSTALL_CONFIGDIR
+    NM_LOG("init: NM_UNINSTALL_CONFIGDIR: checking if config dir exists");
+    if (access(NM_CONFIG_DIR, F_OK) && errno == ENOENT) {
+        NM_LOG("init: config dir does not exist, uninstalling");
+        nm_failsafe_uninstall(fs);
+        goto stop;
+    }
+    #endif
+
     NM_LOG("init: parsing config");
     size_t items_n;
     nm_menu_item_t **items;