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 675753 - [PATCH] Remove gnome-python dependency
[PATCH] Remove gnome-python dependency
Status: RESOLVED FIXED
Product: gnome-schedule
Classification: Other
Component: general
2.2.0
Other Linux
: Normal normal
: ---
Assigned To: GNOME Schedule Maintainers
GNOME Schedule Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-05-09 14:26 UTC by Balló György
Modified: 2013-05-10 09:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove gnome-python dependency (5.01 KB, patch)
2012-05-09 14:26 UTC, Balló György
none Details | Review

Description Balló György 2012-05-09 14:26:17 UTC
Created attachment 213736 [details] [review]
Remove gnome-python dependency

This patch removes libgnomeui/libgnomeui dependency, which is a deprecated library[1], by removing the usage of gnome-python bindings.

The only drawback is that the terminal application is no longer configurable, it's hard-coded to gnome-terminal.

[1] https://live.gnome.org/ProjectRidley
Comment 1 Gaute Hope 2012-05-09 19:32:43 UTC
(In reply to comment #0)
> Created an attachment (id=213736) [details] [review]
> Remove gnome-python dependency
> 
> This patch removes libgnomeui/libgnomeui dependency, which is a deprecated
> library[1], by removing the usage of gnome-python bindings.

Great! Nice work, thanks.
 
> The only drawback is that the terminal application is no longer configurable,
> it's hard-coded to gnome-terminal.
> 
> [1] https://live.gnome.org/ProjectRidley

This is quite important, but its replacement xdg-terminal is coming up in xdg-utils:

  http://cgit.freedesktop.org/xdg/xdg-utils/tree/scripts/xdg-terminal

I would suggest implementing it using xdg-terminal now, and wait with a release at least until a stable version of xdg-utils with it included is released. 

xdg-terminal still seem to have some serious issues. I am using a custom script at the moment.

- Gaute
Comment 2 Balló György 2012-05-10 02:17:21 UTC
xdg-terminal is not distributed in the latest 1.1.0-rc1 release of xdg-utils. Are you sure that it will be distributed in the next release?
Comment 3 Gaute Hope 2012-05-10 07:42:37 UTC
(In reply to comment #2)
> xdg-terminal is not distributed in the latest 1.1.0-rc1 release of xdg-utils.
> Are you sure that it will be distributed in the next release?

No. But hard coding gnome-terminal is not really an option (it would for instance not work for me), according to ProjectRidley xdg-terminal will be the replacement for this functionality in libgnome.

I will try to figure out if xdg-terminal is on its way any time soon, but it seems to have received some work in the latest commits. If it continues to be on standby for another N years we would have to find another solution.

- Gaute
Comment 4 Balló György 2012-11-02 04:26:46 UTC
I don't see any activity on xdg-terminal:

  http://cgit.freedesktop.org/xdg/xdg-utils/log/scripts/xdg-terminal

An alternative solution is that gsettings-desktop-schemas contains a org.gnome.desktop.default-applications.terminal schema to store the default terminal application and its exec argument. So gnome-schedule could read these settings simply with the following commands:

  gsettings get org.gnome.desktop.default-applications.terminal exec
  gsettings get org.gnome.desktop.default-applications.terminal exec-arg
Comment 5 Gaute Hope 2012-11-02 08:14:39 UTC
(In reply to comment #4)
> I don't see any activity on xdg-terminal:
> 
>   http://cgit.freedesktop.org/xdg/xdg-utils/log/scripts/xdg-terminal
> 
> An alternative solution is that gsettings-desktop-schemas contains a
> org.gnome.desktop.default-applications.terminal schema to store the default
> terminal application and its exec argument. So gnome-schedule could read these
> settings simply with the following commands:
> 
>   gsettings get org.gnome.desktop.default-applications.terminal exec
>   gsettings get org.gnome.desktop.default-applications.terminal exec-arg

Is that not exactly what gnome-python does?
Comment 6 Balló György 2012-11-02 08:33:02 UTC
No, gnome-python stores similar settings from GConf, and it's deprecated a long time ago.
Comment 7 Gaute Hope 2012-11-02 08:35:18 UTC
(In reply to comment #6)
> No, gnome-python stores similar settings from GConf, and it's deprecated a long
> time ago.

Alright, then this could be a good solution! Do you want to update the patch?
Comment 8 Gaute Hope 2013-05-10 09:53:32 UTC
Merged and updated to use gsettings. Thanks.