Ref #559: Geeqie won't start if clutter fails to init
[geeqie.git] / src / main.c
index d6931a4..fd3343e 100644 (file)
@@ -42,6 +42,7 @@
 #include "layout.h"
 #include "layout_image.h"
 #include "layout_util.h"
+#include "misc.h"
 #include "options.h"
 #include "remote.h"
 #include "secure_save.h"
 #include <clutter-gtk/clutter-gtk.h>
 #endif
 
+#ifdef HAVE_GTHREAD
+/** @FIXME see below */
+#include <X11/Xlib.h>
+#endif
 
 gboolean thumb_format_changed = FALSE;
 static RemoteConnection *remote_connection = NULL;
 
+gchar *gq_prefix;
+gchar *gq_localedir;
+gchar *gq_helpdir;
+gchar *gq_htmldir;
+gchar *gq_app_dir;
+gchar *gq_bin_dir;
+gchar *desktop_file_template;
+
 /*
  *-----------------------------------------------------------------------------
  * keyboard functions
@@ -134,7 +147,7 @@ static void parse_command_line_add_file(const gchar *file_path, gchar **path, gc
                {
                if (!*path) *path = remove_level_from_path(path_parsed);
                if (!*file) *file = g_strdup(path_parsed);
-               *list = g_list_prepend(*list, file_data_new_group(path_parsed));
+               *list = g_list_prepend(*list, path_parsed);
                }
 }
 
@@ -222,6 +235,11 @@ static void parse_command_line(gint argc, gchar *argv[])
        gboolean remote_do = FALSE;
        gchar *first_dir = NULL;
        gchar *app_lock;
+       gchar *pwd;
+       gchar *current_dir;
+       gchar *geometry = NULL;
+       GtkWidget *dialog_warning;
+       GString *command_line_errors = g_string_new(NULL);
 
        command_line = g_new0(CommandLine, 1);
 
@@ -257,6 +275,9 @@ static void parse_command_line(gint argc, gchar *argv[])
                                parse_command_line_process_file(cmd_all, &command_line->path, &command_line->file,
                                                                &list, &command_line->collection_list, &first_dir);
                                }
+                       else if (download_web_file(cmd_line, FALSE, NULL))
+                               {
+                               }
                        else if (is_collection(cmd_line))
                                {
                                gchar *path = NULL;
@@ -270,6 +291,10 @@ static void parse_command_line(gint argc, gchar *argv[])
                                {
                                /* do nothing but do not produce warnings */
                                }
