GNOME Bugzilla – Bug 762248
Don't use g_desktop_app_info_set_desktop_env
Last modified: 2016-03-01 10:52:52 UTC
is deprecated, find an alternative!
Created attachment 321642 [details] [review] main: remove g_desktop_app_info_set_desktop_env The environment variable used in present is XDG_CURRENT_DESKTOP.
(In reply to Razvan Chitu from comment #1) > Created attachment 321642 [details] [review] [review] > main: remove g_desktop_app_info_set_desktop_env > > The environment variable used in present is XDG_CURRENT_DESKTOP. I assumed that the value of XDG_CURRENT_DESKTOP should not be set within nautilus, because it would overwrite the information from the system. If it should be set, the patch can easily be modified.
Cosimo, I'm not sure about this, can you take a look?
To be clear, I'm mostly sure that this is correct. But I'm not sure if removing have some implications.
Review of attachment 321642 [details] [review]: Looks good to me, sorry for the late review.
Comment on attachment 321642 [details] [review] main: remove g_desktop_app_info_set_desktop_env Attachment 321642 [details] pushed as 0e31141 - main: remove g_desktop_app_info_set_desktop_env