All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Hongxu Jia <hongxu.jia@windriver.com>
Cc: "openembedded-devel@lists.openembedded.org"
	<openembedded-devel@lists.openembedded.org>
Subject: Re: RFC: Add DISTRO_FEATURES(xfce) to control meta-xfce
Date: Mon, 1 Jul 2019 15:48:25 +0300	[thread overview]
Message-ID: <20190701124825.GA7483@localhost> (raw)
In-Reply-To: <d889a55b-ae41-dd51-c37f-a8114b234753@windriver.com>

On Fri, Jun 28, 2019 at 10:23:51PM +0800, Hongxu Jia wrote:
> On 6/28/19 8:58 PM, Adrian Bunk wrote:
> > On Fri, Jun 28, 2019 at 10:20:36AM +0800, Hongxu Jia wrote:
> > > Hi all,
> > > 
> > > Currently, I am improving OE to follow Yocto Project Compatible,
> > > One rule of Yocto Project Compatible is when a new layer is added,
> > > it should not affect other layer, but unfortunately, bbappend in
> > > meta-xfce broke the rule.
> > > 
> > > So I suggest to add DISTRO_FEATURES(xfce) as a switch to control,
> > > in short term, we could use it to control bbappend in meta-xfce,
> > > in long term, we could use it to control more xfce specific features
> > > 
> > > (As a alternative way, I also tried to remove bbappend directly,
> > > but I found the existence of bbappend is indeed necessary)
> > blueman upstream defaults to installing the files for thunar-sendto.
> > Andreas, was there any specific reason why you made this conditional
> > instead of keeping it enabled unconditional with only an RSUGGESTS
> > on thunar?
> 
> Currently DISTRO_FEATURES(xfce) is used to control this bbappend,
> 
> if we could merge the bbappend to recipe thunar, I think it is not
> 
> necessary to add DISTRO_FEATURES(xfce) in short term

I just sent
  blueman: Enable thunar support by default but don't rdepend on it
that should also fix this problem.

> > gdmflexiserver looks like a workaround for something that should be
> > fixed properly instead to me.
> > But short-term adding it to the normal lxdm recipe in meta-oe might
> > be an options?
> 
> No need to add extra lxdm, I am working to remove this bbappend,
> 
> and modify xfce4-panel to use lxdm to instead of lightdm directly,
> 
> we do not need this bbappend to provide a fake gdmflexiserver

Thanks, that looks a lot better than what I had in mind.

> //Hongxu

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed



  reply	other threads:[~2019-07-01 12:48 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-28  2:20 RFC: Add DISTRO_FEATURES(xfce) to control meta-xfce Hongxu Jia
2019-06-28 12:58 ` Adrian Bunk
2019-06-28 14:23   ` Hongxu Jia
2019-07-01 12:48     ` Adrian Bunk [this message]
2019-07-02  1:23       ` Hongxu Jia

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=20190701124825.GA7483@localhost \
    --to=bunk@stusta.de \
    --cc=hongxu.jia@windriver.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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.