All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: "Bajjuri, Praneeth" <praneeth@ti.com>
Cc: meta-arago@arago-project.org
Subject: Re: [dunfell/master][PATCH 1/4] chromium: apply bbappend with fix dynamically based on "browser-layer" presence
Date: Mon, 3 May 2021 17:32:51 -0400	[thread overview]
Message-ID: <20210503213251.GE21867@denix.org> (raw)
In-Reply-To: <ee17cc62-5edd-d830-f109-6cb3179c791e@ti.com>

On Mon, May 03, 2021 at 04:29:06PM -0500, Bajjuri, Praneeth wrote:
> 
> 
> On 5/3/2021 4:12 PM, Denys Dmytriyenko wrote:
> >Praneeth,
> >
> >Any plans to merge this series?
> >
> >Even though the config is disabled by default, it does some recipe cleanup and
> >establishes dynamic layer dependency framework.
> 
> Thanks for the bump, pulled to dunfell-next.
> 
> Will sync to -master in  a day once the nightly build is done.

Thanks! Might want to pull corresponding oe-layersetup patch as well, even 
though it only adds extra comments :)


> --Praneeth
> 
> >
> >
> >On Tue, Mar 23, 2021 at 05:15:04PM -0400, Denys Dmytriyenko wrote:
> >>From: Denys Dmytriyenko <denys@konsulko.com>
> >>
> >>This solves the dangling bbappend warning when browser-layer is not enabled.
> >>
> >>As chromium still requires Python2, corresponding meta-python2 layer is required.
> >>
> >>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> >>---
> >>  meta-arago-distro/conf/layer.conf                                     | 4 ++++
> >>  .../chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch     | 0
> >>  .../recipes-browser/chromium/chromium-ozone-wayland_%.bbappend        | 0
> >>  3 files changed, 4 insertions(+)
> >>  rename meta-arago-distro/{ => dynamic-layers/browser-layer}/recipes-browser/chromium/chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch (100%)
> >>  rename meta-arago-distro/{ => dynamic-layers/browser-layer}/recipes-browser/chromium/chromium-ozone-wayland_%.bbappend (100%)
> >>
> >>diff --git a/meta-arago-distro/conf/layer.conf b/meta-arago-distro/conf/layer.conf
> >>index 2931bf9..9014511 100644
> >>--- a/meta-arago-distro/conf/layer.conf
> >>+++ b/meta-arago-distro/conf/layer.conf
> >>@@ -24,6 +24,10 @@ LAYERRECOMMENDS_meta-arago-distro = " \
> >>      browser-layer \
> >>  "
> >>+BBFILES_DYNAMIC += " \
> >>+    browser-layer:${LAYERDIR}/dynamic-layers/browser-layer/recipes*/*/*.bbappend \
> >>+"
> >>+
> >>  BB_DANGLINGAPPENDS_WARNONLY = "true"
> >>  SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
> >>diff --git a/meta-arago-distro/recipes-browser/chromium/chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch b/meta-arago-distro/dynamic-layers/browser-layer/recipes-browser/chromium/chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch
> >>similarity index 100%
> >>rename from meta-arago-distro/recipes-browser/chromium/chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch
> >>rename to meta-arago-distro/dynamic-layers/browser-layer/recipes-browser/chromium/chromium-ozone-wayland/chromium-75.0.3770.80-SIOCGSTAMP.patch
> >>diff --git a/meta-arago-distro/recipes-browser/chromium/chromium-ozone-wayland_%.bbappend b/meta-arago-distro/dynamic-layers/browser-layer/recipes-browser/chromium/chromium-ozone-wayland_%.bbappend
> >>similarity index 100%
> >>rename from meta-arago-distro/recipes-browser/chromium/chromium-ozone-wayland_%.bbappend
> >>rename to meta-arago-distro/dynamic-layers/browser-layer/recipes-browser/chromium/chromium-ozone-wayland_%.bbappend
> >>-- 
> >>2.7.4
> >>
> >>_______________________________________________
> >>meta-arago mailing list
> >>meta-arago@arago-project.org
> >>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >>
> >
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964


  reply	other threads:[~2021-05-03 21:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-23 21:15 [dunfell/master][PATCH 1/4] chromium: apply bbappend with fix dynamically based on "browser-layer" presence Denys Dmytriyenko
2021-03-23 21:15 ` [dunfell/master][PATCH 2/4] images/packagegroups: add extra browser-layer check when including chromium Denys Dmytriyenko
2021-03-24 22:44   ` [EXTERNAL] " Tammana, Gowtham
2021-03-23 21:15 ` [dunfell/master][PATCH 3/4] chromium: fix building of version 75 against more recent pulseaudio Denys Dmytriyenko
2021-03-23 21:15 ` [dunfell/master][PATCH 4/4] qtwebengine: fix building latest qtwebengine with meta-python2 Denys Dmytriyenko
2021-03-24 14:30   ` [EXTERNAL] " Tammana, Gowtham
2021-05-03 21:12 ` [dunfell/master][PATCH 1/4] chromium: apply bbappend with fix dynamically based on "browser-layer" presence Denys Dmytriyenko
2021-05-03 21:29   ` Bajjuri, Praneeth
2021-05-03 21:32     ` Denys Dmytriyenko [this message]
2021-05-03 21:37       ` Bajjuri, Praneeth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210503213251.GE21867@denix.org \
    --to=denis@denix.org \
    --cc=meta-arago@arago-project.org \
    --cc=praneeth@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.