+                       else if (strncmp(cmd_line, "--disable-clutter", 17) == 0 && (cmd_line[17] == '\0'))
+                               {
+                               /* do nothing but do not produce warnings */
+                               }
                        else if (strcmp(cmd_line, "+t") == 0 ||
                                 strcmp(cmd_line, "--with-tools") == 0)
                                {
@@ -333,6 +358,14 @@ static void parse_command_line(gint argc, gchar *argv[])
                                {
                                set_regexp(g_strdup(cmd_line+3));
                                }
+                       else if (strncmp(cmd_line, "-n", 2) == 0)
+                               {
+                               command_line->new_instance = TRUE;
+                               }
+                       else if (strncmp(cmd_line, "--new-instance", 14) == 0)
+                               {
+                               command_line->new_instance = TRUE;
+                               }
                        else if (strcmp(cmd_line, "-rh") == 0 ||
                                 strcmp(cmd_line, "--remote-help") == 0)
                                {
@@ -346,7 +379,7 @@ static void parse_command_line(gint argc, gchar *argv[])
                        else if (strcmp(cmd_line, "-v") == 0 ||
                                 strcmp(cmd_line, "--version") == 0)
                                {
-                               printf_term(FALSE, "%s %s\n", GQ_APPNAME, VERSION);
+                               printf_term(FALSE, "%s %s GTK%d\n", GQ_APPNAME, VERSION, gtk_major_version);
                                exit(0);
                                }
                        else if (strcmp(cmd_line, "--alternate") == 0)
@@ -368,6 +401,7 @@ static void parse_command_line(gint argc, gchar *argv[])
                                print_term(FALSE, _("  -l, --list [files] [collections] open collection window for command line\n"));
                                print_term(FALSE, _("      --blank                      start with blank file list\n"));
                                print_term(FALSE, _("      --geometry=XxY+XOFF+YOFF     set main window location\n"));
+                               print_term(FALSE, _("  -n, --new-instance               open a new instance of Geeqie\n"));
                                print_term(FALSE, _("  -r, --remote                     send following commands to open window\n"));
                                print_term(FALSE, _("  -rh,--remote-help                print remote command list\n"));
 #ifdef DEBUG
@@ -377,7 +411,8 @@ static void parse_command_line(gint argc, gchar *argv[])
                                print_term(FALSE, _("  +w, --show-log-window            show log window\n"));
                                print_term(FALSE, _("  -o:<file>, --log-file:<file>     save log data to file\n"));
                                print_term(FALSE, _("  -v, --version                    print version info\n"));
-                               print_term(FALSE, _("  -h, --help                       show this message\n\n"));
+                               print_term(FALSE, _("  -h, --help                       show this message\n"));
+                               print_term(FALSE, _("      --disable-clutter            disable use of Clutter library (i.e. GPU accel.)\n\n"));
 
 #if 0
                                /* these options are not officially supported!
@@ -390,13 +425,28 @@ static void parse_command_line(gint argc, gchar *argv[])
                                }
                        else if (!remote_do)
                                {
-                               printf_term(TRUE, _("invalid or ignored: %s\nUse --help for options\n"), cmd_line);
+                               command_line_errors = g_string_append(command_line_errors, cmd_line);
+                               command_line_errors = g_string_append(command_line_errors, "\n");
                                }
 
                        g_free(cmd_all);
                        g_free(cmd_line);
                        i++;
                        }
+
+               if (command_line_errors->len > 0)
+                       {
+                       dialog_warning = gtk_message_dialog_new(NULL, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
+                       gtk_message_dialog_format_secondary_text(GTK_MESSAGE_DIALOG(dialog_warning), "%s", command_line_errors->str);
+                       gtk_window_set_title(GTK_WINDOW(dialog_warning), GQ_APPNAME);
+                       gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
+                       gtk_dialog_run(GTK_DIALOG(dialog_warning));
+                       gtk_widget_destroy(dialog_warning);
+                       g_string_free(command_line_errors, TRUE);
+
+                       exit(EXIT_FAILURE);
+                       }
+
                g_free(base_dir);
                parse_out_relatives(command_line->path);
                parse_out_relatives(command_line->file);
@@ -413,16 +463,24 @@ static void parse_command_line(gint argc, gchar *argv[])
                }
        g_free(first_dir);
 
-       /* If Geeqie is already running, prevent a second instance
-        * from being started. Open a new window instead.
-        */
-       app_lock = g_build_filename(get_rc_dir(), ".command", NULL);
-       if (remote_server_exists(app_lock) && !remote_do)
+       if (!command_line->new_instance)
                {
-               remote_do = TRUE;
-               remote_list = g_list_append(remote_list, "--new-window");
-       }
-       g_free(app_lock);
+               /* If Geeqie is already running, prevent a second instance
+                * from being started. Open a new window instead.
+                */
+               app_lock = g_build_filename(get_rc_dir(), ".command", NULL);
+               if (remote_server_exists(app_lock) && !remote_do)
+                       {
+                       remote_do = TRUE;
+                       if (command_line->geometry)
+                               {
+                               geometry = g_strdup_printf("--geometry=%s", command_line->geometry);
+                               remote_list = g_list_prepend(remote_list, geometry);
+                               }
+                       remote_list = g_list_prepend(remote_list, "--new-window");
+                       }
+               g_free(app_lock);
+               }
 
        if (remote_do)
                {
@@ -430,20 +488,40 @@ static void parse_command_line(gint argc, gchar *argv[])
                        {
                        GList *work = remote_errors;
 
-                       printf_term(TRUE,_("Invalid or ignored remote options: "));
                        while (work)
                                {
                                gchar *opt = work->data;
 
-                               printf_term(TRUE, "%s%s", (work == remote_errors) ? "" : ", ", opt);
+                               command_line_errors = g_string_append(command_line_errors, opt);
+                               command_line_errors = g_string_append(command_line_errors, "\n");
                                work = work->next;
                                }
 
-                       printf_term(TRUE, _("\nUse --remote-help for valid remote options.\n"));
+                       dialog_warning = gtk_message_dialog_new(NULL, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
+                       gtk_message_dialog_format_secondary_text(GTK_MESSAGE_DIALOG(dialog_warning), "%s", command_line_errors->str);
+                       gtk_window_set_title(GTK_WINDOW(dialog_warning), GQ_APPNAME);
+                       gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
+                       gtk_dialog_run(GTK_DIALOG(dialog_warning));
+                       gtk_widget_destroy(dialog_warning);
+                       g_string_free(command_line_errors, TRUE);
+
+                       exit(EXIT_FAILURE);
                        }
 
+               /* prepend the current dir the remote command was made from,
+                * for use by any remote command that needs it
+                */
+               current_dir = g_get_current_dir();
+               pwd = g_strconcat("--PWD:", current_dir, NULL);
+               remote_list = g_list_prepend(remote_list, pwd);
+
                remote_control(argv[0], remote_list, command_line->path, list, command_line->collection_list);
+               /* There is no return to this point
+                */
+               g_free(pwd);
+               g_free(current_dir);
                }
+       g_free(geometry);
        g_list_free(remote_list);
 
        if (list && list->next)
@@ -452,7 +530,7 @@ static void parse_command_line(gint argc, gchar *argv[])
                }
        else
                {
-               filelist_free(list);
+               string_list_free(list);
                command_line->cmd_list = NULL;
                }
 
@@ -503,6 +581,31 @@ static void parse_command_line_for_debug_option(gint argc, gchar *argv[])
 #endif
 }
 
+#ifdef HAVE_CLUTTER
+static gboolean parse_command_line_for_clutter_option(gint argc, gchar *argv[])
+{
+       const gchar *clutter_option = "--disable-clutter";
+       gint len = strlen(clutter_option);
+       gboolean ret = FALSE;
+
+       if (argc > 1)
+               {
+               gint i;
+
+               for (i = 1; i < argc; i++)
+                       {
+                       const gchar *cmd_line = argv[i];
+                       if (strncmp(cmd_line, clutter_option, len) == 0)
+                               {
+                               ret = TRUE;
+                               }
+                       }
+               }
+
+       return ret;
+}
+#endif
+
 /*
  *-----------------------------------------------------------------------------
  * startup, init, and exit
@@ -510,11 +613,12 @@ static void parse_command_line_for_debug_option(gint argc, gchar *argv[])
  */
 
 #define RC_HISTORY_NAME "history"
+#define RC_MARKS_NAME "marks"
 
 static void setup_env_path(void)
 {
        const gchar *old_path = g_getenv("PATH");
-       gchar *path = g_strconcat(GQ_BIN_DIR, ":", old_path, NULL);
+       gchar *path = g_strconcat(gq_bin_dir, ":", old_path, NULL);
         g_setenv("PATH", path, TRUE);
        g_free(path);
 }
@@ -537,6 +641,24 @@ static void keys_save(void)
        g_free(path);
 }
 
+static void marks_load(void)
+{
+       gchar *path;
+
+       path = g_build_filename(get_rc_dir(), RC_MARKS_NAME, NULL);
+       marks_list_load(path);
+       g_free(path);
+}
+
+static void marks_save(gboolean save)
+{
+       gchar *path;
+
+       path = g_build_filename(get_rc_dir(), RC_MARKS_NAME, NULL);
+       marks_list_save(path, save);
+       g_free(path);
+}
+
 static void mkdir_if_not_exists(const gchar *path)
 {
        if (isdir(path)) return;
@@ -668,6 +790,8 @@ static void gtkrc_load(void)
 static void exit_program_final(void)
 {
        LayoutWindow *lw = NULL;
+       GList *list;
+       LayoutWindow *tmp_lw;
 
         /* make sure that external editors are loaded, we would save incomplete configuration otherwise */
        layout_editors_reload_finish();
@@ -676,6 +800,21 @@ static void exit_program_final(void)
 
        collect_manager_flush();
 
+       /* Save the named windows */
+       if (layout_window_list && layout_window_list->next)
+               {
+               list = layout_window_list;
+               while (list)
+                       {
+                       tmp_lw = list->data;
+                       if (!g_str_has_prefix(tmp_lw->options.id, "lw"))
+                               {
+                               save_layout(list->data);
+                               }
+                       list = list->next;
+                       }
+               }
+
        save_options(options);
        keys_save();
        accel_map_save();
@@ -753,6 +892,8 @@ void exit_program(void)
 
        if (metadata_write_queue_confirm(FALSE, exit_program_write_metadata_cb, NULL)) return;
 
+       options->marks_save ? marks_save(TRUE) : marks_save(FALSE);
+
        if (exit_confirm_dlg()) return;
 
        exit_program_final();
@@ -766,7 +907,7 @@ void exit_program(void)
  * crash otherwise.
  * Ideas for improvement are welcome ;)
  */
-/* FIXME: this probably needs some better ifdefs. Please report any compilation problems */
+/** @FIXME this probably needs some better ifdefs. Please report any compilation problems */
 
 #if defined(SIGBUS) && defined(SA_SIGINFO)
 static void sigbus_handler_cb(int signum, siginfo_t *info, void *context)
@@ -789,12 +930,54 @@ static void setup_sigbus_handler(void)
 #endif
 }
 
