Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 207081 - gnome-base/gnome-menus and inotify
Summary: gnome-base/gnome-menus and inotify
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-22 21:10 UTC by Daniel Drake (RETIRED)
Modified: 2008-02-16 04:20 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Drake (RETIRED) gentoo-dev 2008-01-22 21:10:29 UTC
The gnome-menus ebuild is enabling inotify unconditionally on Linux. When this is the case, gnome-menus will use inotify and inotify alone, with no fallback (not exactly the greatest practice).

If the configure flag is not passed, gnome-menus will use pam instead, which is now implemented by gamin and uses inotify internally if it's available, and standard file monitoring if it's not (i.e. the desirable approach).

Have you considered not passing the configure flag, and hence always using gamin (which has the correct fallback mechanism)? This would simplify the ebuild a bit (no more using linux-info or kernel_linux stuff).
Comment 1 Rémi Cardona (RETIRED) gentoo-dev 2008-02-11 09:14:19 UTC
This issue has been fixed in upcoming 2.21.91 where gnome-menus will depend on gio (the gnome-vfs replacement library inside glib-2.15) for all file monitoring operations.

Bottom line, it will only depend on glib, no more famd/gamin/inotify cruft. Given this improvement, I don't feel like it's worth modifying the current ebuilds.

Daniel, how strongly do you feel about this? Herd, what say you?
Comment 2 Daniel Drake (RETIRED) gentoo-dev 2008-02-13 16:54:47 UTC
that sounds good enough for me
feel free to close this bug
Comment 3 Daniel Gryniewicz (RETIRED) gentoo-dev 2008-02-16 04:20:20 UTC
I agree.  Marking fixed, since the overlay version is fixed.