diff options
author | June McEnroe <june@causal.agency> | 2022-07-17 18:44:58 -0400 |
---|---|---|
committer | June McEnroe <june@causal.agency> | 2022-07-17 18:44:58 -0400 |
commit | f2ca5b05e8ff38120aacda2eb105961bf0be7fee (patch) | |
tree | 11b82259ed0fbf54f9d96a1c7e248942ee243c8b | |
parent | Update copyrights, authors (diff) | |
download | pounce-f2ca5b05e8ff38120aacda2eb105961bf0be7fee.tar.gz pounce-f2ca5b05e8ff38120aacda2eb105961bf0be7fee.zip |
Use xdg.c for pounce-palaver
-rw-r--r-- | Makefile | 2 | ||||
-rw-r--r-- | palaver.c | 43 |
2 files changed, 12 insertions, 33 deletions
diff --git a/Makefile b/Makefile index 029d184..6d72bdd 100644 --- a/Makefile +++ b/Makefile @@ -32,7 +32,7 @@ OBJS.pounce += xdg.o OBJS.pounce-edit = edit.o xdg.o OBJS.pounce-notify = notify.o -OBJS.pounce-palaver = palaver.o +OBJS.pounce-palaver = palaver.o xdg.o OBJS += ${OBJS.calico} OBJS += ${OBJS.pounce} diff --git a/palaver.c b/palaver.c index 37182eb..1453551 100644 --- a/palaver.c +++ b/palaver.c @@ -43,11 +43,11 @@ #include <tls.h> #include <unistd.h> +char *dataPath(char *buf, size_t cap, const char *path, int i); + // Why must it return (const unsigned char *)? #define sqlite3_column_text(...) (const char *)sqlite3_column_text(__VA_ARGS__) -#define DATABASE_PATH "pounce/palaver.sqlite" - #define SQL(...) #__VA_ARGS__ #define ARRAY_LEN(a) (sizeof(a) / sizeof((a)[0])) @@ -70,48 +70,27 @@ static void dbOpen(const char *path, int flags) { sqlite3_busy_timeout(db, 10000); } -static void dbFind(char *path) { +static void dbFind(const char *path) { if (path) { dbOpen(path, SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE); if (db) return; errx(EX_NOINPUT, "%s: database not found", path); } - const char *home = getenv("HOME"); - const char *dataHome = getenv("XDG_DATA_HOME"); - const char *dataDirs = getenv("XDG_DATA_DIRS"); - if (!dataDirs) dataDirs = "/usr/local/share:/usr/share"; - char buf[PATH_MAX]; - if (dataHome) { - snprintf(buf, sizeof(buf), "%s/" DATABASE_PATH, dataHome); - } else { - if (!home) errx(EX_CONFIG, "HOME unset"); - snprintf(buf, sizeof(buf), "%s/.local/share/" DATABASE_PATH, home); - } - dbOpen(buf, SQLITE_OPEN_READWRITE); - if (db) return; - - char create[PATH_MAX]; - snprintf(create, sizeof(create), "%s", buf); - - while (*dataDirs) { - size_t len = strcspn(dataDirs, ":"); - snprintf(buf, sizeof(buf), "%.*s/" DATABASE_PATH, (int)len, dataDirs); + for (int i = 0; dataPath(buf, sizeof(buf), "palaver.sqlite", i); ++i) { dbOpen(buf, SQLITE_OPEN_READWRITE); if (db) return; - dataDirs += len; - if (*dataDirs) dataDirs++; } - char *base = strrchr(create, '/'); - *base = '\0'; - int error = mkdir(create, 0700); - if (error && errno != EEXIST) err(EX_CANTCREAT, "%s", create); - *base = '/'; + int error = mkdir(dataPath(buf, sizeof(buf), "", 0), 0700); + if (error && errno != EEXIST) err(EX_CANTCREAT, "%s", buf); - dbOpen(create, SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE); - if (!db) errx(EX_CANTCREAT, "%s: cannot create database", create); + dbOpen( + dataPath(buf, sizeof(buf), "palaver.sqlite", 0), + SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE + ); + if (!db) errx(EX_CANTCREAT, "%s: cannot create database", buf); } static int dbParam(sqlite3_stmt *stmt, const char *param) { |