Message ID | 39072254f3d646691a8c637e551ffaede7b9928c.1429725549.git.yann.morin.1998@free.fr |
---|---|
State | Changes Requested |
Headers | show |
On 22/04/15 20:09, Yann E. MORIN wrote: > Using dnotify, matchbox-panel can automatically reload its menu. > > dnotify requires a kernel >=2.4, which we can consider to be > virtually for granted. So, why not enable this unconditionally? Regards, Arnout > > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> > --- > package/matchbox/matchbox-panel/Config.in | 7 +++++++ > package/matchbox/matchbox-panel/matchbox-panel.mk | 6 ++++++ > 2 files changed, 13 insertions(+) > > diff --git a/package/matchbox/matchbox-panel/Config.in b/package/matchbox/matchbox-panel/Config.in > index 93241f0..6daf2db 100644 > --- a/package/matchbox/matchbox-panel/Config.in > +++ b/package/matchbox/matchbox-panel/Config.in > @@ -18,4 +18,11 @@ config BR2_PACKAGE_MATCHBOX_PANEL_ACPI > otherwise the battery monitor mini-applet is not > built. > > +config BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY > + bool "use dnotify" > + help > + Using dnotify allows matchbox-panel to automatically reload > + its menu when entries are changed (e.g. when a new application > + is installed). > + > endif # BR2_PACKAGE_MATCHBOX_PANEL > diff --git a/package/matchbox/matchbox-panel/matchbox-panel.mk b/package/matchbox/matchbox-panel/matchbox-panel.mk > index aa9018c..dedbab6 100644 > --- a/package/matchbox/matchbox-panel/matchbox-panel.mk > +++ b/package/matchbox/matchbox-panel/matchbox-panel.mk > @@ -26,6 +26,12 @@ else > MATCHBOX_PANEL_CONF_OPTS += --disable-acpi-linux > endif > > +ifeq ($(BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY),y) > +MATCHBOX_PANEL_CONF_OPTS += --enable-dnotify > +else > +MATCHBOX_PANEL_CONF_OPTS += --disable-dnotify > +endif > + > ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS),y) > MATCHBOX_PANEL_DEPENDENCIES += wireless_tools > endif >
Arnout, All, On 2015-05-01 23:44 +0200, Arnout Vandecappelle spake thusly: > On 22/04/15 20:09, Yann E. MORIN wrote: > > Using dnotify, matchbox-panel can automatically reload its menu. > > > > dnotify requires a kernel >=2.4, which we can consider to be > > virtually for granted. > > So, why not enable this unconditionally? Well... OK, done. Regards, Yann E. MORIN. > > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> > > --- > > package/matchbox/matchbox-panel/Config.in | 7 +++++++ > > package/matchbox/matchbox-panel/matchbox-panel.mk | 6 ++++++ > > 2 files changed, 13 insertions(+) > > > > diff --git a/package/matchbox/matchbox-panel/Config.in b/package/matchbox/matchbox-panel/Config.in > > index 93241f0..6daf2db 100644 > > --- a/package/matchbox/matchbox-panel/Config.in > > +++ b/package/matchbox/matchbox-panel/Config.in > > @@ -18,4 +18,11 @@ config BR2_PACKAGE_MATCHBOX_PANEL_ACPI > > otherwise the battery monitor mini-applet is not > > built. > > > > +config BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY > > + bool "use dnotify" > > + help > > + Using dnotify allows matchbox-panel to automatically reload > > + its menu when entries are changed (e.g. when a new application > > + is installed). > > + > > endif # BR2_PACKAGE_MATCHBOX_PANEL > > diff --git a/package/matchbox/matchbox-panel/matchbox-panel.mk b/package/matchbox/matchbox-panel/matchbox-panel.mk > > index aa9018c..dedbab6 100644 > > --- a/package/matchbox/matchbox-panel/matchbox-panel.mk > > +++ b/package/matchbox/matchbox-panel/matchbox-panel.mk > > @@ -26,6 +26,12 @@ else > > MATCHBOX_PANEL_CONF_OPTS += --disable-acpi-linux > > endif > > > > +ifeq ($(BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY),y) > > +MATCHBOX_PANEL_CONF_OPTS += --enable-dnotify > > +else > > +MATCHBOX_PANEL_CONF_OPTS += --disable-dnotify > > +endif > > + > > ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS),y) > > MATCHBOX_PANEL_DEPENDENCIES += wireless_tools > > endif > > > > > -- > Arnout Vandecappelle arnout at mind be > Senior Embedded Software Architect +32-16-286500 > Essensium/Mind http://www.mind.be > G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven > LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle > GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
diff --git a/package/matchbox/matchbox-panel/Config.in b/package/matchbox/matchbox-panel/Config.in index 93241f0..6daf2db 100644 --- a/package/matchbox/matchbox-panel/Config.in +++ b/package/matchbox/matchbox-panel/Config.in @@ -18,4 +18,11 @@ config BR2_PACKAGE_MATCHBOX_PANEL_ACPI otherwise the battery monitor mini-applet is not built. +config BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY + bool "use dnotify" + help + Using dnotify allows matchbox-panel to automatically reload + its menu when entries are changed (e.g. when a new application + is installed). + endif # BR2_PACKAGE_MATCHBOX_PANEL diff --git a/package/matchbox/matchbox-panel/matchbox-panel.mk b/package/matchbox/matchbox-panel/matchbox-panel.mk index aa9018c..dedbab6 100644 --- a/package/matchbox/matchbox-panel/matchbox-panel.mk +++ b/package/matchbox/matchbox-panel/matchbox-panel.mk @@ -26,6 +26,12 @@ else MATCHBOX_PANEL_CONF_OPTS += --disable-acpi-linux endif +ifeq ($(BR2_PACKAGE_MATCHBOX_PANEL_DNOTIFY),y) +MATCHBOX_PANEL_CONF_OPTS += --enable-dnotify +else +MATCHBOX_PANEL_CONF_OPTS += --disable-dnotify +endif + ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS),y) MATCHBOX_PANEL_DEPENDENCIES += wireless_tools endif
Using dnotify, matchbox-panel can automatically reload its menu. dnotify requires a kernel >=2.4, which we can consider to be virtually for granted. Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> --- package/matchbox/matchbox-panel/Config.in | 7 +++++++ package/matchbox/matchbox-panel/matchbox-panel.mk | 6 ++++++ 2 files changed, 13 insertions(+)