[PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Andrea Adami]

---
 conf/distro/angstrom-2008.1.conf |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/conf/distro/angstrom-2008.1.conf b/conf/distro/angstrom-2008.1.conf
index cde01e1..d8f53af 100644
--- a/conf/distro/angstrom-2008.1.conf
+++ b/conf/distro/angstrom-2008.1.conf
@@ -141,6 +141,9 @@ PREFERRED_PROVIDER_virtual/libx11       ?= "libx11"
 PREFERRED_PROVIDER_virtual/libxine      ?= "libxine-x11"
 PREFERRED_PROVIDER_virtual/xserver      ?= "xserver-kdrive"
 
+#Silence warning during parsing
+PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"
+
 # Others:
 PREFERRED_PROVIDER_linux-libc-headers = "linux-libc-headers"
 PREFERRED_PROVIDER_dbus-glib             = "dbus-glib"
@@ -176,6 +179,8 @@ PREFERRED_PROVIDER_gdk-pixbuf-loader-xpm  ?= "gtk+"
 #Silence a warning during parsing
 PREFERRED_PROVIDER_task-bootstrap = "task-bootstrap"
 
+
+
 #This is needed to get a correct PACKAGE_ARCH for packages that have PACKAGE_ARCH = ${MACHINE_ARCH}
 ARM_ABI ?= "${@['','oabi'][bb.data.getVar('MACHINE',d) in ['collie','h3600', 'h3800', 'simpad', 'htcwallaby']]}"
 require conf/distro/include/angstrom${ARM_ABI}.inc
-- 
1.6.0.6

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Andrea Adami]

was:
NOTE: Preparing runqueue
ERROR: Multiple .bb files are due to be built which each provide
virtual/psplash
(/oe/org.openembedded.dev/recipes/psplash/psplash-angstrom_svn.bb
/oe/org.openembedded.dev/recipes/psplash/psplash_svn.bb).
 This usually means one provides something the other doesn't and should.


IMHO the introduction of the new virtual/psplash implies each distro
sets a preferred provider.
Attached patch for Angstrom.

Regards

Andrea

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Koen Kooi]

On 26-03-09 01:10, Andrea Adami wrote:

> +#Silence warning during parsing
> +PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"

NACK, the 'psplash' package should get deleted and replaced with 
'psplash-oe' or something. It was left in place to give people time to 
switch to the new way of installing branded psplash images.

regards,

Koen

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Otavio Salvador]

Koen Kooi <k.kooi@student.utwente.nl> writes:

> On 26-03-09 01:10, Andrea Adami wrote:
>
>> +#Silence warning during parsing
>> +PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"
>
> NACK, the 'psplash' package should get deleted and replaced with
> psplash-oe' or something. It was left in place to give people time to
> switch to the new way of installing branded psplash images.

Hello Koen but I belive those are two different issues.

I added the virtual/psplash package to allow others to properly depends
on an alternative. Currently the only recipe using it is
initramfs-module-psplash but others may come.

I fully agree that we should drop plain psplash recipe and have
people to use the branded one but I also think angstrom needs to
properly set the preferred version since it will then allow that
its users to use it without much hassle.

My 2c.

-- 
Otavio Salvador                  O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Koen Kooi]

On 30-03-09 13:47, Otavio Salvador wrote:
> Koen Kooi<k.kooi@student.utwente.nl>  writes:
>
>> On 26-03-09 01:10, Andrea Adami wrote:
>>
>>> +#Silence warning during parsing
>>> +PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"
>>
>> NACK, the 'psplash' package should get deleted and replaced with
>> psplash-oe' or something. It was left in place to give people time to
>> switch to the new way of installing branded psplash images.
>
> Hello Koen but I belive those are two different issues.

> I added the virtual/psplash package to allow others to properly depends
> on an alternative. Currently the only recipe using it is
> initramfs-module-psplash but others may come.

I don't think virtual/psplash is the way to go, but I don't have a 
better suggestion.

> I fully agree that we should drop plain psplash recipe and have
> people to use the branded one but I also think angstrom needs to
> properly set the preferred version since it will then allow that
> its users to use it without much hassle.

Unfortunately this will end up with people spreading FUD like "can't 
change psplash logo in angstrom" just because they see that line in 
angstrom.conf.
But then again, people are still claiming angstrom is bloated and 
minimal isn't when minimal is currently angstrom.conf with sed 
s/angstrom/minimal/ applied to it...

regards,

Koen

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Otavio Salvador]

Koen Kooi <k.kooi@student.utwente.nl> writes:

> On 30-03-09 13:47, Otavio Salvador wrote:
>> Koen Kooi<k.kooi@student.utwente.nl>  writes:
>>
>>> On 26-03-09 01:10, Andrea Adami wrote:
>>>
>>>> +#Silence warning during parsing
>>>> +PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"
>>>
>>> NACK, the 'psplash' package should get deleted and replaced with
>>> psplash-oe' or something. It was left in place to give people time to
>>> switch to the new way of installing branded psplash images.
>>
>> Hello Koen but I belive those are two different issues.
>
>> I added the virtual/psplash package to allow others to properly depends
>> on an alternative. Currently the only recipe using it is
>> initramfs-module-psplash but others may come.
>
> I don't think virtual/psplash is the way to go, but I don't have a
> better suggestion.