+/**
+ * @brief Set up the application paths
+ * 
+ * This function is required for use of AppImages. AppImages are
+ * relocatable, and therefore cannot use fixed paths to various components.
+ * These paths were originally #defines created during compilation.
+ * They are now variables, all defined relative to one level above the
+ * directory that the executable is run from.
+ */
+static void create_application_paths()
+{
+       gchar buf[1024];
+       gchar *dirname;
+       gchar *basename;
+       gchar *tmp;
+
+       memset(buf, 0, sizeof(buf));
+       if (readlink("/proc/self/exe", buf, sizeof(buf) - 1) < 0)
+               {
+               /* There was an error. Perhaps the path does not exist
+                * or the buffer is not big enough. */
+               log_printf("Can't get path from /proc/self/exe");
+               exit(1);
+               }
+
+       dirname = g_path_get_dirname(buf); // default is /usr/bin/
+       gq_prefix = g_path_get_dirname(dirname);
+
+       gq_localedir = g_build_filename(gq_prefix, "share", "locale", NULL);
+       tmp = g_build_filename(gq_prefix, "share", "doc", NULL);
+       gq_helpdir = g_strconcat(tmp, G_DIR_SEPARATOR_S, "geeqie-", VERSION, NULL);
+       gq_htmldir = g_build_filename(gq_helpdir, "html", NULL);
+       gq_app_dir = g_build_filename(gq_prefix, "share", "geeqie", NULL);
+       gq_bin_dir = g_build_filename(gq_prefix, "lib", "geeqie", NULL);
+       desktop_file_template = g_build_filename(gq_app_dir, "template.desktop", NULL);
+
+       g_free(tmp);
+       g_free(dirname);
+}
+
 gint main(gint argc, gchar *argv[])
 {
        CollectionData *first_collection = NULL;
        gchar *buf;
        CollectionData *cd = NULL;
-       gchar *app_lock;
+       gboolean disable_clutter = FALSE;
+       gboolean single_dir = TRUE;
+       LayoutWindow *lw;
 
 #ifdef HAVE_GTHREAD
 #if !GLIB_CHECK_VERSION(2,32,0)
@@ -808,11 +991,13 @@ gint main(gint argc, gchar *argv[])
        /* init execution time counter (debug only) */
        init_exec_time();
 
+       create_application_paths();
+
        /* setup locale, i18n */
        setlocale(LC_ALL, "");
 
 #ifdef ENABLE_NLS
-       bindtextdomain(PACKAGE, GQ_LOCALEDIR);
+       bindtextdomain(PACKAGE, gq_localedir);
        bind_textdomain_codeset(PACKAGE, "UTF-8");
        textdomain(PACKAGE);
 #endif
@@ -841,10 +1026,19 @@ gint main(gint argc, gchar *argv[])
        parse_command_line_for_debug_option(argc, argv);
        DEBUG_1("%s main: gtk_init", get_exec_time());
 #ifdef HAVE_CLUTTER
-       if (gtk_clutter_init(&argc, &argv) != CLUTTER_INIT_SUCCESS)
+       if (parse_command_line_for_clutter_option(argc, argv))
                {
-               log_printf("Can't initialize clutter-gtk.\n");
-               exit(1);
+               disable_clutter = TRUE;
+               gtk_init(&argc, &argv);
+               }
+       else
+               {
+               if (gtk_clutter_init(&argc, &argv) != CLUTTER_INIT_SUCCESS)
+                       {
+                       log_printf("Can't initialize clutter-gtk.\nStart Geeqie with the option \"geeqie --disable-clutter\"");
+                       runcmd("zenity --error --title=\"Geeqie\" --text \"Can't initialize clutter-gtk.\n\nStart Geeqie with the option:\n geeqie --disable-clutter\" --width=300");
+                       exit(1);
+                       }
                }
 #else
        gtk_init(&argc, &argv);
@@ -866,6 +1060,10 @@ gint main(gint argc, gchar *argv[])
        DEBUG_1("%s main: setting default options before commandline handling", get_exec_time());
        options = init_options(NULL);
        setup_default_options(options);
+       if (disable_clutter)
+               {
+               options->disable_gpu = TRUE;
+               }
 
        DEBUG_1("%s main: parse_command_line", get_exec_time());
        parse_command_line(argc, argv);
@@ -876,6 +1074,7 @@ gint main(gint argc, gchar *argv[])
        mkdir_if_not_exists(get_collections_dir());
        mkdir_if_not_exists(get_thumbnails_cache_dir());
        mkdir_if_not_exists(get_metadata_cache_dir());
+       mkdir_if_not_exists(get_window_layouts_dir());
 
        setup_env_path();
 
@@ -893,6 +1092,23 @@ gint main(gint argc, gchar *argv[])
                filter_rebuild();
                }
 
