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 340488 - Sabayon should use gnome-vfs for monitoring
Sabayon should use gnome-vfs for monitoring
Status: RESOLVED FIXED
Product: sabayon
Classification: Deprecated
Component: general
SVN trunk
Other Linux
: Normal enhancement
: ---
Assigned To: Federico Mena Quintero
Maintainers of sabayon
Depends on:
Blocks:
 
 
Reported: 2006-05-03 04:35 UTC by Hans Petter Jansson
Modified: 2007-09-19 19:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Removes gamin dependency, uses gnome-vfs instead. (5.39 KB, patch)
2006-05-03 04:36 UTC, Hans Petter Jansson
none Details | Review
Updated sabayon-2.12.3-gnomevfs-monitoring.patch (5.49 KB, patch)
2007-05-18 23:10 UTC, Federico Mena Quintero
committed Details | Review
sabayon-268903-dont-recurse-ignored-directory.diff (1.95 KB, patch)
2007-05-18 23:11 UTC, Federico Mena Quintero
committed Details | Review

Description Hans Petter Jansson 2006-05-03 04:35:11 UTC
Sabayon should not depend on gamin, and instead use gnome-vfs for file change monitoring. I've written a patch that accomplishes this. It passes the unit tests.
Comment 1 Hans Petter Jansson 2006-05-03 04:36:48 UTC
Created attachment 64720 [details] [review]
Removes gamin dependency, uses gnome-vfs instead.
Comment 2 Baptiste Mille-Mathias 2006-07-11 20:13:25 UTC
Hello mister(s) maintainer(s).

is there a chance you give a reply to a person who coded to improve sabayon ? :)

thanks
Comment 3 Alexander Larsson 2006-07-28 03:46:27 UTC
Why exactly is using gnome-vfs better than using gamin directly?
Comment 4 Hans Petter Jansson 2006-11-09 00:03:17 UTC
gnome-vfs provides a higher-level API than gamin, so the underlying notification mechanism can be swapped out. For instance, on SuSE, we use inotify.
Comment 5 Federico Mena Quintero 2007-05-18 23:10:54 UTC
Created attachment 88424 [details] [review]
Updated sabayon-2.12.3-gnomevfs-monitoring.patch
Comment 6 Federico Mena Quintero 2007-05-18 23:11:37 UTC
Created attachment 88425 [details] [review]
sabayon-268903-dont-recurse-ignored-directory.diff
Comment 7 Federico Mena Quintero 2007-07-12 18:42:24 UTC
Committed.

2007-07-11  Federico Mena Quintero  <federico@novell.com>

	Patches from https://bugzilla.novell.com/show_bug.cgi?id=230018
	and https://bugzilla.novell.com/show_bug.cgi?id=268903

	* lib/dirmonitor.py (DirectoryMonitor.__handle_gnomevfs_event): If
	we get an event for a directory, call the callback before
	recursing into the directory and adding watches to it.  This way
	the log of changes will be in the correct order (it needs to know
	that a dir is created before its subdirs are).
	(DirectoryMonitor.__should_ignore_dir): Don't print here whether
	we ignore the directory.
	(DirectoryMonitor.__should_ignore_file): Likewise.
	(DirectoryMonitor.__handle_gnomevfs_event): Print it here instead.
	(DirectoryMonitor.__monitor_dir_recurse): Likewise.
	(DirectoryMonitor.__monitor_dir_recurse): Always invoke the
	callback.
	(DirectoryMonitor.set_directories_to_ignore): Assert that we don't
	have any watches yet.
	(DirectoryMonitor.set_files_to_ignore): Likewise.

	* lib/sources/mozillasource.py (MozillaDelegate.handle_change):
	Log this.

2007-07-11  Federico Mena Quintero  <federico@novell.com>

	* lib/dirmonitor.py: Use gnome-vfs for monitoring instead of
	gamin.  Patch from http://bugzilla.gnome.org/show_bug.cgi?id=340488