[Buildroot] [PATCH 16/32 v2] package/matchbox-common: fix dependencies

Yann E. MORIN yann.morin.1998 at free.fr
Tue Jun 9 21:51:04 UTC 2015


Thomas, All,

On 2015-06-09 23:04 +0200, Thomas Petazzoni spake thusly:
> On Wed,  3 Jun 2015 22:41:33 +0200, Yann E. MORIN wrote:
> > matchbox-common can be used by both matchbox-desktop and matchbox-panel,
> > so it needs to depend on either, not both.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> > Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
> > ---
> >  package/matchbox-common/Config.in | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/package/matchbox-common/Config.in b/package/matchbox-common/Config.in
> > index 33d6cc4..5a038e1 100644
> > --- a/package/matchbox-common/Config.in
> > +++ b/package/matchbox-common/Config.in
> > @@ -1,7 +1,7 @@
> >  config BR2_PACKAGE_MATCHBOX_COMMON
> >  	bool "matchbox-common"
> >  	select BR2_PACKAGE_MATCHBOX_LIB
> > -	depends on BR2_PACKAGE_MATCHBOX_PANEL && BR2_PACKAGE_MATCHBOX_DESKTOP
> > +	depends on BR2_PACKAGE_MATCHBOX_PANEL || BR2_PACKAGE_MATCHBOX_DESKTOP
> >  	help
> >  	  Common desktop folders and matchbox-session script.
> >  
> 
> I'll apply because I want to apply the rest of the series, but why are
> we doing this in the first place? I don't think packages in Buildroot
> have a "depends on" on things that can *use* them. The dependencies in
> Buildroot are always the other way around: you use a 'depends on' or a
> 'select' because you need some other thing.
> 
> Why isn't matchbox-common maybe always visible, like we do for all
> packages ?

As far as I understand it, matchbox-common installs the "desktop
folders" which are only meaningful with a "desktop manager", i.e. either
matchbox-panel or matchbox-desktop.

Conversely, neither "matchbox-panel" nor "matchbox-desktop" require
"matchbox-common".

So I still believe the dependency is correct:
  - no reason to expose matchbox-common if neither matchbox-panel nor
    matchbox-desktop is available,
  - expose matchbox-common when either or both are enabled,
  - no reason to forcibly select matchbox-common from either of those
    two packages.

Of course, someone actually using matchbox may well prove me wrong. ;-)
Until then, that's all I can explain of what I understand of the
matchbox stack.

[
Unrelated to that patch, but worth commenting on, however:

Besides, matchbox-panel and matchbox-desktop are not mutually exclusive
at build time; at runtime, they can't both run simultaneously on the
same X display, probably (it would be like running both Kwin and Unity
at the same time on the same X display).
]

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'



More information about the buildroot mailing list