* [PATCH] matchbox-session-sato: Make the battery applet depend on machine features
@ 2018-05-02 17:33 Jaewon Lee
2018-05-02 18:32 ` ✗ patchtest: failure for matchbox-session-sato: Make the battery applet depend on machine features (rev2) Patchwork
2018-05-03 8:25 ` [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Eran Matityahu
0 siblings, 2 replies; 4+ messages in thread
From: Jaewon Lee @ 2018-05-02 17:33 UTC (permalink / raw)
To: openembedded-core; +Cc: Eran Matityahu
From: Eran Matityahu <eran.m@variscite.com>
matchbox-panel enables the battery plugin only if the
acpi/apm machine features are enabled,
so enable the battery applet in the session script
under the same conditions.
This avoids the 'Failed to load applet "battery"' warning at runtime,
in case these machine features are not defined.
Signed-off-by: Eran Matityahu <eran.m@variscite.com>
---
meta/recipes-sato/matchbox-sato/matchbox-session-sato/session | 5 ++++-
meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb | 2 +-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
index f6313bd..3a70574 100644
--- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
+++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
@@ -20,7 +20,10 @@ matchbox-desktop &
# doesn't have the feature "foo".
START_APPLETS=showdesktop,windowselector
-END_APPLETS=clock,battery,$KEYBOARD_APPLET,systray,startup-notify,notify
+END_APPLETS=$KEYBOARD_APPLET,systray,startup-notify,notify
+END_APPLETS=battery,$END_APPLETS # feature-acpi
+END_APPLETS=battery,$END_APPLETS # feature-apm
+END_APPLETS=clock,$END_APPLETS
END_APPLETS=openmoko-panel-gsm,$END_APPLETS # feature-phone
matchbox-panel --start-applets $START_APPLETS --end-applets $END_APPLETS &
diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
index d146e83..8230e95 100644
--- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
+++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
@@ -26,7 +26,7 @@ FILES_${PN} += "${datadir}/themes/Sato/index.theme"
do_install() {
# This is the set of machine features that the script has markers for
- FEATURES="phone"
+ FEATURES="acpi apm phone"
SCRIPT="${S}/sedder"
rm -f $SCRIPT
touch $SCRIPT
--
2.7.4
This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.
^ permalink raw reply related [flat|nested] 4+ messages in thread
* ✗ patchtest: failure for matchbox-session-sato: Make the battery applet depend on machine features (rev2)
2018-05-02 17:33 [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Jaewon Lee
@ 2018-05-02 18:32 ` Patchwork
2018-05-03 8:25 ` [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Eran Matityahu
1 sibling, 0 replies; 4+ messages in thread
From: Patchwork @ 2018-05-02 18:32 UTC (permalink / raw)
To: Eran Matityahu; +Cc: openembedded-core
== Series Details ==
Series: matchbox-session-sato: Make the battery applet depend on machine features (rev2)
Revision: 2
URL : https://patchwork.openembedded.org/series/8026/
State : failure
== Summary ==
Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:
* Issue Series does not apply on top of target branch [test_series_merge_on_head]
Suggested fix Rebase your series on top of targeted branch
Targeted branch master (currently at 1b03cdf290)
If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).
---
Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] matchbox-session-sato: Make the battery applet depend on machine features
2018-05-02 17:33 [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Jaewon Lee
2018-05-02 18:32 ` ✗ patchtest: failure for matchbox-session-sato: Make the battery applet depend on machine features (rev2) Patchwork
@ 2018-05-03 8:25 ` Eran Matityahu
1 sibling, 0 replies; 4+ messages in thread
From: Eran Matityahu @ 2018-05-03 8:25 UTC (permalink / raw)
To: openembedded-core
Hi.
This is a duplicate of the original patch 142525 that I've submitted
on July 30, 2017, only it has tabs replaced with spaces so it doesn't
apply.
The original patch applies cleanly. Please use it instead this one.
Regards,
Eran
On Wed, May 2, 2018 at 8:33 PM, Jaewon Lee <jaewon.lee@xilinx.com> wrote:
> From: Eran Matityahu <eran.m@variscite.com>
>
> matchbox-panel enables the battery plugin only if the
> acpi/apm machine features are enabled,
> so enable the battery applet in the session script
> under the same conditions.
>
> This avoids the 'Failed to load applet "battery"' warning at runtime,
> in case these machine features are not defined.
>
> Signed-off-by: Eran Matityahu <eran.m@variscite.com>
> ---
> meta/recipes-sato/matchbox-sato/matchbox-session-sato/session | 5 ++++-
> meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb | 2 +-
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
> index f6313bd..3a70574 100644
> --- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
> +++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
> @@ -20,7 +20,10 @@ matchbox-desktop &
> # doesn't have the feature "foo".
>
> START_APPLETS=showdesktop,windowselector
> -END_APPLETS=clock,battery,$KEYBOARD_APPLET,systray,startup-notify,notify
> +END_APPLETS=$KEYBOARD_APPLET,systray,startup-notify,notify
> +END_APPLETS=battery,$END_APPLETS # feature-acpi
> +END_APPLETS=battery,$END_APPLETS # feature-apm
> +END_APPLETS=clock,$END_APPLETS
> END_APPLETS=openmoko-panel-gsm,$END_APPLETS # feature-phone
>
> matchbox-panel --start-applets $START_APPLETS --end-applets $END_APPLETS &
> diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
> index d146e83..8230e95 100644
> --- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
> +++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
> @@ -26,7 +26,7 @@ FILES_${PN} += "${datadir}/themes/Sato/index.theme"
>
> do_install() {
> # This is the set of machine features that the script has markers for
> - FEATURES="phone"
> + FEATURES="acpi apm phone"
> SCRIPT="${S}/sedder"
> rm -f $SCRIPT
> touch $SCRIPT
> --
> 2.7.4
>
> This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] matchbox-session-sato: Make the battery applet depend on machine features
@ 2017-07-30 15:24 Eran Matityahu
0 siblings, 0 replies; 4+ messages in thread
From: Eran Matityahu @ 2017-07-30 15:24 UTC (permalink / raw)
To: openembedded-core; +Cc: Eran Matityahu
matchbox-panel enables the battery plugin only if the
acpi/apm machine features are enabled,
so enable the battery applet in the session script
under the same conditions.
This avoids the 'Failed to load applet "battery"' warning at runtime,
in case these machine features are not defined.
Signed-off-by: Eran Matityahu <eran.m@variscite.com>
---
meta/recipes-sato/matchbox-sato/matchbox-session-sato/session | 5 ++++-
meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb | 2 +-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
index f6313bd..3a70574 100644
--- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
+++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato/session
@@ -20,7 +20,10 @@ matchbox-desktop &
# doesn't have the feature "foo".
START_APPLETS=showdesktop,windowselector
-END_APPLETS=clock,battery,$KEYBOARD_APPLET,systray,startup-notify,notify
+END_APPLETS=$KEYBOARD_APPLET,systray,startup-notify,notify
+END_APPLETS=battery,$END_APPLETS # feature-acpi
+END_APPLETS=battery,$END_APPLETS # feature-apm
+END_APPLETS=clock,$END_APPLETS
END_APPLETS=openmoko-panel-gsm,$END_APPLETS # feature-phone
matchbox-panel --start-applets $START_APPLETS --end-applets $END_APPLETS &
diff --git a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
index d146e83..8230e95 100644
--- a/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
+++ b/meta/recipes-sato/matchbox-sato/matchbox-session-sato_0.1.bb
@@ -26,7 +26,7 @@ FILES_${PN} += "${datadir}/themes/Sato/index.theme"
do_install() {
# This is the set of machine features that the script has markers for
- FEATURES="phone"
+ FEATURES="acpi apm phone"
SCRIPT="${S}/sedder"
rm -f $SCRIPT
touch $SCRIPT
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-05-03 8:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-02 17:33 [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Jaewon Lee
2018-05-02 18:32 ` ✗ patchtest: failure for matchbox-session-sato: Make the battery applet depend on machine features (rev2) Patchwork
2018-05-03 8:25 ` [PATCH] matchbox-session-sato: Make the battery applet depend on machine features Eran Matityahu
-- strict thread matches above, loose matches on Subject: below --
2017-07-30 15:24 Eran Matityahu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.