So what we should do? IMO if there's no better solution we use this
until we find one.

>> I fully agree that we should drop plain psplash recipe and have
>> people to use the branded one but I also think angstrom needs to
>> properly set the preferred version since it will then allow that
>> its users to use it without much hassle.
>
> Unfortunately this will end up with people spreading FUD like "can't
> change psplash logo in angstrom" just because they see that line in
> angstrom.conf.

Well, we could add a comment for that in the angstrom.conf file.

Something like:

# This enforces a specific brand to be used, in our case the angstrom
# one. In case you want to change the logo or something like that we
# suggest you to create another recipe (e.g: psplash-<brand>) for that
# and change this line according.
PREFERRED_VERSION_...

> But then again, people are still claiming angstrom is bloated and
> minimal isn't when minimal is currently angstrom.conf with sed
> s/angstrom/minimal/ applied to it...

I think this is another problem; even minimal isn't in a useful state
and it does need to be fixed, this doesn't fix the virtual dependency
problem.

So?

-- 
Otavio Salvador                  O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Koen Kooi]

On 30-03-09 17:33, Otavio Salvador wrote:
> Koen Kooi<k.kooi@student.utwente.nl>  writes:
>
>> On 30-03-09 13:47, Otavio Salvador wrote:
>>> Koen Kooi<k.kooi@student.utwente.nl>   writes:
>>>
>>>> On 26-03-09 01:10, Andrea Adami wrote:
>>>>
>>>>> +#Silence warning during parsing
>>>>> +PREFERRED_PROVIDER_virtual/psplash ?= "psplash-angstrom"
>>>>
>>>> NACK, the 'psplash' package should get deleted and replaced with
>>>> psplash-oe' or something. It was left in place to give people time to
>>>> switch to the new way of installing branded psplash images.
>>>
>>> Hello Koen but I belive those are two different issues.
>>
>>> I added the virtual/psplash package to allow others to properly depends
>>> on an alternative. Currently the only recipe using it is
>>> initramfs-module-psplash but others may come.
>>
>> I don't think virtual/psplash is the way to go, but I don't have a
>> better suggestion.
>
> So what we should do? IMO if there's no better solution we use this
> until we find one.
>
>>> I fully agree that we should drop plain psplash recipe and have
>>> people to use the branded one but I also think angstrom needs to
>>> properly set the preferred version since it will then allow that
>>> its users to use it without much hassle.
>>
>> Unfortunately this will end up with people spreading FUD like "can't
>> change psplash logo in angstrom" just because they see that line in
>> angstrom.conf.
>
> Well, we could add a comment for that in the angstrom.conf file.
>
> Something like:
>
> # This enforces a specific brand to be used, in our case the angstrom
> # one. In case you want to change the logo or something like that we
> # suggest you to create another recipe (e.g: psplash-<brand>) for that
> # and change this line according.
> PREFERRED_VERSION_...

Maybe

# This enforces a specific brand to be used, in our case the angstrom
# one. In case you want to change the logo or something like that we
# suggest you to create another recipe (e.g: psplash-<brand>) for that
# and add that into your image.
# You can also opt to change or override this line, but that isn't
# needed because psplash is using update-alternatives.

regards,

Koen

[PATCH] angstrom-2008.1.conf: add PREFERRED_VERSION_psplash

[Andrea Adami]

 -second try
---
 conf/distro/angstrom-2008.1.conf |   11 ++++++++++-
 1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/conf/distro/angstrom-2008.1.conf b/conf/distro/angstrom-2008.1.conf
index 57048d6..6c2c1bd 100644
--- a/conf/distro/angstrom-2008.1.conf
+++ b/conf/distro/angstrom-2008.1.conf
@@ -76,6 +76,15 @@ PREFERRED_VERSION_glibc-initial 	?= "2.6.1"
 PREFERRED_VERSION_uclibc        	?= "0.9.30"
 PREFERRED_VERSION_uclibc-initial	?= "0.9.30"
 
+# Preferred version for psplash
+# This enforces a specific brand to be used, in our case the angstrom
+# one. In case you want to change the logo or something like that we
+# suggest you to create another recipe (e.g: psplash-<brand>) for that
+# and add that into your image.
+# You can also opt to change or override this line, but that isn't
+# needed because psplash is using update-alternatives.
+PREFERRED_VERSION_psplash               ?= "psplash-angstrom"
+
 PREFERRED_PROVIDER_avahi = "avahi"
 
 #Small machines prefer kdrive, but we might ship full Xorg in other images
