.gitignore

@@ -1,7 +1,6 @@
 takl.sqlite3
 takl
 takl-daemon
-takl.sock
 
 out
 .vscode

Makefile

@@ -1,6 +1,5 @@
-FLAGS       = -g -Wall -Wextra -DLOG_USE_COLOR
-LD_FLAGS    = -lsqlite3 -lpthread
-NOTIF_FLAGS = `pkg-config --cflags --libs libnotify`
+FLAGS    = -g -Wall -Wextra -DLOG_USE_COLOR `pkg-config --cflags --libs libnotify`
+LD_FLAGS = -lsqlite3 -lpthread
 
 
 $(shell mkdir -p out)
@@ -12,9 +11,6 @@ all: takl takl-daemon
 out/%.o: src/%.c
 	$(CC) -c $(FLAGS) $^ -o $@
 
-out/notification.o: src/main/notification.c
-	$(CC) -c $(FLAGS) $(NOTIF_FLAGS) $^ -o $@
-
 out/%.o: src/main/%.c
 	$(CC) -c $(FLAGS) $^ -o $@
 
@@ -24,8 +20,8 @@ out/%.o: src/log/%.c
 takl: out/main.o out/db.o out/tasks.o out/utils.o
 	$(CC) $(FLAGS) $(LD_FLAGS) $^ -o $@
 
-takl-daemon: out/daemon.o out/db.o out/tasks.o out/utils.o out/log.o out/notification.o
-	$(CC) $(FLAGS) $(LD_FLAGS) $(NOTIF_FLAGS) $^ -o $@
+takl-daemon: out/daemon.o out/db.o out/tasks.o out/utils.o out/log.o
+	$(CC) $(FLAGS) $(LD_FLAGS) $^ -o $@
 
 
 install: takl takl-daemon

README.md

@@ -5,7 +5,7 @@ facile d'utilisation et donnant rapidement les choses demandées
 
 ### Installation
 
-Les seules dépendances nécessaires sont `gcc` et `libnotify`.
+Les seules dépendances nécessaires sont `gcc` et `g++`.
 (Peut-être que des bibliothèques comme sqlite3 sont à installer sur certaines distributions/ OS)
 
 ```bash
@@ -24,17 +24,14 @@ Pour gnome, `make gnome-install` lancera le programme à tous les prochains red
 ### Utilisation
 
 ```
-Utilisation: takl ( list | add | reschedule | info | rm | done )
-        list [category] [-a:voir les tâches complétées]
-        add [category:]<task> [date]
-        reschedule <id> [date]
+Utilisation: takl ( list | add | info | rm | done )
+        list [-a:voir les tâches complétées]
+        add <task> [date]
+        Format de la date:
+                Relatif: min+%d, h+%d, j+%d
+                Absolu:  dd/mm (pas de changement d'année pour l'instant)
         info <id1> <id2> ...
         done <id1> <id2> ...
         rm   <id1> <id2> ...
