public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added.
       [not found] <3826>
@ 2013-02-01  6:45 ` yanjun.zhu
  2013-02-01 19:21   ` Saul Wold
  2013-02-04  6:48 ` [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com> yanjun.zhu
  2013-02-04  6:54 ` [V2 PATCH] pulseaudio_2.1: Remove the dependence of fftw yanjun.zhu
  2 siblings, 1 reply; 7+ messages in thread
From: yanjun.zhu @ 2013-02-01  6:45 UTC (permalink / raw)
  To: openembedded-core

From: "yanjun.zhu" <yanjun.zhu@windriver.com>

Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
---
 meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
index a576368..5110cca 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
@@ -1,8 +1,8 @@
 require pulseaudio.inc
 
-PR = "r15"
+PR = "r16"
 
-DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
+DEPENDS += "libjson gdbm speex libxml-parser-perl-native fftw"
 
 inherit gettext perlnative
 
-- 
1.7.11




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added.
  2013-02-01  6:45 ` [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added yanjun.zhu
@ 2013-02-01 19:21   ` Saul Wold
  2013-02-04  2:52     ` yzhu1
  0 siblings, 1 reply; 7+ messages in thread
From: Saul Wold @ 2013-02-01 19:21 UTC (permalink / raw)
  To: yanjun.zhu; +Cc: openembedded-core

On 01/31/2013 10:45 PM, yanjun.zhu wrote:
> From: "yanjun.zhu" <yanjun.zhu@windriver.com>
>
> Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
> ---
>   meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
> index a576368..5110cca 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
> @@ -1,8 +1,8 @@
>   require pulseaudio.inc
>
> -PR = "r15"
> +PR = "r16"
>
> -DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
> +DEPENDS += "libjson gdbm speex libxml-parser-perl-native fftw"
>
This looks like a possibly host contamination issue, since oe-core 
itself does not provide fftw, you could instead add --without-fftw.

This will ensure that fftw is not picked up from the host.

Thanks
	Sau!

>   inherit gettext perlnative
>
>



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added.
  2013-02-01 19:21   ` Saul Wold
@ 2013-02-04  2:52     ` yzhu1
  2013-02-04  6:38       ` yzhu1
  0 siblings, 1 reply; 7+ messages in thread
From: yzhu1 @ 2013-02-04  2:52 UTC (permalink / raw)
  To: Saul Wold; +Cc: openembedded-core

On 02/02/2013 03:21 AM, Saul Wold wrote:
> On 01/31/2013 10:45 PM, yanjun.zhu wrote:
>> From: "yanjun.zhu" <yanjun.zhu@windriver.com>
>>
>> Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
>> ---
>>   meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb 
>> b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>> index a576368..5110cca 100644
>> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>> @@ -1,8 +1,8 @@
>>   require pulseaudio.inc
>>
>> -PR = "r15"
>> +PR = "r16"
>>
>> -DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
>> +DEPENDS += "libjson gdbm speex libxml-parser-perl-native fftw"
>>
> This looks like a possibly host contamination issue, since oe-core 
> itself does not provide fftw, you could instead add --without-fftw.
>
> This will ensure that fftw is not picked up from the host.
>
> Thanks
>     Sau!
>
OK. I will follow your advice and make a test.

Thanks a lot.
Zhu Yanjun
>>   inherit gettext perlnative
>>
>>
>
>




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added.
  2013-02-04  2:52     ` yzhu1
@ 2013-02-04  6:38       ` yzhu1
  0 siblings, 0 replies; 7+ messages in thread
From: yzhu1 @ 2013-02-04  6:38 UTC (permalink / raw)
  To: yzhu1; +Cc: openembedded-core

On 02/04/2013 10:52 AM, yzhu1 wrote:
> On 02/02/2013 03:21 AM, Saul Wold wrote:
>> On 01/31/2013 10:45 PM, yanjun.zhu wrote:
>>> From: "yanjun.zhu" <yanjun.zhu@windriver.com>
>>>
>>> Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
>>> ---
>>>   meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb | 4 ++--
>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb 
>>> b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>>> index a576368..5110cca 100644
>>> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>>> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.1.bb
>>> @@ -1,8 +1,8 @@
>>>   require pulseaudio.inc
>>>
>>> -PR = "r15"
>>> +PR = "r16"
>>>
>>> -DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
>>> +DEPENDS += "libjson gdbm speex libxml-parser-perl-native fftw"
>>>
>> This looks like a possibly host contamination issue, since oe-core 
>> itself does not provide fftw, you could instead add --without-fftw.
>>
>> This will ensure that fftw is not picked up from the host.
>>
>> Thanks
>>     Sau!
>>
> OK. I will follow your advice and make a test.
>
> Thanks a lot.
> Zhu Yanjun
Hi, Saul

