Re: [PATCH] setenv "DISPLAY" when activation extension menu item

[Christian Neumair <chris gnome-de org>]

Am Mittwoch, den 19.04.2006, 10:55 +0200 schrieb Christian Neumair:
> Am Dienstag, den 18.04.2006, 18:22 -0500 schrieb Federico Mena Quintero:
> > On Wed, 2006-04-19 at 01:10 +0200, Christian Neumair wrote:
> > > > while it should be using
> > > > 
> > > > 	gnome_desktop_item_launch_on_screen()
> > > > 	gdk_spawn_on_screen()
> > > 
> > > How can I get the GdkScreen associated with the activated menu item?
> > 
> > If you can get hold of the GtkWidget, you can do
> > gtk_widget_get_screen().
> > 
> > Hmm, but this is an extension, right?  I guess you can use the "window"
> > argument that gets passed to NautilusMenuProvider::get_file_items().
> 
> Indeed, thanks. I still don't see why we shouldn't pre-set DISPLAY to a
> meaningful value, though.

On a sidenote: The value isn't really meaningful after applying the
patch, because I'm g_freeing the setenved string. Maybe a static
variable should be used for storing the current nautilus-owned value,
which should be g_freed it after each g_setenv (i.e. after the new value
was stored).

-- 
Christian Neumair <chris gnome-de org>