-
-
-Format des dates:
-        Relatif: min+%d, h+%d, j+%d
-        Absolu:  dd/mm (pas de changement d'année pour l'instant)
 ```
 Appeler l'exécutable sans arguments vous renverra cette petite liste de commandes que vous pouvez utiliser.

src/daemon.c

@@ -22,7 +22,6 @@
 #include "main/include/utils.h"
 #include "main/include/colors.h"
 #include "main/include/config.h"
-#include "main/include/notification.h"
 
 
 #define EVENT_SIZE  ( sizeof (struct inotify_event) )
@@ -36,9 +35,10 @@ pthread_mutex_t tasks_lock = PTHREAD_MUTEX_INITIALIZER; // locks tasks and has_c
 
 
 
-void* inotify_check_changes(void* sk_path) {
-    char* socket_path = (char*)sk_path;
+void* inotify_check_changes(void* arg) {
+    (void)arg;
     char buffer[EVENT_BUF_LEN];
+    char* socket_path = get_socket_path();
 
     while (1) { // Regarde si /tmp/takl.$USER a changé (donc si la BDD a changé et doit être rechargée)
         int fd = inotify_init();
@@ -67,11 +67,14 @@ void* inotify_check_changes(void* sk_path) {
         inotify_rm_watch(fd, wd);
         close(fd);
     }
+    free(socket_path);
     return NULL;
 }
 
 
-int existing_takl_daemon(char* socket_path) {
+int existing_takl_daemon() {
+    char* socket_path = get_socket_path();
+
     if (!access(socket_path, F_OK)) { // tmp file exists
         FILE* fp = fopen(socket_path, "r+");
         int daemon_pid;
@@ -87,16 +90,15 @@ int existing_takl_daemon(char* socket_path) {
     fprintf(fp, "%d", getpid());
     fclose(fp);
 
+    free(socket_path);
     return false;
 }
 
 
 int main() {
     bool notified_no_change = false;
-    char* socket_path = get_socket_path();
-    log_debug("Socket path:%s", socket_path);
 
-    if (!existing_takl_daemon(socket_path)) {
+    if (!existing_takl_daemon()) {
         log_info("TaKl " VERSION " -- Daemon started");
     } else {
         log_info("TaKl Daemon déjà en cours d'exécution. Arrêt");
@@ -106,7 +108,7 @@ int main() {
     tasks = get_task_list(false, false);
 
     pthread_t inotify_id = 0; // Lancement d'inotify dans un autre fil
-    pthread_create(&inotify_id, NULL, inotify_check_changes, (void*)socket_path);
+    pthread_create(&inotify_id, NULL, inotify_check_changes, NULL);
 
     while (1) {
         pthread_mutex_lock(&tasks_lock);
@@ -167,6 +169,5 @@ int main() {
     pthread_join(inotify_id, NULL);
 
     free_task_list(tasks);
-    free(socket_path);
     return 0;
 }

src/main/db.c

@@ -25,17 +25,8 @@ void sqlAssert(int ret, sqlite3* db, char* file, int line) {
 
 
 char* get_db_path() {
-    /*
-    Checks if $TAKL_DB env variable is set
-    else, the DB is located at $HOME/.config/takl.sqlite3
-    */
-    char* env_db_path = getenv("TAKL_DB");
-    if (env_db_path) {
-        char* db_path = malloc(sizeof(char)*(strlen(env_db_path)+1));
-        strcpy(db_path, env_db_path);
-
-        return db_path;
-    } else {
+    #ifndef DB_FILE
+        //! We should check that $HOME/.config already exists even if it should be the case
         char* base_path = ".config/takl.sqlite3";
 
         char* home_dir = getenv("HOME");
@@ -45,7 +36,12 @@ char* get_db_path() {
         sprintf(db_path, "%s/%s", home_dir, base_path);
 
         return db_path;
-    }
+    #else
+        char* db_path = malloc(sizeof(char)*(strlen(DB_FILE)+1));
+        memcpy(db_path, DB_FILE, sizeof(char)*(strlen(DB_FILE)+1));
+
+        return db_path;
+    #endif
 }
 
 
@@ -63,7 +59,7 @@ sqlite3* get_db() {
             "CREATE TABLE tasks ( \
             id INTEGER PRIMARY KEY,\
             text TEXT NOT NULL,\
-            category TEXT DEFAULT NULL, \
+            category TEXT DEFAULT '', \
             done INTEGER, \
             due_to INTEGER \
             );",
@@ -157,8 +153,6 @@ int update_task(task_t t) {
     sprintf(str_due_to, "%ld", t.due_to);
     sqlCheck( sqlite3_bind_text(stmt, 4, str_due_to, -1, SQLITE_STATIC) );
 
-    sqlite3_step(stmt);
-
     sqlCheck( sqlite3_finalize(stmt) );
     sqlCheck( sqlite3_close(db) );
     notify_change();
@@ -208,12 +202,8 @@ void get_task(int id, task_t* t) {
         strcpy(t->text, text);
 
         char* category = (char*)sqlite3_column_text(stmt, 4);
-        if (category) {
-            t->category = malloc(sizeof(char)*(strlen(category)+1));
-            strcpy(t->category, category);
-        } else {
-            t->category = NULL;
-        }
+        t->category = malloc(sizeof(char)*(strlen(category)+1));
+        strcpy(t->category, category);
     }
 
     sqlCheck( sqlite3_finalize(stmt) );
@@ -248,13 +238,8 @@ task_list_t* get_task_list(char* input_category, bool include_completed) {
             strcpy(t.text, text);
 
             char* category = (char*)sqlite3_column_text(stmt, 4);
-            if (category) {
-                t.category = malloc(sizeof(char)*(strlen(category)+1));
-                strcpy(t.category, category);
-            } else {
-                t.category = NULL;
-            }
-            
+            t.category = malloc(sizeof(char)*(strlen(category)+1));
+            strcpy(t.category, category);
 
             task_list_t* cur = malloc(sizeof(task_list_t)); // Add the parsed task to the beginning of the list
             cur->task = t;

src/main/include/config.h

@@ -1,11 +1,15 @@
 #ifndef DEF_CONFIG_H
 #define DEF_CONFIG_H
 
-#define VERSION "1.3.5"
+#define VERSION "1.2.1"
+
+// By default, $HOME/.config/takl.sqlite3 is used. You can change this behaviour here
+//#define DB_FILE "takl.sqlite3"
+
+// By default, /tmp/takl.$USER is used. You can change this behaviour here
+//#define SOCKET_FILE "takl.sock"
 
 #define MAX_TASK_ID 10000 // max is set to MAX_TASK_ID-1
 #define NEW_TASK_ID_MAX_RETRIES 10000 // number of retries before giving up
 
-#define TMPDIR "/tmp" // Some Unix systems don't use /tmp as tmp dir (eg Android)
-
 #endif

src/main/include/notification.h

@@ -1,10 +0,0 @@
-#ifndef DEF_NOTIFICATION_H
-#define DEF_NOTIFICATION_H
-
-#include "tasks.h"
-/*
-Envoyer une notification avec t
-*/
-void desktop_notification(task_t t);
-
-#endif

src/main/include/tasks.h

@@ -29,4 +29,9 @@ Libère la mémoire allouée à une liste de tâches
 */
 void free_task_list(task_list_t* list);
 
+/*
+Envoyer une notification avec t
+*/
+void desktop_notification(task_t t);
+
 #endif

src/main/notification.c

@@ -1,11 +0,0 @@
-#include <libnotify/notify.h>
-
-#include "include/notification.h"
-
-void desktop_notification(task_t t) {
-    notify_init ("TaKl");
-	NotifyNotification * Notif = notify_notification_new ("TaKl Daemon", t.text, "dialog-information");
-	notify_notification_show (Notif, NULL);
-	g_object_unref(G_OBJECT(Notif));
-	notify_uninit();
-}

src/main/tasks.c

@@ -5,6 +5,7 @@ Fonctions utilitaires concernant les tâches
 #include <stdio.h>
 #include <time.h>
 #include <string.h>
+#include <libnotify/notify.h>
 
 #include "include/db.h"
 #include "include/struct.h"
@@ -130,4 +131,12 @@ void free_task_list(task_list_t* list) {
         free(list);
         list = next;
     }
+}
+
+void desktop_notification(task_t t) {
+    notify_init ("TaKl");
+	NotifyNotification * Notif = notify_notification_new ("TaKl Daemon", t.text, "dialog-information");
+	notify_notification_show (Notif, NULL);
+	g_object_unref(G_OBJECT(Notif));
+	notify_uninit();
 }

src/main/utils.c

@@ -8,30 +8,22 @@
 #include "include/struct.h"
 
 char* get_socket_path() {
-    /*
-    Checks if $TAKL_SOCKET env variable is set
-    else, the socket is located at /tmp/takl
-    */
-    char* env_socket_path = getenv("TAKL_SOCKET");
-    if (env_socket_path) {
-        char* socket_path = malloc(sizeof(char)*(strlen(env_socket_path)+1));
-        strcpy(socket_path, env_socket_path);
-
-        return socket_path;
-    } else {
-        char* base_path = TMPDIR "/takl";
+    #ifndef SOCKET_FILE
+        char* base_path = "/tmp/takl";
 
         char* username = getenv("USER");
-        if (!username)
-            username = getenv("USERNAME");
-
         assert(username != NULL);
 
         char* socket_path = malloc(sizeof(char)*(strlen(base_path)+strlen(username)+1));
         sprintf(socket_path, "%s.%s", base_path, username);
 
         return socket_path;
-    }
+    #else
+        char* socket_path = malloc(sizeof(char)*(strlen(SOCKET_FILE)+1));
+        memcpy(socket_path, SOCKET_FILE, sizeof(char)*(strlen(SOCKET_FILE)+1));
+
+        return socket_path;
+    #endif
 }
 
 

tests/main.sh

@@ -1,42 +0,0 @@
-#!/bin/bash
-
-TMPDIR=$(mktemp -d)
-
-#! On ne teste pas que l'on peut bien récupérer ces chemins automatiquement,
-#! ce qui peut-être une source d'erreurs
-export TAKL_DB="$TMPDIR/takl.sqlite3"
-export TAKL_SOCKET="$TMPDIR/takl.socket"
-echo "Using $TMPDIR"
-
-make -j
-
-# Catégories
-./takl add "nocategory"
-./takl add "category1:test1"
-
-# Dates
-./takl add "date:avec date" min+5
-./takl add "date:avec date2" h+5
-./takl add "date:avec date3" j+5
-./takl add "date:avec date4" "31/12"
-
-TASK_ID=$(./takl add "done soon" | sed 's/\x1B\[[0-9;]\{1,\}[A-Za-z]//g' | awk '-F[' '{ print $2 }' | awk '-F]' '{ print $1 }')
-
-# List
-./takl list
-./takl list date
-
-# Reschedule
-./takl reschedule $TASK_ID "22/11"
-
-# Get info
-./takl info $TASK_ID
-
-# Mark as done
-./takl done $TASK_ID
-./takl list -a
-
-./takl rm $TASK_ID
-
-#* À rajouter: tests sur le fait que les changements ont bien eu lieu
-rm $TMPDIR -r