In the Debian packages, it is important that changes to
the system configuration (or menu) get quickly noticed
and reloaded into the running fbpanels. Since killall
-USR1 fbpanel is prone to error, I instead decided to
implement a "config watcher", to keep note of all of
the config files we're using, and if any of them gets
updated, we reload the panel.
There appears to be a memory leak that occurs on panel
reload, which I find faintly disturbing, but I haven't
been able to distinguish if it's my changes or a
pre-existing leak (probably in GTK).
The current implementation is available from
(click '(all patches)' for the complete changeset).
Fixes will be included there. For testing, I've run
fbpanel and touched various included files and each
time it causes fbpanel to reload. I will be including
the patch into the Debian fbpanel packages, so it'll
soon get a whole lot of exercise. Feel free to hold
off on applying this patch for a few weeks to get all
the kinks out in the Debian package first.
Log in to post a comment.