Just now I made a test. Using --without-fftw, the file 
module-equalizer-sink.c which requires "fftw3.h" is not compiled.
So this can fix this defect.

Thanks a lot.
Zhu Yanjun
>>>   inherit gettext perlnative
>>>
>>>
>>
>>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
       [not found] <3826>
  2013-02-01  6:45 ` [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added yanjun.zhu
@ 2013-02-04  6:48 ` yanjun.zhu
  2013-02-04  6:51   ` yzhu1
  2013-02-04  6:54 ` [V2 PATCH] pulseaudio_2.1: Remove the dependence of fftw yanjun.zhu
  2 siblings, 1 reply; 7+ messages in thread
From: yanjun.zhu @ 2013-02-04  6:48 UTC (permalink / raw)
  To: openembedded-core

From: "yanjun.zhu" <yanjun.zhu@windriver.com>

---
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index 6633e38..2fed4f9 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -40,6 +40,7 @@ EXTRA_OECONF = "\
 		--disable-openssl \
 		--disable-xen \
 		--with-database=simple \
+		--without-fftw \
 "
 
 PARALLEL_MAKE = ""
-- 
1.7.11




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
  2013-02-04  6:48 ` [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com> yanjun.zhu
@ 2013-02-04  6:51   ` yzhu1
  0 siblings, 0 replies; 7+ messages in thread
From: yzhu1 @ 2013-02-04  6:51 UTC (permalink / raw)
  To: yanjun.zhu; +Cc: openembedded-core

Hi, Saul

Sorry. The title is wrong. Please ignore this email.
I will resend this patch.

Thanks a lot.
Zhu Yanjun

On 02/04/2013 02:48 PM, yanjun.zhu wrote:
> From: "yanjun.zhu" <yanjun.zhu@windriver.com>
>
> ---
>   meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> index 6633e38..2fed4f9 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> @@ -40,6 +40,7 @@ EXTRA_OECONF = "\
>   		--disable-openssl \
>   		--disable-xen \
>   		--with-database=simple \
> +		--without-fftw \
>   "
>   
>   PARALLEL_MAKE = ""




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [V2 PATCH] pulseaudio_2.1: Remove the dependence of fftw.
       [not found] <3826>
  2013-02-01  6:45 ` [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added yanjun.zhu
  2013-02-04  6:48 ` [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com> yanjun.zhu
@ 2013-02-04  6:54 ` yanjun.zhu
  2 siblings, 0 replies; 7+ messages in thread
From: yanjun.zhu @ 2013-02-04  6:54 UTC (permalink / raw)
  To: openembedded-core

From: "yanjun.zhu" <yanjun.zhu@windriver.com>

Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com>
---
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index 6633e38..2fed4f9 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -40,6 +40,7 @@ EXTRA_OECONF = "\
 		--disable-openssl \
 		--disable-xen \
 		--with-database=simple \
+		--without-fftw \
 "
 
 PARALLEL_MAKE = ""
-- 
1.7.11




^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-02-04  7:10 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <3826>
2013-02-01  6:45 ` [PATCH] pulseaudio_2.1: when do_compile, pulseaudio needs fftw3.h header file. In order to provide fftw3.h, the dependence of fftw is added yanjun.zhu
2013-02-01 19:21   ` Saul Wold
2013-02-04  2:52     ` yzhu1
2013-02-04  6:38       ` yzhu1
2013-02-04  6:48 ` [V2PATCH] pulseaudio_2.1: Remove the dependence of fftw. Signed-off-by: yanjun.zhu <yanjun.zhu@windriver.com> yanjun.zhu
2013-02-04  6:51   ` yzhu1
2013-02-04  6:54 ` [V2 PATCH] pulseaudio_2.1: Remove the dependence of fftw yanjun.zhu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox