* [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
@ 2015-02-10 19:19 Denys Dmytriyenko
2015-02-10 19:50 ` Nelson, Sam
0 siblings, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2015-02-10 19:19 UTC (permalink / raw)
To: meta-arago
From: Denys Dmytriyenko <denys@ti.com>
Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
meta-arago-distro/recipes-core/images/arago-image.inc | 2 +-
meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb | 2 ++
.../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb | 3 +--
3 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-arago-distro/recipes-core/images/arago-image.inc
index 98af6c3..977ce8f 100644
--- a/meta-arago-distro/recipes-core/images/arago-image.inc
+++ b/meta-arago-distro/recipes-core/images/arago-image.inc
@@ -3,7 +3,7 @@
LICENSE = "MIT"
LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
-IMAGE_FEATURES += "package-management"
+IMAGE_FEATURES += "package-management splash"
# 4KB per 1 inode should be enough
EXTRA_IMAGECMD_ext2.gz += "-i 4096"
diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
index e5ce2b5..908325f 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
@@ -3,6 +3,8 @@ COMPATIBLE_MACHINE = "omap3|omapl138|ti33x|keystone"
require arago-image.inc
+SPLASH = ""
+
IMAGE_INSTALL += "\
packagegroup-arago-base \
packagegroup-arago-console \
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb
index 704e840..c291a42 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb
@@ -1,6 +1,6 @@
DESCRIPTION = "Additional packages beyond console packages shared by TI SDKs"
LICENSE = "MIT"
-PR = "r14"
+PR = "r15"
inherit packagegroup
@@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
libpcre \
iptables \
iperf \
- psplash \
arago-gpl-notice \
nfs-utils-client \
"
--
2.2.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 19:19 [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server Denys Dmytriyenko
@ 2015-02-10 19:50 ` Nelson, Sam
2015-02-10 20:05 ` Denys Dmytriyenko
0 siblings, 1 reply; 8+ messages in thread
From: Nelson, Sam @ 2015-02-10 19:50 UTC (permalink / raw)
To: Denys Dmytriyenko, meta-arago@arago-project.org
Is this patch based on master? I am having trouble using the patch AS IS.
See comments inline.
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> Sent: Tuesday, February 10, 2015 2:19 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-tisdk:
> move psplash to IMAGE_FEATURES, sans server
>
> From: Denys Dmytriyenko <denys@ti.com>
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
> meta-arago-distro/recipes-core/images/arago-image.inc | 2 +-
> meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb | 2 ++
> .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb | 3 +--
> 3 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-
> arago-distro/recipes-core/images/arago-image.inc
> index 98af6c3..977ce8f 100644
> --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> @@ -3,7 +3,7 @@
> LICENSE = "MIT"
> LIC_FILES_CHKSUM =
> "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
>
> -IMAGE_FEATURES += "package-management"
> +IMAGE_FEATURES += "package-management splash"
>
> # 4KB per 1 inode should be enough
> EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> index e5ce2b5..908325f 100644
> --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> "omap3|omapl138|ti33x|keystone"
>
> require arago-image.inc
>
> +SPLASH = ""
> +
[Sam] I am not sure why is this needed. Appreciate if you clarify.
> IMAGE_INSTALL += "\
> packagegroup-arago-base \
> packagegroup-arago-console \
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> arago-base-tisdk.bb b/meta-arago-distro/recipes-
> core/packagegroups/packagegroup-arago-base-tisdk.bb
> index 704e840..c291a42 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-
> tisdk.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> base-tisdk.bb
> @@ -1,6 +1,6 @@
> DESCRIPTION = "Additional packages beyond console packages shared by TI
> SDKs"
> LICENSE = "MIT"
> -PR = "r14"
> +PR = "r15"
>
> inherit packagegroup
>
> @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> libpcre \
> iptables \
> iperf \
> - psplash \
[Sam] I see psplash getting removed here. Does not this be moved to a different package group? ( I don't see any addition to different package group).
> arago-gpl-notice \
> nfs-utils-client \
> "
> --
> 2.2.0
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 19:50 ` Nelson, Sam
@ 2015-02-10 20:05 ` Denys Dmytriyenko
2015-02-10 20:31 ` Nelson, Sam
0 siblings, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2015-02-10 20:05 UTC (permalink / raw)
To: Nelson, Sam; +Cc: meta-arago@arago-project.org
On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> Is this patch based on master? I am having trouble using the patch AS IS.
Ah, indeed. Fixed in v2.
> See comments inline.
>
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Tuesday, February 10, 2015 2:19 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-tisdk:
> > move psplash to IMAGE_FEATURES, sans server
> >
> > From: Denys Dmytriyenko <denys@ti.com>
> >
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> > meta-arago-distro/recipes-core/images/arago-image.inc | 2 +-
> > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb | 2 ++
> > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb | 3 +--
> > 3 files changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-
> > arago-distro/recipes-core/images/arago-image.inc
> > index 98af6c3..977ce8f 100644
> > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > @@ -3,7 +3,7 @@
> > LICENSE = "MIT"
> > LIC_FILES_CHKSUM =
> > "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> >
> > -IMAGE_FEATURES += "package-management"
> > +IMAGE_FEATURES += "package-management splash"
1. We add standard "splash" feature to IMAGE_FEATURES list for all our images.
And image class will pull in psplash by default.
> > # 4KB per 1 inode should be enough
> > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > index e5ce2b5..908325f 100644
> > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > "omap3|omapl138|ti33x|keystone"
> >
> > require arago-image.inc
> >
> > +SPLASH = ""
> > +
> [Sam] I am not sure why is this needed. Appreciate if you clarify.
2. For "server" image, we empty SPLASH variable, which is "psplash" by default.
> > IMAGE_INSTALL += "\
> > packagegroup-arago-base \
> > packagegroup-arago-console \
> > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > index 704e840..c291a42 100644
> > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-base-
> > tisdk.bb
> > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > base-tisdk.bb
> > @@ -1,6 +1,6 @@
> > DESCRIPTION = "Additional packages beyond console packages shared by TI
> > SDKs"
> > LICENSE = "MIT"
> > -PR = "r14"
> > +PR = "r15"
> >
> > inherit packagegroup
> >
> > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > libpcre \
> > iptables \
> > iperf \
> > - psplash \
> [Sam] I see psplash getting removed here. Does not this be moved to a
> different package group? ( I don't see any addition to different package
> group).
3. We no longer add psplash explicitly in the packagegroup, since image class
takes care of that for us with the steps above.
> > arago-gpl-notice \
> > nfs-utils-client \
> > "
> > --
> > 2.2.0
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 20:05 ` Denys Dmytriyenko
@ 2015-02-10 20:31 ` Nelson, Sam
2015-02-10 20:35 ` Denys Dmytriyenko
0 siblings, 1 reply; 8+ messages in thread
From: Nelson, Sam @ 2015-02-10 20:31 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, February 10, 2015 3:05 PM
> To: Nelson, Sam
> Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> tisdk: move psplash to IMAGE_FEATURES, sans server
>
> On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> > Is this patch based on master? I am having trouble using the patch AS IS.
>
> Ah, indeed. Fixed in v2.
>
>
> > See comments inline.
> >
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > Sent: Tuesday, February 10, 2015 2:19 PM
> > > To: meta-arago@arago-project.org
> > > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> tisdk:
> > > move psplash to IMAGE_FEATURES, sans server
> > >
> > > From: Denys Dmytriyenko <denys@ti.com>
> > >
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > ---
> > > meta-arago-distro/recipes-core/images/arago-image.inc | 2 +-
> > > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb | 2
> ++
> > > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb | 3
> +--
> > > 3 files changed, 4 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-
> > > arago-distro/recipes-core/images/arago-image.inc
> > > index 98af6c3..977ce8f 100644
> > > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > > @@ -3,7 +3,7 @@
> > > LICENSE = "MIT"
> > > LIC_FILES_CHKSUM =
> > > "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> > >
> > > -IMAGE_FEATURES += "package-management"
> > > +IMAGE_FEATURES += "package-management splash"
>
> 1. We add standard "splash" feature to IMAGE_FEATURES list for all our
> images.
> And image class will pull in psplash by default.
>
>
> > > # 4KB per 1 inode should be enough
> > > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> image.bb
> > > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > index e5ce2b5..908325f 100644
> > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > > "omap3|omapl138|ti33x|keystone"
> > >
> > > require arago-image.inc
> > >
> > > +SPLASH = ""
> > > +
> > [Sam] I am not sure why is this needed. Appreciate if you clarify.
>
> 2. For "server" image, we empty SPLASH variable, which is "psplash" by default.
>
>
> > > IMAGE_INSTALL += "\
> > > packagegroup-arago-base \
> > > packagegroup-arago-console \
> > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > index 704e840..c291a42 100644
> > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> base-
> > > tisdk.bb
> > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > base-tisdk.bb
> > > @@ -1,6 +1,6 @@
> > > DESCRIPTION = "Additional packages beyond console packages shared by TI
> > > SDKs"
> > > LICENSE = "MIT"
> > > -PR = "r14"
> > > +PR = "r15"
> > >
> > > inherit packagegroup
> > >
> > > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > > libpcre \
> > > iptables \
> > > iperf \
> > > - psplash \
>
> > [Sam] I see psplash getting removed here. Does not this be moved to a
> > different package group? ( I don't see any addition to different package
> > group).
>
> 3. We no longer add psplash explicitly in the packagegroup, since image class
> takes care of that for us with the steps above.
>
[Sam] Ok.
So the idea is, to disable psplash for headless distribution, probably need to remove IMAGE_FEATURES_remove = "psplash" ? Does that make sense?
Also I tried the patch v2, and I see psplash does not seem to be added automatically. Can you cross check this?
>
> > > arago-gpl-notice \
> > > nfs-utils-client \
> > > "
> > > --
> > > 2.2.0
> > >
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 20:31 ` Nelson, Sam
@ 2015-02-10 20:35 ` Denys Dmytriyenko
2015-02-10 20:46 ` Nelson, Sam
0 siblings, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2015-02-10 20:35 UTC (permalink / raw)
To: Nelson, Sam; +Cc: meta-arago@arago-project.org
On Tue, Feb 10, 2015 at 03:31:32PM -0500, Nelson, Sam wrote:
>
>
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Tuesday, February 10, 2015 3:05 PM
> > To: Nelson, Sam
> > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > tisdk: move psplash to IMAGE_FEATURES, sans server
> >
> > On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> > > Is this patch based on master? I am having trouble using the patch AS IS.
> >
> > Ah, indeed. Fixed in v2.
> >
> >
> > > See comments inline.
> > >
> > > > -----Original Message-----
> > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > > Sent: Tuesday, February 10, 2015 2:19 PM
> > > > To: meta-arago@arago-project.org
> > > > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > tisdk:
> > > > move psplash to IMAGE_FEATURES, sans server
> > > >
> > > > From: Denys Dmytriyenko <denys@ti.com>
> > > >
> > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > ---
> > > > meta-arago-distro/recipes-core/images/arago-image.inc | 2 +-
> > > > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb | 2
> > ++
> > > > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb | 3
> > +--
> > > > 3 files changed, 4 insertions(+), 3 deletions(-)
> > > >
> > > > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-
> > > > arago-distro/recipes-core/images/arago-image.inc
> > > > index 98af6c3..977ce8f 100644
> > > > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > @@ -3,7 +3,7 @@
> > > > LICENSE = "MIT"
> > > > LIC_FILES_CHKSUM =
> > > > "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> > > >
> > > > -IMAGE_FEATURES += "package-management"
> > > > +IMAGE_FEATURES += "package-management splash"
> >
> > 1. We add standard "splash" feature to IMAGE_FEATURES list for all our
> > images.
> > And image class will pull in psplash by default.
> >
> >
> > > > # 4KB per 1 inode should be enough
> > > > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > image.bb
> > > > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > > index e5ce2b5..908325f 100644
> > > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > > > "omap3|omapl138|ti33x|keystone"
> > > >
> > > > require arago-image.inc
> > > >
> > > > +SPLASH = ""
> > > > +
> > > [Sam] I am not sure why is this needed. Appreciate if you clarify.
> >
> > 2. For "server" image, we empty SPLASH variable, which is "psplash" by default.
> >
> >
> > > > IMAGE_INSTALL += "\
> > > > packagegroup-arago-base \
> > > > packagegroup-arago-console \
> > > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > > index 704e840..c291a42 100644
> > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > base-
> > > > tisdk.bb
> > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > > base-tisdk.bb
> > > > @@ -1,6 +1,6 @@
> > > > DESCRIPTION = "Additional packages beyond console packages shared by TI
> > > > SDKs"
> > > > LICENSE = "MIT"
> > > > -PR = "r14"
> > > > +PR = "r15"
> > > >
> > > > inherit packagegroup
> > > >
> > > > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > > > libpcre \
> > > > iptables \
> > > > iperf \
> > > > - psplash \
> >
> > > [Sam] I see psplash getting removed here. Does not this be moved to a
> > > different package group? ( I don't see any addition to different package
> > > group).
> >
> > 3. We no longer add psplash explicitly in the packagegroup, since image class
> > takes care of that for us with the steps above.
> >
> [Sam] Ok.
> So the idea is, to disable psplash for headless distribution, probably need
> to remove IMAGE_FEATURES_remove = "psplash" ? Does that make sense?
Step #2 above.
> Also I tried the patch v2, and I see psplash does not seem to be added
> automatically. Can you cross check this?
Which image is that? I've checked the resulting RDEPENDS for our rootfs images
and they look fine.
> >
> > > > arago-gpl-notice \
> > > > nfs-utils-client \
> > > > "
> > > > --
> > > > 2.2.0
> > > >
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 20:35 ` Denys Dmytriyenko
@ 2015-02-10 20:46 ` Nelson, Sam
2015-02-10 20:52 ` Denys Dmytriyenko
0 siblings, 1 reply; 8+ messages in thread
From: Nelson, Sam @ 2015-02-10 20:46 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, February 10, 2015 3:35 PM
> To: Nelson, Sam
> Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> tisdk: move psplash to IMAGE_FEATURES, sans server
>
> On Tue, Feb 10, 2015 at 03:31:32PM -0500, Nelson, Sam wrote:
> >
> >
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Tuesday, February 10, 2015 3:05 PM
> > > To: Nelson, Sam
> > > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > > tisdk: move psplash to IMAGE_FEATURES, sans server
> > >
> > > On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> > > > Is this patch based on master? I am having trouble using the patch AS IS.
> > >
> > > Ah, indeed. Fixed in v2.
> > >
> > >
> > > > See comments inline.
> > > >
> > > > > -----Original Message-----
> > > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > > > Sent: Tuesday, February 10, 2015 2:19 PM
> > > > > To: meta-arago@arago-project.org
> > > > > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > > tisdk:
> > > > > move psplash to IMAGE_FEATURES, sans server
> > > > >
> > > > > From: Denys Dmytriyenko <denys@ti.com>
> > > > >
> > > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > > ---
> > > > > meta-arago-distro/recipes-core/images/arago-image.inc | 2
> +-
> > > > > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> | 2
> > > ++
> > > > > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb |
> 3
> > > +--
> > > > > 3 files changed, 4 insertions(+), 3 deletions(-)
> > > > >
> > > > > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc
> b/meta-
> > > > > arago-distro/recipes-core/images/arago-image.inc
> > > > > index 98af6c3..977ce8f 100644
> > > > > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > @@ -3,7 +3,7 @@
> > > > > LICENSE = "MIT"
> > > > > LIC_FILES_CHKSUM =
> > > > >
> "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> > > > >
> > > > > -IMAGE_FEATURES += "package-management"
> > > > > +IMAGE_FEATURES += "package-management splash"
> > >
> > > 1. We add standard "splash" feature to IMAGE_FEATURES list for all our
> > > images.
> > > And image class will pull in psplash by default.
> > >
> > >
> > > > > # 4KB per 1 inode should be enough
> > > > > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > > image.bb
> > > > > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > > > index e5ce2b5..908325f 100644
> > > > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> image.bb
> > > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> image.bb
> > > > > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > > > > "omap3|omapl138|ti33x|keystone"
> > > > >
> > > > > require arago-image.inc
> > > > >
> > > > > +SPLASH = ""
> > > > > +
> > > > [Sam] I am not sure why is this needed. Appreciate if you clarify.
> > >
> > > 2. For "server" image, we empty SPLASH variable, which is "psplash" by
> default.
> > >
> > >
> > > > > IMAGE_INSTALL += "\
> > > > > packagegroup-arago-base \
> > > > > packagegroup-arago-console \
> > > > > diff --git a/meta-arago-distro/recipes-
> core/packagegroups/packagegroup-
> > > > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > > > index 704e840..c291a42 100644
> > > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> arago-
> > > base-
> > > > > tisdk.bb
> > > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> arago-
> > > > > base-tisdk.bb
> > > > > @@ -1,6 +1,6 @@
> > > > > DESCRIPTION = "Additional packages beyond console packages shared
> by TI
> > > > > SDKs"
> > > > > LICENSE = "MIT"
> > > > > -PR = "r14"
> > > > > +PR = "r15"
> > > > >
> > > > > inherit packagegroup
> > > > >
> > > > > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > > > > libpcre \
> > > > > iptables \
> > > > > iperf \
> > > > > - psplash \
> > >
> > > > [Sam] I see psplash getting removed here. Does not this be moved to a
> > > > different package group? ( I don't see any addition to different package
> > > > group).
> > >
> > > 3. We no longer add psplash explicitly in the packagegroup, since image
> class
> > > takes care of that for us with the steps above.
> > >
> > [Sam] Ok.
>
> > So the idea is, to disable psplash for headless distribution, probably need
> > to remove IMAGE_FEATURES_remove = "psplash" ? Does that make sense?
>
> Step #2 above.
[Sam] I thought step #2 enable "psplash" as default for server image. That is why I was thinking about how to remove for keystone2 server filesystem. Am I missing something?
>
>
> > Also I tried the patch v2, and I see psplash does not seem to be added
> > automatically. Can you cross check this?
>
> Which image is that? I've checked the resulting RDEPENDS for our rootfs
> images
> and they look fine.
[Sam] I built tisdk-server-rootfs-image (built for k2hk-evm). I did not see psplash in the file system built.
>
>
> > >
> > > > > arago-gpl-notice \
> > > > > nfs-utils-client \
> > > > > "
> > > > > --
> > > > > 2.2.0
> > > > >
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > meta-arago@arago-project.org
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 20:46 ` Nelson, Sam
@ 2015-02-10 20:52 ` Denys Dmytriyenko
2015-02-10 20:56 ` Nelson, Sam
0 siblings, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2015-02-10 20:52 UTC (permalink / raw)
To: Nelson, Sam; +Cc: meta-arago@arago-project.org
On Tue, Feb 10, 2015 at 03:46:40PM -0500, Nelson, Sam wrote:
>
>
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Tuesday, February 10, 2015 3:35 PM
> > To: Nelson, Sam
> > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > tisdk: move psplash to IMAGE_FEATURES, sans server
> >
> > On Tue, Feb 10, 2015 at 03:31:32PM -0500, Nelson, Sam wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Dmytriyenko, Denys
> > > > Sent: Tuesday, February 10, 2015 3:05 PM
> > > > To: Nelson, Sam
> > > > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > > > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > > > tisdk: move psplash to IMAGE_FEATURES, sans server
> > > >
> > > > On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> > > > > Is this patch based on master? I am having trouble using the patch AS IS.
> > > >
> > > > Ah, indeed. Fixed in v2.
> > > >
> > > >
> > > > > See comments inline.
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > > > > Sent: Tuesday, February 10, 2015 2:19 PM
> > > > > > To: meta-arago@arago-project.org
> > > > > > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > > > tisdk:
> > > > > > move psplash to IMAGE_FEATURES, sans server
> > > > > >
> > > > > > From: Denys Dmytriyenko <denys@ti.com>
> > > > > >
> > > > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > > > ---
> > > > > > meta-arago-distro/recipes-core/images/arago-image.inc | 2
> > +-
> > > > > > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > | 2
> > > > ++
> > > > > > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb |
> > 3
> > > > +--
> > > > > > 3 files changed, 4 insertions(+), 3 deletions(-)
> > > > > >
> > > > > > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc
> > b/meta-
> > > > > > arago-distro/recipes-core/images/arago-image.inc
> > > > > > index 98af6c3..977ce8f 100644
> > > > > > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > > @@ -3,7 +3,7 @@
> > > > > > LICENSE = "MIT"
> > > > > > LIC_FILES_CHKSUM =
> > > > > >
> > "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> > > > > >
> > > > > > -IMAGE_FEATURES += "package-management"
> > > > > > +IMAGE_FEATURES += "package-management splash"
> > > >
> > > > 1. We add standard "splash" feature to IMAGE_FEATURES list for all our
> > > > images.
> > > > And image class will pull in psplash by default.
> > > >
> > > >
> > > > > > # 4KB per 1 inode should be enough
> > > > > > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > > > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > > > image.bb
> > > > > > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > > > > index e5ce2b5..908325f 100644
> > > > > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > image.bb
> > > > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > image.bb
> > > > > > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > > > > > "omap3|omapl138|ti33x|keystone"
> > > > > >
> > > > > > require arago-image.inc
> > > > > >
> > > > > > +SPLASH = ""
> > > > > > +
> > > > > [Sam] I am not sure why is this needed. Appreciate if you clarify.
> > > >
> > > > 2. For "server" image, we empty SPLASH variable, which is "psplash" by
> > default.
> > > >
> > > >
> > > > > > IMAGE_INSTALL += "\
> > > > > > packagegroup-arago-base \
> > > > > > packagegroup-arago-console \
> > > > > > diff --git a/meta-arago-distro/recipes-
> > core/packagegroups/packagegroup-
> > > > > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > > > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > > > > index 704e840..c291a42 100644
> > > > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > arago-
> > > > base-
> > > > > > tisdk.bb
> > > > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > arago-
> > > > > > base-tisdk.bb
> > > > > > @@ -1,6 +1,6 @@
> > > > > > DESCRIPTION = "Additional packages beyond console packages shared
> > by TI
> > > > > > SDKs"
> > > > > > LICENSE = "MIT"
> > > > > > -PR = "r14"
> > > > > > +PR = "r15"
> > > > > >
> > > > > > inherit packagegroup
> > > > > >
> > > > > > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > > > > > libpcre \
> > > > > > iptables \
> > > > > > iperf \
> > > > > > - psplash \
> > > >
> > > > > [Sam] I see psplash getting removed here. Does not this be moved to a
> > > > > different package group? ( I don't see any addition to different package
> > > > > group).
> > > >
> > > > 3. We no longer add psplash explicitly in the packagegroup, since image
> > class
> > > > takes care of that for us with the steps above.
> > > >
> > > [Sam] Ok.
> >
> > > So the idea is, to disable psplash for headless distribution, probably need
> > > to remove IMAGE_FEATURES_remove = "psplash" ? Does that make sense?
> >
> > Step #2 above.
> [Sam] I thought step #2 enable "psplash" as default for server image. That
> is why I was thinking about how to remove for keystone2 server filesystem.
> Am I missing something?
No, psplash is already a default value for "splash" image feature. Hence I
disable it for the server image - isn't it what you needed from the beginning?
> > > Also I tried the patch v2, and I see psplash does not seem to be added
> > > automatically. Can you cross check this?
> >
> > Which image is that? I've checked the resulting RDEPENDS for our rootfs
> > images
> > and they look fine.
> [Sam] I built tisdk-server-rootfs-image (built for k2hk-evm). I did not see
> psplash in the file system built.
Exactly! Since server image is for headless platforms, there shouldn't be
psplash running.
> > > > > > arago-gpl-notice \
> > > > > > nfs-utils-client \
> > > > > > "
> > > > > > --
> > > > > > 2.2.0
> > > > > >
> > > > > > _______________________________________________
> > > > > > meta-arago mailing list
> > > > > > meta-arago@arago-project.org
> > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > meta-arago@arago-project.org
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server
2015-02-10 20:52 ` Denys Dmytriyenko
@ 2015-02-10 20:56 ` Nelson, Sam
0 siblings, 0 replies; 8+ messages in thread
From: Nelson, Sam @ 2015-02-10 20:56 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, February 10, 2015 3:52 PM
> To: Nelson, Sam
> Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> tisdk: move psplash to IMAGE_FEATURES, sans server
>
> On Tue, Feb 10, 2015 at 03:46:40PM -0500, Nelson, Sam wrote:
> >
> >
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Tuesday, February 10, 2015 3:35 PM
> > > To: Nelson, Sam
> > > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-base-
> > > tisdk: move psplash to IMAGE_FEATURES, sans server
> > >
> > > On Tue, Feb 10, 2015 at 03:31:32PM -0500, Nelson, Sam wrote:
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Dmytriyenko, Denys
> > > > > Sent: Tuesday, February 10, 2015 3:05 PM
> > > > > To: Nelson, Sam
> > > > > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > > > > Subject: Re: [meta-arago] [PATCH] arago-image, packagegroup-arago-
> base-
> > > > > tisdk: move psplash to IMAGE_FEATURES, sans server
> > > > >
> > > > > On Tue, Feb 10, 2015 at 07:50:05PM +0000, Nelson, Sam wrote:
> > > > > > Is this patch based on master? I am having trouble using the patch AS
> IS.
> > > > >
> > > > > Ah, indeed. Fixed in v2.
> > > > >
> > > > >
> > > > > > See comments inline.
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > > > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > > > > > Sent: Tuesday, February 10, 2015 2:19 PM
> > > > > > > To: meta-arago@arago-project.org
> > > > > > > Subject: [meta-arago] [PATCH] arago-image, packagegroup-arago-
> base-
> > > > > tisdk:
> > > > > > > move psplash to IMAGE_FEATURES, sans server
> > > > > > >
> > > > > > > From: Denys Dmytriyenko <denys@ti.com>
> > > > > > >
> > > > > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > > > > ---
> > > > > > > meta-arago-distro/recipes-core/images/arago-image.inc |
> 2
> > > +-
> > > > > > > meta-arago-distro/recipes-core/images/tisdk-server-rootfs-image.bb
> > > | 2
> > > > > ++
> > > > > > > .../recipes-core/packagegroups/packagegroup-arago-base-tisdk.bb
> |
> > > 3
> > > > > +--
> > > > > > > 3 files changed, 4 insertions(+), 3 deletions(-)
> > > > > > >
> > > > > > > diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > b/meta-
> > > > > > > arago-distro/recipes-core/images/arago-image.inc
> > > > > > > index 98af6c3..977ce8f 100644
> > > > > > > --- a/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > > > +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
> > > > > > > @@ -3,7 +3,7 @@
> > > > > > > LICENSE = "MIT"
> > > > > > > LIC_FILES_CHKSUM =
> > > > > > >
> > > "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58"
> > > > > > >
> > > > > > > -IMAGE_FEATURES += "package-management"
> > > > > > > +IMAGE_FEATURES += "package-management splash"
> > > > >
> > > > > 1. We add standard "splash" feature to IMAGE_FEATURES list for all our
> > > > > images.
> > > > > And image class will pull in psplash by default.
> > > > >
> > > > >
> > > > > > > # 4KB per 1 inode should be enough
> > > > > > > EXTRA_IMAGECMD_ext2.gz += "-i 4096"
> > > > > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-
> rootfs-
> > > > > image.bb
> > > > > > > b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> image.bb
> > > > > > > index e5ce2b5..908325f 100644
> > > > > > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > > image.bb
> > > > > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-rootfs-
> > > image.bb
> > > > > > > @@ -3,6 +3,8 @@ COMPATIBLE_MACHINE =
> > > > > > > "omap3|omapl138|ti33x|keystone"
> > > > > > >
> > > > > > > require arago-image.inc
> > > > > > >
> > > > > > > +SPLASH = ""
> > > > > > > +
> > > > > > [Sam] I am not sure why is this needed. Appreciate if you clarify.
> > > > >
> > > > > 2. For "server" image, we empty SPLASH variable, which is "psplash" by
> > > default.
[Sam] Ok. Looks like I misunderstood this statement.
> > > > >
> > > > >
> > > > > > > IMAGE_INSTALL += "\
> > > > > > > packagegroup-arago-base \
> > > > > > > packagegroup-arago-console \
> > > > > > > diff --git a/meta-arago-distro/recipes-
> > > core/packagegroups/packagegroup-
> > > > > > > arago-base-tisdk.bb b/meta-arago-distro/recipes-
> > > > > > > core/packagegroups/packagegroup-arago-base-tisdk.bb
> > > > > > > index 704e840..c291a42 100644
> > > > > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > > arago-
> > > > > base-
> > > > > > > tisdk.bb
> > > > > > > +++ b/meta-arago-distro/recipes-
> core/packagegroups/packagegroup-
> > > arago-
> > > > > > > base-tisdk.bb
> > > > > > > @@ -1,6 +1,6 @@
> > > > > > > DESCRIPTION = "Additional packages beyond console packages
> shared
> > > by TI
> > > > > > > SDKs"
> > > > > > > LICENSE = "MIT"
> > > > > > > -PR = "r14"
> > > > > > > +PR = "r15"
> > > > > > >
> > > > > > > inherit packagegroup
> > > > > > >
> > > > > > > @@ -12,7 +12,6 @@ RDEPENDS_${PN} = "\
> > > > > > > libpcre \
> > > > > > > iptables \
> > > > > > > iperf \
> > > > > > > - psplash \
> > > > >
> > > > > > [Sam] I see psplash getting removed here. Does not this be moved to a
> > > > > > different package group? ( I don't see any addition to different
> package
> > > > > > group).
> > > > >
> > > > > 3. We no longer add psplash explicitly in the packagegroup, since image
> > > class
> > > > > takes care of that for us with the steps above.
> > > > >
> > > > [Sam] Ok.
> > >
> > > > So the idea is, to disable psplash for headless distribution, probably need
> > > > to remove IMAGE_FEATURES_remove = "psplash" ? Does that make
> sense?
> > >
> > > Step #2 above.
>
> > [Sam] I thought step #2 enable "psplash" as default for server image. That
> > is why I was thinking about how to remove for keystone2 server filesystem.
> > Am I missing something?
>
> No, psplash is already a default value for "splash" image feature. Hence I
> disable it for the server image - isn't it what you needed from the beginning?
>
>
> > > > Also I tried the patch v2, and I see psplash does not seem to be added
> > > > automatically. Can you cross check this?
> > >
> > > Which image is that? I've checked the resulting RDEPENDS for our rootfs
> > > images
> > > and they look fine.
>
> > [Sam] I built tisdk-server-rootfs-image (built for k2hk-evm). I did not see
> > psplash in the file system built.
>
> Exactly! Since server image is for headless platforms, there shouldn't be
> psplash running.
[Sam] Makes sense. Thanks.
>
>
> > > > > > > arago-gpl-notice \
> > > > > > > nfs-utils-client \
> > > > > > > "
> > > > > > > --
> > > > > > > 2.2.0
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > meta-arago mailing list
> > > > > > > meta-arago@arago-project.org
> > > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > > > _______________________________________________
> > > > > > meta-arago mailing list
> > > > > > meta-arago@arago-project.org
> > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-02-10 20:56 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-10 19:19 [PATCH] arago-image, packagegroup-arago-base-tisdk: move psplash to IMAGE_FEATURES, sans server Denys Dmytriyenko
2015-02-10 19:50 ` Nelson, Sam
2015-02-10 20:05 ` Denys Dmytriyenko
2015-02-10 20:31 ` Nelson, Sam
2015-02-10 20:35 ` Denys Dmytriyenko
2015-02-10 20:46 ` Nelson, Sam
2015-02-10 20:52 ` Denys Dmytriyenko
2015-02-10 20:56 ` Nelson, Sam
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.