> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, February 06, 2015 2:43 PM
> To: Cooper Jr., Franklin
> Cc: Nelson, Sam; [email protected]
> Subject: Re: [meta-arago] [PATCH] branding-mcsdk: Adding MCSDK branding
> include file
> 
> On Fri, Feb 06, 2015 at 03:32:42PM -0500, Cooper Jr., Franklin wrote:
> >
> >
> > > -----Original Message-----
> > > From: [email protected] [mailto:meta-arago-
> > > [email protected]] On Behalf Of Nelson, Sam
> > > Sent: Friday, February 06, 2015 1:51 PM
> > > To: Dmytriyenko, Denys
> > > Cc: [email protected]
> > > Subject: Re: [meta-arago] [PATCH] branding-mcsdk: Adding MCSDK
> > > branding include file
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Dmytriyenko, Denys
> > > > Sent: Friday, February 06, 2015 2:00 PM
> > > > To: Nelson, Sam
> > > > Cc: [email protected]
> > > > Subject: Re: [meta-arago] [PATCH] branding-mcsdk: Adding MCSDK
> > > > branding include file
> > > >
> > > > On Fri, Feb 06, 2015 at 01:54:56PM -0500, Sam Nelson wrote:
> > > > > - To control preferred version and providers for MCSDK
> > > > >
> > > > > Signed-off-by: Sam Nelson <[email protected]>
> > > > > ---
> > > > >  .../conf/distro/include/branding-mcsdk.inc         |   20
> > > > ++++++++++++++++++++
> > > > >  1 file changed, 20 insertions(+)  create mode 100644
> > > > > meta-arago-distro/conf/distro/include/branding-
> > > > mcsdk.inc
> > > > >
> > > > > diff --git
> > > > > a/meta-arago-distro/conf/distro/include/branding-mcsdk.inc
> > > > b/meta-arago-distro/conf/distro/include/branding-mcsdk.inc
> > > > > new file mode 100644
> > > > > index 0000000..d023b67
> > > > > --- /dev/null
> > > > > +++ b/meta-arago-distro/conf/distro/include/branding-mcsdk.inc
> > > > > @@ -0,0 +1,20 @@
> > > > > +# MCSDK branding configuration
> > > > > +
> > > > > +# Use the MCSDK supported versions PREFERRED_VERSION_libnl =
> > > > > +"3.2.24"
> > > > > +
> > > > > +DISTRO_FEATURES += "x11 opengl"
> > > >
> > > > I assume x11 is to make Java happy? I don't think you have x11 env...
> > > >
> > > [Sam] Yes it was to make Java happy.
> > [Franklin] I thought I remembered that the mcsdk were meant for headless
> devices.
> > If so I would recommend taking a look at the discussion here:
> > https://github.com/woglinde/meta-java/issues/45
> >
> > Pulling in x11 and gtk seems painful especially since you're not using it.
> > Anyway it's something to look into if it is indeed meant for headless 
> > devices.
> > But its ur branding file so my above comments aren't meant to block this
> patch from being accepted.
> 
> Well, as explained in that tracker link you provided, pulling X libs for the 
> build
> doesn't necessary mean you'll end up with X component on the target.
> Yes, disabling AWT is a nice touch, hopefully it gets integrated into 
> meta-java
[Franklin] Maybe for the very specific x11 recipes but pulling the generic gtk+ 
package I'm not sure things will end up being as lucky. The changes mentioned 
in the link specifically drops the gtk dependency. Also adding X11 to your 
distro features I'm assuming will affect quite a bit of recipes that may pull 
in additional x11 dependencies that they may have avoided otherwise.

Anyway I was making assumptions so if that isn't occurring in their current 
builds feel free to ignore my crazy rambling :)
> 
> 
> > > > > +# Java preferences.
> > > > > +#PREFERRED_VERSION_openjdk-7-jre = "25b30-2.3.12"
> > > > > +PREFERRED_VERSION_icedtea7-native = "2.1.3"
> > > > > +PREFERRED_PROVIDER_openjdk-7-jre = "openjdk-7-jre"
> > > > > +
> > > > > +PREFERRED_PROVIDER_virtual/java-initial = "cacao-initial"
> > > > > +
> > > > > +PREFERRED_PROVIDER_virtual/java-native = "jamvm-native"
> > > > > +PREFERRED_PROVIDER_virtual/javac-native = "ecj-bootstrap-native"
> > > > > +
> > > > > +PREFERRED_PROVIDER_jpeg-native = "jpeg-native"
> > > > > +#PREFERRED_PROVIDER_jpeg-native = "libjpeg-turbo-native"
> > > > > +PREFERRED_PROVIDER_glibc-gconv-iso8859-1 = "nativesdk-eglibc-
> locale"
> > > > > --
> > > > > 1.7.9.5
> > > > >
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > [email protected]
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > _______________________________________________
> > > meta-arago mailing list
> > > [email protected]
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to