2006-01-09  Rodrigo Moya <rodrigo@novell.com>

	Fixes #326141

	* drwright.c (popup_preferences_cb):
	* egg-spawn.[ch]:
	* Makefile.am: kill egg_spawn_* functions, use gdk_spawn_* instead.
This commit is contained in:
Rodrigo Moya 2006-01-09 12:22:34 +00:00 committed by Rodrigo Moya
parent 90010cf737
commit 83f0ec0aea
5 changed files with 11 additions and 441 deletions

View file

@ -1,3 +1,11 @@
2006-01-09 Rodrigo Moya <rodrigo@novell.com>
Fixes #326141
* drwright.c (popup_preferences_cb):
* egg-spawn.[ch]:
* Makefile.am: kill egg_spawn_* functions, use gdk_spawn_* instead.
2005-11-27 Richard Hult <richard@imendio.com>
* main.c: (main): Don't display a dialog when the monitor is

View file

@ -18,9 +18,7 @@ gnome_typing_monitor_SOURCES = \
drw-selection.c \
drw-selection.h \
eggtrayicon.c \
eggtrayicon.h \
egg-spawn.c \
egg-spawn.h
eggtrayicon.h
gnome_typing_monitor_LDADD = @GNOME_LIBS@ @SCREENSAVER_LIBS@
gnome_typing_monitor_LDFLAGS = -export-dynamic

View file