+#ifdef HAVE_CLUTTER
+/** @FIXME For the background of this see:
+ * https://github.com/BestImageViewer/geeqie/issues/397
+ * The feature CLUTTER_FEATURE_SWAP_EVENTS indictates if the
+ * system is liable to exhibit this problem.
+ * The user is provided with an override in Preferences/Behavior
+ */
+       if (!options->override_disable_gpu && !options->disable_gpu)
+               {
+               DEBUG_1("CLUTTER_FEATURE_SWAP_EVENTS %d",clutter_feature_available(CLUTTER_FEATURE_SWAP_EVENTS));
+               if (clutter_feature_available(CLUTTER_FEATURE_SWAP_EVENTS) != 0)
+                       {
+                       options->disable_gpu = TRUE;
+                       }
+               }
+#endif
+
        /* handle missing config file and commandline additions*/
        if (!layout_window_list)
                {
@@ -902,6 +1118,9 @@ gint main(gint argc, gchar *argv[])
 
        layout_editors_reload_start();
 
+       /* If no --list option, open a separate collection window for each
+        * .gqv file on the command line
+        */
        if (command_line->collection_list && !command_line->startup_command_line_collection)
                {
                GList *work;
@@ -929,34 +1148,59 @@ gint main(gint argc, gchar *argv[])
                command_line->ssi = secure_open(pathl);
                }
 
-       if (command_line->cmd_list ||
-           (command_line->startup_command_line_collection && command_line->collection_list))
+       /* If there is a files list on the command line and no --list option,
+        * check if they are all in the same folder
+        */
+       if (command_line->cmd_list && !(command_line->startup_command_line_collection))
                {
                GList *work;
+               gchar *path = NULL;
 
-               if (command_line->startup_command_line_collection)
-                       {
-                       CollectWindow *cw;
+               work = command_line->cmd_list;
 
-                       cw = collection_window_new("");
-                       cd = cw->cd;
-                       }
-               else
+               while (work && single_dir)
                        {
-                       cd = collection_new("");        /* if we pass NULL, untitled counter is falsely increm. */
+                       gchar *dirname;
+
+                       dirname = g_path_get_dirname(work->data);
+                       if (!path)
+                               {
+                               path = g_strdup(dirname);
+                               }
+                       else
+                               {
+                               if (g_strcmp0(path, dirname) != 0)
+                                       {
+                                       single_dir = FALSE;
+                                       }
+                               }
+                       g_free(dirname);
+                       work = work->next;
                        }
+               g_free(path);
+               }
+
+       /* Files from multiple folders, or --list option given
+        * then open an unnamed collection and insert all files
+        */
+       if ((command_line->cmd_list && !single_dir) || (command_line->startup_command_line_collection && command_line->cmd_list))
+               {
+               GList *work;
+               CollectWindow *cw;
 
-               g_free(cd->path);
-               cd->path = NULL;
-               g_free(cd->name);
-               cd->name = g_strdup(_("Command line"));
+               cw = collection_window_new(NULL);
+               cd = cw->cd;
 
                collection_path_changed(cd);
 
                work = command_line->cmd_list;
                while (work)
                        {
-                       collection_add(cd, (FileData *)work->data, FALSE);
+                       FileData *fd;
+
+                       fd = file_data_new_simple(work->data);
+                       collection_add(cd, fd, FALSE);
+                       file_data_unref(fd);
                        work = work->next;
                        }
 
@@ -984,10 +1228,36 @@ gint main(gint argc, gchar *argv[])
                                            collection_get_first(first_collection));
                }
 
+       /* If the files on the command line are from one folder, select those files
+        * unless it is a command line collection - then leave focus on collection window
+        */
+       lw = NULL;
+       layout_valid(&lw);
+
+       if (single_dir && command_line->cmd_list && !command_line->startup_command_line_collection)
+               {
+               GList *work;
+               GList *selected;
+               FileData *fd;
+
+               selected = NULL;
+               work = command_line->cmd_list;
+               while (work)
+                       {
+                       fd = file_data_new_simple((gchar *)work->data);
+                       selected = g_list_append(selected, fd);
+                       file_data_unref(fd);
+                       work = work->next;
+                       }
+               layout_select_list(lw, selected);
+               }
+
        buf = g_build_filename(get_rc_dir(), ".command", NULL);
        remote_connection = remote_server_init(buf, cd);
        g_free(buf);
 
+       marks_load();
+
        DEBUG_1("%s main: gtk_main", get_exec_time());
        gtk_main();
 #ifdef HAVE_GTHREAD