Fix #684: incompatible pointer types passing struct sockaddr_un
[geeqie.git] / src / remote.c
index c5ecfdc..ac1dac9 100644 (file)
@@ -25,6 +25,7 @@
 #include "cache_maint.h"
 #include "collect.h"
 #include "collect-io.h"
+#include "exif.h"
 #include "filedata.h"
 #include "filefilter.h"
 #include "image.h"
@@ -37,6 +38,7 @@
 #include "slideshow.h"
 #include "ui_fileops.h"
 #include "rcfile.h"
+#include "view_file.h"
 
 #include <sys/socket.h>
 #include <sys/un.h>
@@ -71,8 +73,16 @@ struct _RemoteClient {
 typedef struct _RemoteData RemoteData;
 struct _RemoteData {
        CollectionData *command_collection;
+       GList *file_list;
+       gboolean single_dir;
 };
 
+/* To enable file names containing newlines to be processed correctly,
+ * the --print0 remote option sets returned data to be terminated with a null
+ * character rather a newline
+ */
+static gboolean print0 = FALSE;
+
 /* Remote commands from main.c are prepended with the current dir the remote
  * command was made from. Some remote commands require this. The
  * value is stored here
@@ -120,7 +130,8 @@ static gboolean remote_server_client_cb(GIOChannel *source, GIOCondition conditi
                gchar *buffer = NULL;
                GError *error = NULL;
                gsize termpos;
-
+               /** @FIXME it should be possible to terminate the command with a null character */
+               g_io_channel_set_line_term(source, "<gq_end_of_command>", -1);
                while ((status = g_io_channel_read_line(source, &buffer, NULL, &termpos, &error)) == G_IO_STATUS_NORMAL)
                        {
                        if (buffer)
@@ -130,7 +141,7 @@ static gboolean remote_server_client_cb(GIOChannel *source, GIOCondition conditi
                                if (strlen(buffer) > 0)
                                        {
                                        if (rc->read_func) rc->read_func(rc, buffer, source, rc->read_data);
-                                       g_io_channel_write_chars(source, "\n", -1, NULL, NULL); /* empty line finishes the command */
+                                       g_io_channel_write_chars(source, "<gq_end_of_command>", -1, NULL, NULL); /* empty line finishes the command */
                                        g_io_channel_flush(source, NULL);
                                        }
                                g_free(buffer);
@@ -253,7 +264,7 @@ static RemoteConnection *remote_server_open(const gchar *path)
        addr.sun_family = AF_UNIX;
        sun_path_len = MIN(strlen(path) + 1, UNIX_PATH_MAX);
        strncpy(addr.sun_path, path, sun_path_len);
-       if (bind(fd, &addr, sizeof(addr)) == -1 ||
+       if (bind(fd, (const struct sockaddr*)&addr, sizeof(addr)) == -1 ||
            listen(fd, REMOTE_SERVER_BACKLOG) == -1)
                {
                log_printf("error subscribing to socket: %s\n", strerror(errno));
@@ -302,7 +313,7 @@ static RemoteConnection *remote_client_open(const gchar *path)
        addr.sun_family = AF_UNIX;
        sun_path_len = MIN(strlen(path) + 1, UNIX_PATH_MAX);
        strncpy(addr.sun_path, path, sun_path_len);
-       if (connect(fd, &addr, sizeof(addr)) == -1)
+       if (connect(fd, (struct sockaddr*)&addr, sizeof(addr)) == -1)
                {
                DEBUG_1("error connecting to socket: %s", strerror(errno));
                close(fd);
@@ -346,7 +357,7 @@ static gboolean remote_client_send(RemoteConnection *rc, const gchar *text)
        channel = g_io_channel_unix_new(rc->fd);
 
        g_io_channel_write_chars(channel, text, -1, NULL, &error);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, &error);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, &error);
        g_io_channel_flush(channel, &error);
 
        if (error)
@@ -364,18 +375,33 @@ static gboolean remote_client_send(RemoteConnection *rc, const gchar *text)
                {
                gchar *buffer = NULL;
                gsize termpos;
+               g_io_channel_set_line_term(channel, "<gq_end_of_command>", -1);
                while (g_io_channel_read_line(channel, &buffer, NULL, &termpos, &error) == G_IO_STATUS_NORMAL)
                        {
                        if (buffer)
                                {
-                               if (buffer[0] == '\n') /* empty line finishes the command */
+                               if (g_strstr_len(buffer, -1, "<gq_end_of_command>") == buffer) /* empty line finishes the command */
                                        {
                                        g_free(buffer);
                                        fflush(stdout);
                                        break;
                                        }
                                buffer[termpos] = '\0';
-                               printf("%s\n", buffer);
+                               if (g_strstr_len(buffer, -1, "print0") != 0)
+                                       {
+                                       print0 = TRUE;
+                                       }
+                               else
+                                       {
+                                       if (print0)
+                                               {
+                                               printf("%s%c", buffer, 0);
+                                               }
+                                       else
+                                               {
+                                               printf("%s\n", buffer);
+                                               }
+                                       }
                                g_free(buffer);
                                buffer = NULL;
                                }
@@ -434,7 +460,9 @@ static void gr_new_window(const gchar *text, GIOChannel *channel, gpointer data)
 
        if (!layout_valid(&lw)) return;
 
-       lw_id = layout_menu_new_window(NULL, lw);
+       lw_id = layout_new_from_default();
+
+       layout_set_path(lw_id, pwd);
 }
 
 static gboolean gr_close_window_cb()
@@ -494,8 +522,15 @@ static void gr_lw_id(const gchar *text, GIOChannel *channel, gpointer data)
 static void gr_slideshow_start_rec(const gchar *text, GIOChannel *channel, gpointer data)
 {
        GList *list;
-       FileData *dir_fd = file_data_new_dir(text);
-       list = filelist_recursive(dir_fd);
+       gchar *tilde_filename;
+
+       tilde_filename = expand_tilde(text);
+
+       FileData *dir_fd = file_data_new_dir(tilde_filename);
+       g_free(tilde_filename);
+
+       layout_valid(&lw_id);
+       list = filelist_recursive_full(dir_fd, lw_id->sort_method, lw_id->sort_ascend);
        file_data_unref(dir_fd);
        if (!list) return;
 //printf("length: %d\n", g_list_length(list));
@@ -506,9 +541,13 @@ static void gr_slideshow_start_rec(const gchar *text, GIOChannel *channel, gpoin
 static void gr_cache_thumb(const gchar *text, GIOChannel *channel, gpointer data)
 {
        if (!g_strcmp0(text, "clear"))
-               cache_maintain_home_remote(FALSE, TRUE);
+               {
+               cache_maintain_home_remote(FALSE, TRUE, NULL);
+               }
        else if (!g_strcmp0(text, "clean"))
-               cache_maintain_home_remote(FALSE, FALSE);
+               {
+               cache_maintain_home_remote(FALSE, FALSE, NULL);
+               }
 }
 
 static void gr_cache_shared(const gchar *text, GIOChannel *channel, gpointer data)
@@ -521,29 +560,33 @@ static void gr_cache_shared(const gchar *text, GIOChannel *channel, gpointer dat
 
 static void gr_cache_metadata(const gchar *text, GIOChannel *channel, gpointer data)
 {
-       cache_maintain_home_remote(TRUE, FALSE);
+       cache_maintain_home_remote(TRUE, FALSE, NULL);
 }
 
 static void gr_cache_render(const gchar *text, GIOChannel *channel, gpointer data)
 {
-       cache_manager_render_remote(text, FALSE, FALSE);
+       cache_manager_render_remote(text, FALSE, FALSE, NULL);
 }
 
 static void gr_cache_render_recurse(const gchar *text, GIOChannel *channel, gpointer data)
 {
-       cache_manager_render_remote(text, TRUE, FALSE);
+       cache_manager_render_remote(text, TRUE, FALSE, NULL);
 }
 
 static void gr_cache_render_standard(const gchar *text, GIOChannel *channel, gpointer data)
 {
        if(options->thumbnails.spec_standard)
-               cache_manager_render_remote(text, FALSE, TRUE);
+               {
+               cache_manager_render_remote(text, FALSE, TRUE, NULL);
+               }
 }
 
 static void gr_cache_render_standard_recurse(const gchar *text, GIOChannel *channel, gpointer data)
 {
        if(options->thumbnails.spec_standard)
-               cache_manager_render_remote(text, TRUE, TRUE);
+               {
+               cache_manager_render_remote(text, TRUE, TRUE, NULL);
+               }
 }
 
 static void gr_slideshow_toggle(const gchar *text, GIOChannel *channel, gpointer data)
@@ -647,33 +690,37 @@ static void gr_quit(const gchar *text, GIOChannel *channel, gpointer data)
 static void gr_file_load_no_raise(const gchar *text, GIOChannel *channel, gpointer data)
 {
        gchar *filename;
-       gchar *tilde_filename = expand_tilde(text);
+       gchar *tilde_filename;
 
-       filename = set_pwd(tilde_filename);
-
-       if (isfile(filename))
+       if (!download_web_file(text, TRUE, NULL))
                {
-               if (file_extension_match(filename, GQ_COLLECTION_EXT))
+               tilde_filename = expand_tilde(text);
+               filename = set_pwd(tilde_filename);
+
+               if (isfile(filename))
+                       {
+                       if (file_extension_match(filename, GQ_COLLECTION_EXT))
+                               {
+                               collection_window_new(filename);
+                               }
+                       else
+                               {
+                               layout_set_path(lw_id, filename);
+                               }
+                       }
+               else if (isdir(filename))
                        {
-                       collection_window_new(filename);
+                       layout_set_path(lw_id, filename);
                        }
                else
                        {
-                       layout_set_path(lw_id, filename);
+                       log_printf("remote sent filename that does not exist:\"%s\"\n", filename);
+                       layout_set_path(lw_id, homedir());
                        }
-               }
-       else if (isdir(filename))
-               {
-               layout_set_path(lw_id, filename);
-               }
-       else
-               {
-               log_printf("remote sent filename that does not exist:\"%s\"\n", filename);
-               layout_set_path(lw_id, homedir());
-               }
 
-       g_free(filename);
-       g_free(tilde_filename);
+               g_free(filename);
+               g_free(tilde_filename);
+               }
 }
 
 static void gr_file_load(const gchar *text, GIOChannel *channel, gpointer data)
@@ -690,7 +737,6 @@ static void gr_pixel_info(const gchar *text, GIOChannel *channel, gpointer data)
        gint width, height;
        gint r_mouse, g_mouse, b_mouse;
        PixbufRenderer *pr;
-       LayoutWindow *lw = NULL;
 
        if (!layout_valid(&lw_id)) return;
 
@@ -713,7 +759,7 @@ static void gr_pixel_info(const gchar *text, GIOChannel *channel, gpointer data)
                                                 r_mouse, g_mouse, b_mouse);
 
                        g_io_channel_write_chars(channel, pixel_info, -1, NULL, NULL);
-                       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+                       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
                        g_free(pixel_info);
                        }
@@ -732,7 +778,6 @@ static void gr_rectangle(const gchar *text, GIOChannel *channel, gpointer data)
 {
        gchar *rectangle_info;
        PixbufRenderer *pr;
-       LayoutWindow *lw = NULL;
        gint x1, y1, x2, y2;
 
        if (!options->draw_rectangle) return;
@@ -750,7 +795,7 @@ static void gr_rectangle(const gchar *text, GIOChannel *channel, gpointer data)
                                        (y2 > y1) ? y1 : y2);
 
                g_io_channel_write_chars(channel, rectangle_info, -1, NULL, NULL);
-               g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+               g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
                g_free(rectangle_info);
                }
@@ -780,7 +825,7 @@ static void gr_render_intent(const gchar *text, GIOChannel *channel, gpointer da
                }
 
        g_io_channel_write_chars(channel, render_intent, -1, NULL, NULL);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        g_free(render_intent);
 }
@@ -793,6 +838,7 @@ static void get_filelist(const gchar *text, GIOChannel *channel, gboolean recurs
        FileData *fd;
        GString *out_string = g_string_new(NULL);
        GList *work;
+       gchar *tilde_filename;
 
        if (strcmp(text, "") == 0)
                {
@@ -807,14 +853,17 @@ static void get_filelist(const gchar *text, GIOChannel *channel, gboolean recurs
                }
        else
                {
-               if (isdir(text))
+               tilde_filename = expand_tilde(text);
+               if (isdir(tilde_filename))
                        {
-                       dir_fd = file_data_new_dir(text);
+                       dir_fd = file_data_new_dir(tilde_filename);
                        }
                else
                        {
+                       g_free(tilde_filename);
                        return;
                        }
+               g_free(tilde_filename);
                }
 
        if (recurse)
@@ -850,8 +899,11 @@ static void get_filelist(const gchar *text, GIOChannel *channel, gboolean recurs
                        case FORMAT_CLASS_COLLECTION:
                                out_string = g_string_append(out_string, "    Class: Collection");
                                break;
-                       case FORMAT_CLASS_PDF:
-                               out_string = g_string_append(out_string, "    Class: PDF");
+                       case FORMAT_CLASS_DOCUMENT:
+                               out_string = g_string_append(out_string, "    Class: Document");
+                               break;
+                       case FORMAT_CLASS_ARCHIVE:
+                               out_string = g_string_append(out_string, "    Class: Archive");
                                break;
                        case FORMAT_CLASS_UNKNOWN:
                                out_string = g_string_append(out_string, "    Class: Unknown");
@@ -865,7 +917,7 @@ static void get_filelist(const gchar *text, GIOChannel *channel, gboolean recurs
                }
 
        g_io_channel_write_chars(channel, out_string->str, -1, NULL, NULL);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        g_string_free(out_string, TRUE);
        filelist_free(list);
@@ -886,7 +938,7 @@ static void gr_collection(const gchar *text, GIOChannel *channel, gpointer data)
                }
 
        g_io_channel_write_chars(channel, contents->str, -1, NULL, NULL);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        g_string_free(contents, TRUE);
 }
@@ -911,12 +963,55 @@ static void gr_collection_list(const gchar *text, GIOChannel *channel, gpointer
                }
 
        g_io_channel_write_chars(channel, out_string->str, -1, NULL, NULL);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        string_list_free(collection_list);
        g_string_free(out_string, TRUE);
 }
 
+static gboolean wait_cb(const gpointer data)
+{
+       gint position = GPOINTER_TO_INT(data);
+       gint x = position >> 16;
+       gint y = position - (x << 16);
+
+       gtk_window_move(GTK_WINDOW(lw_id->window), x, y);
+
+       return FALSE;
+}
+
+static void gr_geometry(const gchar *text, GIOChannel *channel, gpointer data)
+{
+       gchar **geometry;
+
+       if (!layout_valid(&lw_id) || !text)
+               {
+               return;
+               }
+
+       if (text[0] == '+')
+               {
+               geometry = g_strsplit_set(text, "+", 3);
+               if (geometry[1] != NULL && geometry[2] != NULL )
+                       {
+                       gtk_window_move(GTK_WINDOW(lw_id->window), atoi(geometry[1]), atoi(geometry[2]));
+                       }
+               }
+       else
+               {
+               geometry = g_strsplit_set(text, "+x", 4);
+               if (geometry[0] != NULL && geometry[1] != NULL)
+                       {
+                       gtk_window_resize(GTK_WINDOW(lw_id->window), atoi(geometry[0]), atoi(geometry[1]));
+                       }
+               if (geometry[2] != NULL && geometry[3] != NULL)
+                       {
+                       /* There is an occasional problem with a window_move immediately after a window_resize */
+                       g_idle_add(wait_cb, GINT_TO_POINTER((atoi(geometry[2]) << 16) + atoi(geometry[3])));
+                       }
+               }
+       g_strfreev(geometry);
+}
 
 static void gr_filelist(const gchar *text, GIOChannel *channel, gpointer data)
 {
@@ -946,19 +1041,141 @@ static void gr_file_tell(const gchar *text, GIOChannel *channel, gpointer data)
                        {
                        out_string = g_strconcat(image_get_path(lw_id->image), NULL);
                        }
+               }
+       else
+               {
+               out_string = g_strconcat(lw_id->dir_fd->path, G_DIR_SEPARATOR_S, NULL);
+               }
+
+       g_io_channel_write_chars(channel, out_string, -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
+
+       g_free(collection_name);
+       g_free(out_string);
+}
+
+static void gr_file_info(const gchar *text, GIOChannel *channel, gpointer data)
+{
+       gchar *filename;
+       FileData *fd;
+       gchar *country_name;
+       gchar *country_code;
+       gchar *timezone;
+       gchar *local_time;
+       GString *out_string;
+       FileFormatClass format_class;
+
+       if (!layout_valid(&lw_id)) return;
+
+       if (image_get_path(lw_id->image))
+               {
+               filename = g_strdup(image_get_path(lw_id->image));
+               fd = file_data_new_group(filename);
+               out_string = g_string_new(NULL);
+
+               format_class = filter_file_get_class(image_get_path(lw_id->image));
+               if (format_class)
+                       {
+                       g_string_append_printf(out_string, _("Class: %s\n"), format_class_list[format_class]);
+                       }
+
+               if (fd->page_total > 1)
+                       {
+                       g_string_append_printf(out_string, _("Page no: %d/%d\n"), fd->page_num + 1, fd->page_total);
+                       }
+
+               if (fd->exif)
+                       {
+                       country_name = exif_get_data_as_text(fd->exif, "formatted.countryname");
+                       if (country_name)
+                               {
+                               g_string_append_printf(out_string, _("Country name: %s\n"), country_name);
+                               g_free(country_name);
+                               }
+
+                       country_code = exif_get_data_as_text(fd->exif, "formatted.countrycode");
+                       if (country_name)
+                               {
+                               g_string_append_printf(out_string, _("Country code: %s\n"), country_code);
+                               g_free(country_code);
+                               }
+
+                       timezone = exif_get_data_as_text(fd->exif, "formatted.timezone");
+                       if (timezone)
+                               {
+                               g_string_append_printf(out_string, _("Timezone: %s\n"), timezone);
+                               g_free(timezone);
+                               }
+
+                       local_time = exif_get_data_as_text(fd->exif, "formatted.localtime");
+                       if (local_time)
+                               {
+                               g_string_append_printf(out_string, ("Local time: %s\n"), local_time);
+                               g_free(local_time);
+                               }
+                       }
+
+               g_io_channel_write_chars(channel, out_string->str, -1, NULL, NULL);
+               g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
+
+               g_string_free(out_string, TRUE);
+               file_data_unref(fd);
+               g_free(filename);
+               }
+}
+
+static gchar *config_file_path(const gchar *param)
+{
+       gchar *path = NULL;
+       gchar *full_name = NULL;
+
+       if (file_extension_match(param, ".xml"))
+               {
+               path = g_build_filename(get_window_layouts_dir(), param, NULL);
+               }
+       else if (file_extension_match(param, NULL))
+               {
+               full_name = g_strconcat(param, ".xml", NULL);
+               path = g_build_filename(get_window_layouts_dir(), full_name, NULL);
+               }
+
+       if (!isfile(path))
+               {
+               g_free(path);
+               path = NULL;
+               }
 
-               g_io_channel_write_chars(channel, out_string, -1, NULL, NULL);
-               g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_free(full_name);
+       return path;
+}
 
-               g_free(collection_name);
-               g_free(out_string);
+static gboolean is_config_file(const gchar *param)
+{
+       gchar *name = NULL;
+
+       name = config_file_path(param);
+       if (name)
+               {
+               g_free(name);
+               return TRUE;
                }
+       return FALSE;
 }
 
 static void gr_config_load(const gchar *text, GIOChannel *channel, gpointer data)
 {
        gchar *filename = expand_tilde(text);
 
+       if (!g_strstr_len(filename, -1, G_DIR_SEPARATOR_S))
+               {
+               if (is_config_file(filename))
+                       {
+                       gchar *tmp = config_file_path(filename);
+                       g_free(filename);
+                       filename = tmp;
+                       }
+               }
+
        if (isfile(filename))
                {
                load_config_from_file(filename, FALSE);
@@ -981,7 +1198,7 @@ static void gr_get_sidecars(const gchar *text, GIOChannel *channel, gpointer dat
        if (fd->parent) fd = fd->parent;
 
        g_io_channel_write_chars(channel, fd->path, -1, NULL, NULL);
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        work = fd->sidecar_files;
 
@@ -990,7 +1207,7 @@ static void gr_get_sidecars(const gchar *text, GIOChannel *channel, gpointer dat
                fd = work->data;
                work = work->next;
                g_io_channel_write_chars(channel, fd->path, -1, NULL, NULL);
-               g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+               g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
                }
        g_free(filename);
 }
@@ -1003,7 +1220,7 @@ static void gr_get_destination(const gchar *text, GIOChannel *channel, gpointer
        if (fd->change && fd->change->dest)
                {
                g_io_channel_write_chars(channel, fd->change->dest, -1, NULL, NULL);
-               g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+               g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
                }
        g_free(filename);
 }
@@ -1024,47 +1241,92 @@ static void gr_list_clear(const gchar *text, GIOChannel *channel, gpointer data)
 {
        RemoteData *remote_data = data;
 
-       if (remote_data->command_collection)
-               {
-               collection_unref(remote_data->command_collection);
-               remote_data->command_collection = NULL;
-               }
+       remote_data->command_collection = NULL;
+       remote_data->file_list = NULL;
+       remote_data->single_dir = TRUE;
 }
 
 static void gr_list_add(const gchar *text, GIOChannel *channel, gpointer data)
 {
        RemoteData *remote_data = data;
        gboolean new = TRUE;
+       gchar *path = NULL;
+       FileData *fd;
+       FileData *first;
 
-       if (!remote_data->command_collection)
+       /* If there is a files list on the command line
+        * check if they are all in the same folder
+        */
+       if (remote_data->single_dir)
                {
+               GList *work;
+               work = remote_data->file_list;
+               while (work && remote_data->single_dir)
+                       {
+                       gchar *dirname;
+                       dirname = g_path_get_dirname(((FileData *)work->data)->path);
+                       if (!path)
+                               {
+                               path = g_strdup(dirname);
+                               }
+                       else
+                               {
+                               if (g_strcmp0(path, dirname) != 0)
+                                       {
+                                       remote_data->single_dir = FALSE;
+                                       }
+                               }
+                       g_free(dirname);
+                       work = work->next;
+                       }
+               g_free(path);
+               }
+
+       gchar *pathname = g_path_get_dirname(text);
+       layout_set_path(lw_id, pathname);
+       g_free(pathname);
+
+       fd = file_data_new_simple(text);
+       remote_data->file_list = g_list_append(remote_data->file_list, fd);
+       file_data_unref(fd);
+
+       vf_select_none(lw_id->vf);
+       remote_data->file_list = g_list_reverse(remote_data->file_list);
+
+       layout_select_list(lw_id, remote_data->file_list);
+       layout_refresh(lw_id);
+       first = (FileData *)(g_list_first(vf_selection_get_list(lw_id->vf))->data);
+       layout_set_fd(lw_id, first);
+
                CollectionData *cd;
+               CollectWindow *cw;
 
-               cd = collection_new("");
+       if (!remote_data->command_collection && !remote_data->single_dir)
+               {
+               cw = collection_window_new(NULL);
+               cd = cw->cd;
 
-               g_free(cd->path);
-               cd->path = NULL;
-               g_free(cd->name);
-               cd->name = g_strdup(_("Command line"));
+               collection_path_changed(cd);
 
                remote_data->command_collection = cd;
                }
-       else
+       else if (!remote_data->single_dir)
                {
                new = (!collection_get_first(remote_data->command_collection));
                }
 
-       if (collection_add(remote_data->command_collection, file_data_new_group(text), FALSE) && new)
+       if (!remote_data->single_dir)
                {
-               layout_image_set_collection(NULL, remote_data->command_collection,
-                                           collection_get_first(remote_data->command_collection));
+               layout_image_set_collection(lw_id, remote_data->command_collection, collection_get_first(remote_data->command_collection));
+               if (collection_add(remote_data->command_collection, file_data_new_group(text), FALSE) && new)
+                       {
+                       layout_image_set_collection(lw_id, remote_data->command_collection, collection_get_first(remote_data->command_collection));
+                       }
                }
 }
 
 static void gr_raise(const gchar *text, GIOChannel *channel, gpointer data)
 {
-       LayoutWindow *lw = NULL;
-
        if (layout_valid(&lw_id))
                {
                gtk_window_present(GTK_WINDOW(lw_id->window));
@@ -1075,8 +1337,17 @@ static void gr_pwd(const gchar *text, GIOChannel *channel, gpointer data)
 {
        LayoutWindow *lw = NULL;
 
+       layout_valid(&lw);
+
        g_free(pwd);
        pwd = g_strdup(text);
+       lw_id = lw;
+}
+
+static void gr_print0(const gchar *text, GIOChannel *channel, gpointer data)
+{
+       g_io_channel_write_chars(channel, "print0", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 }
 
 #ifdef HAVE_LUA
@@ -1105,7 +1376,7 @@ static void gr_lua(const gchar *text, GIOChannel *channel, gpointer data)
                g_io_channel_write_chars(channel, N_("lua error: no data"), -1, NULL, NULL);
                }
 
-       g_io_channel_write_chars(channel, "\n", -1, NULL, NULL);
+       g_io_channel_write_chars(channel, "<gq_end_of_command>", -1, NULL, NULL);
 
        g_strfreev(lua_command);
        g_free(result);
@@ -1125,56 +1396,59 @@ struct _RemoteCommandEntry {
 
 static RemoteCommandEntry remote_commands[] = {
        /* short, long                  callback,               extra, prefer, parameter, description */
-       { "-n", "--next",               gr_image_next,          FALSE, FALSE, NULL, N_("next image") },
        { "-b", "--back",               gr_image_prev,          FALSE, FALSE, NULL, N_("previous image") },
+       { NULL, "--close-window",       gr_close_window,        FALSE, FALSE, NULL, N_("close window") },
+       { NULL, "--config-load: ",       gr_config_load,         TRUE,  FALSE, N_("<FILE>|layout ID"), N_(" load configuration from FILE") },
+       { "-cm","--cache-metadata",      gr_cache_metadata,               FALSE, FALSE, NULL, N_("clean the metadata cache") },
+       { "-cr:", "--cache-render:",    gr_cache_render,        TRUE, FALSE, N_("<folder>  "), N_(" render thumbnails") },
+       { "-crr:", "--cache-render-recurse:", gr_cache_render_recurse, TRUE, FALSE, N_("<folder> "), N_("render thumbnails recursively") },
+       { "-crs:", "--cache-render-shared:", gr_cache_render_standard, TRUE, FALSE, N_("<folder> "), N_(" render thumbnails (see Help)") },
+       { "-crsr:", "--cache-render-shared-recurse:", gr_cache_render_standard_recurse, TRUE, FALSE, N_("<folder>"), N_(" render thumbnails recursively (see Help)") },
+       { "-cs:", "--cache-shared:",    gr_cache_shared,        TRUE, FALSE, N_("clear|clean"), N_("clear or clean shared thumbnail cache") },
+       { "-ct:", "--cache-thumbs:",    gr_cache_thumb,         TRUE, FALSE, N_("clear|clean"), N_("clear or clean thumbnail cache") },
+       { "-d", "--delay=",             gr_slideshow_delay,     TRUE,  FALSE, N_("<[H:][M:][N][.M]>"), N_("set slide show delay to Hrs Mins N.M seconds") },
        { NULL, "--first",              gr_image_first,         FALSE, FALSE, NULL, N_("first image") },
-       { NULL, "--last",               gr_image_last,          FALSE, FALSE, NULL, N_("last image") },
        { "-f", "--fullscreen",         gr_fullscreen_toggle,   FALSE, TRUE,  NULL, N_("toggle full screen") },
+       { NULL, "--file:",              gr_file_load,           TRUE,  FALSE, N_("<FILE>|<URL>"), N_("open FILE or URL, bring Geeqie window to the top") },
+       { NULL, "file:",                gr_file_load,           TRUE,  FALSE, N_("<FILE>|<URL>"), N_("open FILE or URL, bring Geeqie window to the top") },
+       { NULL, "--File:",              gr_file_load_no_raise,  TRUE,  FALSE, N_("<FILE>|<URL>"), N_("open FILE or URL, do not bring Geeqie window to the top") },
+       { NULL, "File:",                gr_file_load_no_raise,  TRUE,  FALSE, N_("<FILE>|<URL>"), N_("open FILE or URL, do not bring Geeqie window to the top") },
        { "-fs","--fullscreen-start",   gr_fullscreen_start,    FALSE, FALSE, NULL, N_("start full screen") },
        { "-fS","--fullscreen-stop",    gr_fullscreen_stop,     FALSE, FALSE, NULL, N_("stop full screen") },
-       { "-s", "--slideshow",          gr_slideshow_toggle,    FALSE, TRUE,  NULL, N_("toggle slide show") },
-       { "-ss","--slideshow-start",    gr_slideshow_start,     FALSE, FALSE, NULL, N_("start slide show") },
-       { "-sS","--slideshow-stop",     gr_slideshow_stop,      FALSE, FALSE, NULL, N_("stop slide show") },
-       { NULL, "--slideshow-recurse:", gr_slideshow_start_rec, TRUE,  FALSE, N_("<FOLDER>"), N_("start recursive slide show in FOLDER") },
-       { "-d", "--delay=",             gr_slideshow_delay,     TRUE,  FALSE, N_("<[H:][M:][N][.M]>"), N_("set slide show delay to Hrs Mins N.M seconds") },
-       { "+t", "--tools-show",         gr_tools_show,          FALSE, TRUE,  NULL, N_("show tools") },
-       { "-t", "--tools-hide",         gr_tools_hide,          FALSE, TRUE,  NULL, N_("hide tools") },
-       { "-q", "--quit",               gr_quit,                FALSE, FALSE, NULL, N_("quit") },
-       { NULL, "--config-load:",       gr_config_load,         TRUE,  FALSE, N_("<FILE>"), N_("load configuration from FILE") },
-       { NULL, "--get-sidecars:",      gr_get_sidecars,        TRUE,  FALSE, N_("<FILE>"), N_("get list of sidecars of FILE") },
-       { NULL, "--get-destination:",   gr_get_destination,     TRUE,  FALSE, N_("<FILE>"), N_("get destination path of FILE") },
-       { NULL, "file:",                gr_file_load,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE, bring Geeqie window to the top") },
-       { NULL, "--file:",              gr_file_load,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE, bring Geeqie window to the top") },
-       { NULL, "File:",                gr_file_load_no_raise,  TRUE,  FALSE, N_("<FILE>"), N_("open FILE, do not bring Geeqie window to the top") },
-       { NULL, "--File:",              gr_file_load_no_raise,  TRUE,  FALSE, N_("<FILE>"), N_("open FILE, do not bring Geeqie window to the top") },
-       { NULL, "--tell",               gr_file_tell,           FALSE, FALSE, NULL, N_("print filename [and Collection] of current image") },
-       { NULL, "--pixel-info",         gr_pixel_info,          FALSE, FALSE, NULL, N_("print pixel info of mouse pointer on current image") },
-       { NULL, "--get-rectangle",      gr_rectangle,           FALSE, FALSE, NULL, N_("get rectangle co-ordinates") },
-       { NULL, "--get-render-intent",  gr_render_intent,       FALSE, FALSE, NULL, N_("get render intent") },
-       { NULL, "--get-filelist:",      gr_filelist,            TRUE,  FALSE, N_("[<FOLDER>]"), N_("get list of files and class") },
-       { NULL, "--get-filelist-recurse:", gr_filelist_recurse, TRUE,  FALSE, N_("[<FOLDER>]"), N_("get list of files and class recursive") },
+       { NULL, "--geometry=",          gr_geometry,            TRUE, FALSE, N_("<GEOMETRY>"), N_("set window geometry") },
        { NULL, "--get-collection:",    gr_collection,          TRUE,  FALSE, N_("<COLLECTION>"), N_("get collection content") },
        { NULL, "--get-collection-list", gr_collection_list,    FALSE, FALSE, NULL, N_("get collection list") },
-       { NULL, "view:",                gr_file_view,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE in new window") },
-       { NULL, "--view:",              gr_file_view,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE in new window") },
-       { NULL, "--list-clear",         gr_list_clear,          FALSE, FALSE, NULL, N_("clear command line collection list") },
-       { NULL, "--list-add:",          gr_list_add,            TRUE,  FALSE, N_("<FILE>"), N_("add FILE to command line collection list") },
-       { NULL, "raise",                gr_raise,               FALSE, FALSE, NULL, N_("bring the Geeqie window to the top") },
-       { NULL, "--raise",              gr_raise,               FALSE, FALSE, NULL, N_("bring the Geeqie window to the top") },
+       { NULL, "--get-destination:",   gr_get_destination,     TRUE,  FALSE, N_("<FILE>"), N_("get destination path of FILE (See Plugins Configuration)") },
+       { NULL, "--get-file-info",      gr_file_info,           FALSE, FALSE, NULL, N_("get file info") },
+       { NULL, "--get-filelist:",      gr_filelist,            TRUE,  FALSE, N_("[<FOLDER>]"), N_("get list of files and class") },
+       { NULL, "--get-filelist-recurse:", gr_filelist_recurse, TRUE,  FALSE, N_("[<FOLDER>]"), N_("get list of files and class recursive") },
+       { NULL, "--get-rectangle",      gr_rectangle,           FALSE, FALSE, NULL, N_("get rectangle co-ordinates") },
+       { NULL, "--get-render-intent",  gr_render_intent,       FALSE, FALSE, NULL, N_("get render intent") },
+       { NULL, "--get-sidecars:",      gr_get_sidecars,        TRUE,  FALSE, N_("<FILE>"), N_("get list of sidecars of FILE") },
        { NULL, "--id:",                gr_lw_id,               TRUE, FALSE, N_("<ID>"), N_("window id for following commands") },
-       { NULL, "--new-window",         gr_new_window,          FALSE, FALSE, NULL, N_("new window") },
-       { NULL, "--close-window",       gr_close_window,        FALSE, FALSE, NULL, N_("close window") },
-       { "-ct:", "--cache-thumbs:",    gr_cache_thumb,         TRUE, FALSE, N_("clear|clean"), N_("clear or clean thumbnail cache") },
-       { "-cs:", "--cache-shared:",    gr_cache_shared,        TRUE, FALSE, N_("clear|clean"), N_("clear or clean shared thumbnail cache") },
-       { "-cm","--cache-metadata",      gr_cache_metadata,               FALSE, FALSE, NULL, N_("    clean the metadata cache") },
-       { "-cr:", "--cache-render:",    gr_cache_render,        TRUE, FALSE, N_("<folder>  "), N_(" render thumbnails") },
-       { "-crr:", "--cache-render-recurse:", gr_cache_render_recurse, TRUE, FALSE, N_("<folder> "), N_("render thumbnails recursively") },
-       { "-crs:", "--cache-render-shared:", gr_cache_render_standard, TRUE, FALSE, N_("<folder> "), N_(" render thumbnails (see Help)") },
-       { "-crsr:", "--cache-render-shared-recurse:", gr_cache_render_standard_recurse, TRUE, FALSE, N_("<folder>"), N_(" render thumbnails recursively (see Help)") },
+       { NULL, "--last",               gr_image_last,          FALSE, FALSE, NULL, N_("last image") },
+       { NULL, "--list-add:",          gr_list_add,            TRUE,  FALSE, N_("<FILE>"), N_("add FILE to command line collection list") },
+       { NULL, "--list-clear",         gr_list_clear,          FALSE, FALSE, NULL, N_("clear command line collection list") },
 #ifdef HAVE_LUA
        { NULL, "--lua:",               gr_lua,                 TRUE, FALSE, N_("<FILE>,<lua script>"), N_("run lua script on FILE") },
 #endif
-       { NULL, "--PWD:",               gr_pwd,                 TRUE, FALSE, N_("<PWD>"), N_("for internal use only") },
+       { NULL, "--new-window",         gr_new_window,          FALSE, FALSE, NULL, N_("new window") },
+       { "-n", "--next",               gr_image_next,          FALSE, FALSE, NULL, N_("next image") },
+       { NULL, "--pixel-info",         gr_pixel_info,          FALSE, FALSE, NULL, N_("print pixel info of mouse pointer on current image") },
+       { NULL, "--print0",             gr_print0,              TRUE, FALSE, NULL, N_("terminate returned data with null character instead of newline") },
+       { NULL, "--PWD:",               gr_pwd,                 TRUE, FALSE, N_("<PWD>"), N_("use PWD as working directory for following commands") },
+       { "-q", "--quit",               gr_quit,                FALSE, FALSE, NULL, N_("quit") },
+       { NULL, "--raise",              gr_raise,               FALSE, FALSE, NULL, N_("bring the Geeqie window to the top") },
+       { NULL, "raise",                gr_raise,               FALSE, FALSE, NULL, N_("bring the Geeqie window to the top") },
+       { "-s", "--slideshow",          gr_slideshow_toggle,    FALSE, TRUE,  NULL, N_("toggle slide show") },
+       { NULL, "--slideshow-recurse:", gr_slideshow_start_rec, TRUE,  FALSE, N_("<FOLDER>"), N_("start recursive slide show in FOLDER") },
+       { "-ss","--slideshow-start",    gr_slideshow_start,     FALSE, FALSE, NULL, N_("start slide show") },
+       { "-sS","--slideshow-stop",     gr_slideshow_stop,      FALSE, FALSE, NULL, N_("stop slide show") },
+       { NULL, "--tell",               gr_file_tell,           FALSE, FALSE, NULL, N_("print filename [and Collection] of current image") },
+       { "+t", "--tools-show",         gr_tools_show,          FALSE, TRUE,  NULL, N_("show tools") },
+       { "-t", "--tools-hide",         gr_tools_hide,          FALSE, TRUE,  NULL, N_("hide tools") },
+       { NULL, "--view:",              gr_file_view,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE in new window") },
+       { NULL, "view:",                gr_file_view,           TRUE,  FALSE, N_("<FILE>"), N_("open FILE in new window") },
        { NULL, NULL, NULL, FALSE, FALSE, NULL, NULL }
 };
 
@@ -1246,19 +1520,18 @@ void remote_help(void)
                {
                if (remote_commands[i].description)
                        {
-                       s_opt_param = g_strconcat(remote_commands[i].opt_s, remote_commands[i].parameter, NULL);
+                       s_opt_param = g_strdup(remote_commands[i].opt_s  ? remote_commands[i].opt_s : "" );
                        l_opt_param = g_strconcat(remote_commands[i].opt_l, remote_commands[i].parameter, NULL);
-                       printf_term(FALSE, "  %-11s%-1s %-30s%-s\n",
-                                   (remote_commands[i].opt_s) ? s_opt_param : "",
-                                   (remote_commands[i].opt_s && remote_commands[i].opt_l) ? "," : " ",
-                                   (remote_commands[i].opt_l) ? l_opt_param : "",
-                                   _(remote_commands[i].description));
+                       printf_term(FALSE, "  %-4s %-40s%-s\n",
+                                       s_opt_param,
+                                       l_opt_param,
+                                       remote_commands[i].description);
                        g_free(s_opt_param);
                        g_free(l_opt_param);
                        }
                i++;
                }
-       printf_term(FALSE, N_("\n  All other command line parameters are used as plain files if they exists.\n"));
+       printf_term(FALSE, N_("\n\n  All other command line parameters are used as plain files if they exist.\n\n  The name of a collection, with or without either path or extension (.gqv) may be used.\n"));
 }
 
 GList *remote_build_list(GList *list, gint argc, gchar *argv[], GList **errors)
@@ -1275,7 +1548,7 @@ GList *remote_build_list(GList *list, gint argc, gchar *argv[], GList **errors)
                        {
                        list = g_list_append(list, argv[i]);
                        }
-               else if (errors && !isfile(argv[i]))
+               else if (errors && !isname(argv[i]))
                        {
                        *errors = g_list_append(*errors, argv[i]);
                        }
@@ -1286,11 +1559,11 @@ GList *remote_build_list(GList *list, gint argc, gchar *argv[], GList **errors)
 }
 
 /**
- * \param arg_exec Binary (argv0)
- * \param remote_list Evaluated and recognized remote commands
- * \param path The current path
- * \param cmd_list List of all non collections in Path
- * \param collection_list List of all collections in argv
+ * @param arg_exec Binary (argv0)
+ * @param remote_list Evaluated and recognized remote commands
+ * @param path The current path
+ * @param cmd_list List of all non collections in Path (gchar *path)
+ * @param collection_list List of all collections in argv
  */
 void remote_control(const gchar *arg_exec, GList *remote_list, const gchar *path,
                    GList *cmd_list, GList *collection_list)
@@ -1324,6 +1597,11 @@ void remote_control(const gchar *arg_exec, GList *remote_list, const gchar *path
                        entry = remote_command_find(text, NULL);
                        if (entry)
                                {
+                               /* If Geeqie is not running, stop the --new-window command opening a second window */
+                               if (g_strcmp0(text, "--new-window") == 0)
+                                       {
+                                       remote_list = g_list_remove(remote_list, text);
+                                       }
                                if (entry->prefer_command_line)
                                        {
                                        remote_list = g_list_remove(remote_list, text);
@@ -1397,15 +1675,12 @@ void remote_control(const gchar *arg_exec, GList *remote_list, const gchar *path
                work = cmd_list;
                while (work)
                        {
-                       FileData *fd;
                        gchar *text;
 
-                       fd = work->data;
-                       work = work->next;
-
-                       text = g_strconcat(prefix, fd->path, NULL);
+                       text = g_strconcat(prefix, work->data, NULL);
                        remote_client_send(rc, text);
                        g_free(text);
+                       work = work->next;
 
                        sent = TRUE;
                        }