http://bugzilla.novell.com/show_bug.cgi?id=536778 User mxwu@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=536778#c13 --- Comment #13 from Mingxi Wu <mxwu@novell.com> 2009-09-07 02:40:27 MDT --- (In reply to comment #12)
Thanks for debugging and patching this. The patch is fine for the quick fix for the M7 deadline but for long term the patch goes the wrong way. It makes the low level library depend on explicit knowledge of a particular upper layer app that uses it. What needs to happen is something like bookmark_agent_get_instance() needs to be modified to take an additional parameter that is the calling apps directory prefix.
Agree. I found there are a lot of work to remove the dependency of gnome-main-menu, eg gconf entries, so I've just done a quick fix here. If upstream fix this before GNOME 2.28 release, maybe we can use it in 11.2. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.