After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 571495 - Migrate from deprecated gnome_execute to g_spawn/xdg-terminal
Migrate from deprecated gnome_execute to g_spawn/xdg-terminal
Status: RESOLVED FIXED
Product: gnome-commander
Classification: Other
Component: application
1.2.x
Other Linux
: Normal normal
: 1.4
Assigned To: GNOME Commander maintainer(s)
epiotr
Depends on:
Blocks: 589072
 
 
Reported: 2009-02-12 15:53 UTC by André Klapper
Modified: 2014-10-05 19:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch removing gnome_execute_shell (3.53 KB, patch)
2009-04-21 12:48 UTC, Ilya Barygin
needs-work Details | Review

Description André Klapper 2009-02-12 15:53:53 UTC
To get rid of libgnome and according to http://live.gnome.org/LibgnomeMustDie , gnome-exec shall be replaced by GSpawn which covers all of the useful parts of gnome-exec.
gnome_execute_terminal* can be implemented using xdg-terminal (which will use "the active desktop's terminal program" rather than "the gnome-configured terminal program")

See
http://library.gnome.org/devel/libgnome/stable/libgnome-gnome-exec.html
http://library.gnome.org/devel/glib/stable/glib-Spawning-Processes.html



./gnome-commander/plugins/fileroller/file-roller-plugin.c:    gnome_execute_shell (t, cmd);
./gnome-commander/plugins/fileroller/file-roller-plugin.c:    gnome_execute_shell (active_dir_path, cmd);
./gnome-commander/src/utils.cc:    gnome_execute_shell (dir, command);
./gnome-commander/src/gnome-cmd-user-actions.cc:    gnome_execute_shell (dir && gnome_cmd_dir_is_local (dir) ? dir_path.c_str() : NULL, cmd.c_str());
./gnome-commander/src/gnome-cmd-user-actions.cc:    gnome_execute_terminal_shell (dpath, NULL);
Comment 1 Ilya Barygin 2009-04-21 12:48:53 UTC
Created attachment 133036 [details] [review]
Patch removing gnome_execute_shell

Let me propose a patch for removing gnome_execute_shell. What's xdg-terminal? I can't find any information on how to use it.
Comment 2 André Klapper 2010-05-29 12:00:42 UTC
ping - can this get a review?
Comment 3 epiotr 2010-06-03 19:11:44 UTC
Review of attachment 133036 [details] [review]:

Partially committed (file-roller-plugin.c | gnome-cmd-user-actions.cc). Part dealing with utils.cc needs some more evaluation.
Comment 4 Uwe Scholz 2014-10-05 19:40:55 UTC
Hi,

the remaining issue with utils.cc has been fixed in gcmd's software repository. The fix will go into the next release, v1.4.4.

Best wishes,
Uwe