@ -27,6 +27,7 @@
#include <glib/gi18n.h>
#include <gdk/gdk.h>
#include <gdk/gdkkeysyms.h>
#include <gdk/gdkspawn.h>
#include <gtk/gtk.h>
#include <gconf/gconf-client.h>
@ -35,7 +36,6 @@
#include "drw-monitor.h"
#include "drw-utils.h"
#include "eggtrayicon.h"
#include "egg-spawn.h"
#define BLINK_TIMEOUT 200
#define BLINK_TIMEOUT_MIN 120
@ -572,7 +572,7 @@ popup_preferences_cb (gpointer callback_data,
screen = gtk_widget_get_screen (widget);
if (!egg_spawn_command_line_async_on_screen ("gnome-keyboard-properties --typing-break", screen, &error)) {
if (!gdk_spawn_command_line_on_screen (screen, "gnome-keyboard-properties --typing-break", &error)) {
GtkWidget *error_dialog;
error_dialog = gtk_message_dialog_new (NULL, 0,

View file

@ -1,355 +0,0 @@
/* egg-spawn.c:
*
* Copyright (C) 2002 Sun Microsystems Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Library General Public
* License as published by the Free Software Foundation; either
* version 2 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Library General Public License for more details.
*
* You should have received a copy of the GNU Library General Public
* License along with this library; if not, write to the
* Free Software Foundation, Inc., 59 Temple Place - Suite 330,
* Boston, MA 02111-1307, USA.
*
* Authors: Mark McLoughlin <mark@skynet.ie>
*/
#include <config.h>
#include <string.h>
#include "egg-spawn.h"
#include <glib.h>
#include <gdk/gdk.h>
extern char **environ;
/**
* egg_make_spawn_environment_for_screen:
* @screen: A #GdkScreen
* @envp: program environment to copy, or NULL to use current environment.
*
* Returns a modified copy of the program environment @envp (or the current
* environment if @envp is NULL) with $DISPLAY set such that a launched
* application (which calls gdk_display_open()) inheriting this environment
* would have @screen as its default screen..
*
* Returns: a newly-allocated %NULL-terminated array of strings or
* %NULL on error. Use g_strfreev() to free it.
**/
gchar **
egg_make_spawn_environment_for_screen (GdkScreen *screen,
gchar **envp)
{
gchar **retval = NULL;
gchar *display_name;
gint display_index = -1;
gint i, env_len;
g_return_val_if_fail (GDK_IS_SCREEN (screen), NULL);
if (envp == NULL)
envp = environ;
for (env_len = 0; envp [env_len]; env_len++)
if (!strncmp (envp [env_len], "DISPLAY", strlen ("DISPLAY")))
display_index = env_len;
if (display_index == -1)
display_index = env_len++;
retval = g_new (char *, env_len + 1);
retval [env_len] = NULL;
display_name = gdk_screen_make_display_name (screen);
for (i = 0; i < env_len; i++)
if (i == display_index)
retval [i] = g_strconcat ("DISPLAY=", display_name, NULL);
else
retval [i] = g_strdup (envp [i]);
g_assert (i == env_len);
g_free (display_name);
return retval;
}
/**
* egg_spawn_async_on_screen:
* @working_directory: child's current working directory, or %NULL to inherit parent's
* @argv: child's argument vector
* @envp: child's environment, or %NULL to inherit parent's
* @flags: flags from #GSpawnFlags
* @child_setup: function to run in the child just before <function>exec()</function>
* @user_data: user data for @child_setup
* @screen: a #GdkScreen
* @child_pid: return location for child process ID, or %NULL
* @error: return location for error
*
* Like g_spawn_async(), except the child process is spawned in such
* an environment that on calling gdk_display_open() it would be
* returned a #GdkDisplay with @screen as the default screen.
*
* This is useful for applications which wish to launch an application
* on a specific screen.
*
* Return value: %TRUE on success, %FALSE if error is set
**/
gboolean
egg_spawn_async_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gint *child_pid,
GError **error)
{
GdkScreen *default_screen;
gchar **new_envp = NULL;
gboolean retval;
g_return_val_if_fail (GDK_IS_SCREEN (screen), FALSE);
default_screen = gdk_display_get_default_screen (
gdk_screen_get_display (screen));
if (screen != default_screen)
new_envp = egg_make_spawn_environment_for_screen (screen, envp);
retval = g_spawn_async (working_directory, argv,
new_envp ? new_envp : envp,
flags, child_setup, user_data,
child_pid, error);
g_strfreev (new_envp);
return retval;
}
/**
* egg_spawn_async_with_pipes_on_screen:
* @working_directory: child's current working directory, or %NULL to inherit parent's
* @argv: child's argument vector
* @envp: child's environment, or %NULL to inherit parent's
* @flags: flags from #GSpawnFlags
* @child_setup: function to run in the child just before <function>exec()</function>
* @user_data: user data for @child_setup
* @screen: a #GdkScreen
* @child_pid: return location for child process ID, or %NULL
* @standard_input: return location for file descriptor to write to child's stdin, or %NULL
* @standard_output: return location for file descriptor to read child's stdout, or %NULL
* @standard_error: return location for file descriptor to read child's stderr, or %NULL
* @error: return location for error
*
* Like g_spawn_async_with_pipes(), except the child process is
* spawned in such an environment that on calling gdk_display_open()
* it would be returned a #GdkDisplay with @screen as the default
* screen.
*
* This is useful for applications which wish to launch an application
* on a specific screen.
*
* Return value: %TRUE on success, %FALSE if an error was set
**/
gboolean
egg_spawn_async_with_pipes_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gint *child_pid,
gint *standard_input,
gint *standard_output,
gint *standard_error,
GError **error)
{
GdkScreen *default_screen;
gchar **new_envp = NULL;
gboolean retval;
g_return_val_if_fail (GDK_IS_SCREEN (screen), FALSE);
default_screen = gdk_display_get_default_screen (
gdk_screen_get_display (screen));
if (screen != default_screen)
new_envp = egg_make_spawn_environment_for_screen (screen, envp);
retval = g_spawn_async_with_pipes (working_directory, argv,
new_envp ? new_envp : envp,
flags, child_setup, user_data,
child_pid, standard_input,
standard_output, standard_error,
error);
g_strfreev (new_envp);
return retval;
}
/**
* egg_spawn_sync_on_screen:
* @working_directory: child's current working directory, or %NULL to inherit parent's
* @argv: child's argument vector
* @envp: child's environment, or %NULL to inherit parent's
* @flags: flags from #GSpawnFlags
* @child_setup: function to run in the child just before <function>exec()</function>
* @user_data: user data for @child_setup
* @screen: a #GdkScreen
* @standard_output: return location for child output
* @standard_error: return location for child error messages
* @exit_status: child exit status, as returned by <function>waitpid()</function>
* @error: return location for error
*
* Like g_spawn_sync(), except the child process is spawned in such
* an environment that on calling gdk_display_open() it would be
* returned a #GdkDisplay with @screen as the default screen.
*
* This is useful for applications which wish to launch an application
* on a specific screen.
*
* Return value: %TRUE on success, %FALSE if an error was set.
**/
gboolean
egg_spawn_sync_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gchar **standard_output,
gchar **standard_error,
gint *exit_status,
GError **error)
{
GdkScreen *default_screen;
gchar **new_envp = NULL;
gboolean retval;
g_return_val_if_fail (GDK_IS_SCREEN (screen), FALSE);
default_screen = gdk_display_get_default_screen (
gdk_screen_get_display (screen));
if (screen != default_screen)
new_envp = egg_make_spawn_environment_for_screen (screen, envp);
retval = g_spawn_sync (working_directory, argv,
new_envp ? new_envp : envp,
flags, child_setup, user_data,
standard_output, standard_error,
exit_status, error);
g_strfreev (new_envp);
return retval;
}
/**
* egg_spawn_command_line_sync_on_screen:
* @command_line: a command line
* @screen: a #GdkScreen
* @standard_output: return location for child output
* @standard_error: return location for child errors
* @exit_status: return location for child exit status
* @error: return location for errors
*
* Like g_spawn_command_line_sync(), except the child process is
* spawned in such an environment that on calling gdk_display_open()
* it would be returned a #GdkDisplay with @screen as the default
* screen.
*
* This is useful for applications which wish to launch an application
* on a specific screen.
*
* Return value: %TRUE on success, %FALSE if an error was set
**/
gboolean
egg_spawn_command_line_sync_on_screen (const gchar *command_line,
GdkScreen *screen,
gchar **standard_output,
gchar **standard_error,
gint *exit_status,
GError **error)
{
gchar **argv = NULL;
gboolean retval;
g_return_val_if_fail (command_line != NULL, FALSE);
if (!g_shell_parse_argv (command_line,
NULL, &argv,
error))
return FALSE;
retval = egg_spawn_sync_on_screen (NULL,
argv,
NULL,
G_SPAWN_SEARCH_PATH,
NULL,
NULL,
screen,
standard_output,
standard_error,
exit_status,
error);
g_strfreev (argv);
return retval;
}
/**
* egg_spawn_command_line_async_on_screen:
* @command_line: a command line
* @screen: a #GdkScreen
* @error: return location for errors
*
* Like g_spawn_command_line_async(), except the child process is
* spawned in such an environment that on calling gdk_display_open()
* it would be returned a #GdkDisplay with @screen as the default
* screen.
*
* This is useful for applications which wish to launch an application
* on a specific screen.
*
* Return value: %TRUE on success, %FALSE if error is set.
**/
gboolean
egg_spawn_command_line_async_on_screen (const gchar *command_line,
GdkScreen *screen,
GError **error)
{
gchar **argv = NULL;
gboolean retval;
g_return_val_if_fail (command_line != NULL, FALSE);
if (!g_shell_parse_argv (command_line,
NULL, &argv,
error))
return FALSE;
retval = egg_spawn_async_on_screen (NULL,
argv,
NULL,
G_SPAWN_SEARCH_PATH,
NULL,
NULL,
screen,
NULL,
error);
g_strfreev (argv);
return retval;
}

View file

@ -1,81 +0,0 @@
/* egg-spawn.h:
*
* Copyright (C) 2002 Sun Microsystems Inc.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Library General Public
* License as published by the Free Software Foundation; either
* version 2 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Library General Public License for more details.
*
* You should have received a copy of the GNU Library General Public
* License along with this library; if not, write to the
* Free Software Foundation, Inc., 59 Temple Place - Suite 330,
* Boston, MA 02111-1307, USA.
*
* Authors: Mark McLoughlin <mark@skynet.ie>
*/
#ifndef __EGG_SPAWN_H__
#define __EGG_SPAWN_H__
#include <gdk/gdk.h>
#include <glib/gspawn.h>
G_BEGIN_DECLS
gchar **egg_make_spawn_environment_for_screen (GdkScreen *screen,
gchar **envp);
gboolean egg_spawn_async_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gint *child_pid,
GError **error);
gboolean egg_spawn_async_with_pipes_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gint *child_pid,
gint *standard_input,
gint *standard_output,
gint *standard_error,
GError **error);
gboolean egg_spawn_sync_on_screen (const gchar *working_directory,
gchar **argv,
gchar **envp,
GSpawnFlags flags,
GSpawnChildSetupFunc child_setup,
gpointer user_data,
GdkScreen *screen,
gchar **standard_output,
gchar **standard_error,
gint *exit_status,
GError **error);
gboolean egg_spawn_command_line_sync_on_screen (const gchar *command_line,
GdkScreen *screen,
gchar **standard_output,
gchar **standard_error,
gint *exit_status,
GError **error);
gboolean egg_spawn_command_line_async_on_screen (const gchar *command_line,
GdkScreen *screen,
GError **error);
G_END_DECLS
#endif /* __EGG_SPAWN_H__ */