@@ -144,7 +153,7 @@ PREFERRED_PROVIDER_virtual/xserver      ?= "xserver-kdrive"
 # Others:
 PREFERRED_PROVIDER_linux-libc-headers = "linux-libc-headers"
 PREFERRED_PROVIDER_dbus-glib             = "dbus-glib"
-PREFERRED_PROVIDER_esound 		?= "pulseaudio"
+PREFERRED_PROVIDER_esound               ?= "pulseaudio"
 PREFERRED_PROVIDER_gconf                ?= "gconf-dbus"
 PREFERRED_PROVIDER_gnome-vfs            ?= "gnome-vfs"
 PREFERRED_PROVIDER_gnome-vfs-plugin-file ?= "gnome-vfs"
-- 
1.6.0.6

Re: [PATCH] angstrom-2008.1.conf: add PREFERRED_VERSION_psplash

[Koen Kooi]

On 01-04-09 01:45, Andrea Adami wrote:
>
>   -second try
> ---
>   conf/distro/angstrom-2008.1.conf |   11 ++++++++++-
>   1 files changed, 10 insertions(+), 1 deletions(-)
>
> diff --git a/conf/distro/angstrom-2008.1.conf b/conf/distro/angstrom-2008.1.conf
> index 57048d6..6c2c1bd 100644
> --- a/conf/distro/angstrom-2008.1.conf
> +++ b/conf/distro/angstrom-2008.1.conf
> @@ -76,6 +76,15 @@ PREFERRED_VERSION_glibc-initial 	?= "2.6.1"
>   PREFERRED_VERSION_uclibc        	?= "0.9.30"
>   PREFERRED_VERSION_uclibc-initial	?= "0.9.30"
>
> +# Preferred version for psplash
> +# This enforces a specific brand to be used, in our case the angstrom
> +# one. In case you want to change the logo or something like that we
> +# suggest you to create another recipe (e.g: psplash-<brand>) for that
> +# and add that into your image.
> +# You can also opt to change or override this line, but that isn't
> +# needed because psplash is using update-alternatives.
> +PREFERRED_VERSION_psplash               ?= "psplash-angstrom"
> +
>   PREFERRED_PROVIDER_avahi = "avahi"
>
>   #Small machines prefer kdrive, but we might ship full Xorg in other images
> @@ -144,7 +153,7 @@ PREFERRED_PROVIDER_virtual/xserver      ?= "xserver-kdrive"
>   # Others:
>   PREFERRED_PROVIDER_linux-libc-headers = "linux-libc-headers"
>   PREFERRED_PROVIDER_dbus-glib             = "dbus-glib"
> -PREFERRED_PROVIDER_esound 		?= "pulseaudio"
> +PREFERRED_PROVIDER_esound               ?= "pulseaudio"
>   PREFERRED_PROVIDER_gconf                ?= "gconf-dbus"
>   PREFERRED_PROVIDER_gnome-vfs            ?= "gnome-vfs"
>   PREFERRED_PROVIDER_gnome-vfs-plugin-file ?= "gnome-vfs"

What does pulseaudio have to do with psplash? Looks OK otherwise

regards,

Koen

Re: [PATCH] angstrom-2008.1.conf: add PREFERRED_VERSION_psplash

[Andrea Adami]

> What does pulseaudio have to do with psplash? Looks OK otherwise

While I was there I cleaned some tabs...but I'm lazy and did not check
whole file...

Cheers

Andrea

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Michael 'Mickey' Lauer]

On Monday 30 March 2009 14:07:50 Koen Kooi wrote:
> But then again, people are still claiming angstrom is bloated and
> minimal isn't when minimal is currently angstrom.conf with sed
> s/angstrom/minimal/ applied to it...

Indeed, I think it's rather the style of presentation that makes people feel 
angstrom is bloated -- that's why I regrouped the settings and split them in 
individual .inc. Perhaps eventually we find a way of working together again so 
that we could share some of the .inc files.

:M:

Re: [PATCH 1/1] angstrom-2008.1.conf: add PREFERRED_PROVIDER_virtual/psplash -silence warning during parsing

[Philip Balister]

[-- Attachment #1: Type: text/plain, Size: 918 bytes --]

Michael 'Mickey' Lauer wrote:
> On Monday 30 March 2009 14:07:50 Koen Kooi wrote:
>> But then again, people are still claiming angstrom is bloated and
>> minimal isn't when minimal is currently angstrom.conf with sed
>> s/angstrom/minimal/ applied to it...
> 
> Indeed, I think it's rather the style of presentation that makes people feel 
> angstrom is bloated -- that's why I regrouped the settings and split them in 
> individual .inc. Perhaps eventually we find a way of working together again so 
> that we could share some of the .inc files.

I just did a quick scan of minimal.conf, it looks like more than the 
minimal lines to build simple images.

I still think we need a minimal.conf showing the bare minimum needed in 
a distro file to build simple images. I don't think it should become an 
alternative to Angstrom, Kaelios, slugos, oe one of the other distros 
present in OE.

Philip

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3303 bytes --]