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 166303 - Use new gnome_desktop_item_set_launch_time() API
Use new gnome_desktop_item_set_launch_time() API
Status: RESOLVED FIXED
Product: gnome-utils
Classification: Deprecated
Component: general
trunk
Other Linux
: Normal normal
: ---
Assigned To: gnome-utils Maintainers
gnome-utils Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-02-04 17:15 UTC by Elijah Newren
Modified: 2005-03-07 18:14 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Use new gnome_desktop_item_set_launch_time() API (729 bytes, patch)
2005-02-04 17:16 UTC, Elijah Newren 		none Details | Review
Update configure.in to depend on newer gnome-desktop version (544 bytes, patch)
2005-02-05 17:56 UTC, Elijah Newren 		none Details | Review

Description Elijah Newren 2005-02-04 17:15:56 UTC 
Bug 150910 added a new gnome_desktop_item_set_launch_time() API.  All users of
gnome_desktop_item_launch* should be using this API.  I did a search through all
code paths that call any such function and all code paths are such that
gtk_get_current_event_time() will return the right result (i.e. they're all in
signal handlers such as activate, clicked, or button_press).  So the patch is
pretty easy--I'll attach it in a minute.
Comment 1 Elijah Newren 2005-02-04 17:16:27 UTC 
Created attachment 36999 [details] [review]
Use new gnome_desktop_item_set_launch_time() API
Comment 2 Dennis Cranston 2005-02-04 21:12:26 UTC 
Fixed in CVS.

2005-02-41  Dennis Cranston <dennis_cranston@yahoo.com>

	* gsearchtool-support.c: (open_file_with_application): 
	Use new gnome_desktop_item_set_launch_time() API.  
	Patch provided by Elijah Newren.  Fixes bug #166303.
Comment 3 Elijah Newren 2005-02-05 17:56:09 UTC 
Oops, we also need to update configure.in.
Comment 4 Elijah Newren 2005-02-05 17:56:55 UTC 
Created attachment 37033 [details] [review]
Update configure.in to depend on newer gnome-desktop version
Comment 5 Vincent Noel 2005-03-07 18:09:22 UTC 
crap, this was never applied :-(
Comment 6 Vincent Noel 2005-03-07 18:14:16 UTC 
Ok, it's in now.