[PATCH] cogl-1.0.inc: set RDEPENDS_${PN}-dev = ""

[PATCH] cogl-1.0.inc: set RDEPENDS_${PN}-dev = ""

[Adrian Ratiu]

${PN}-dev by default depends on ${PN} but in this case ${PN} is empty
by design (FILES_${PN} = "") and is not created, leading to
installation dependency errors like the following:

Collected errors:
 * Solver encountered 1 problem(s):
 * Problem 1/1:
 *   - nothing provides cogl-1.0 = 1.22.2-r0 needed by cogl-1.0-dev-1.22.2-r0.armv7at2hf-neon-imx
 *
 * Solution 1:
 *   - do not ask to install a package providing cogl-1.0-dev

Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
---
 meta/recipes-graphics/cogl/cogl-1.0.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-graphics/cogl/cogl-1.0.inc b/meta/recipes-graphics/cogl/cogl-1.0.inc
index 3e392fa5ec..a388023a03 100644
--- a/meta/recipes-graphics/cogl/cogl-1.0.inc
+++ b/meta/recipes-graphics/cogl/cogl-1.0.inc
@@ -75,4 +75,6 @@ RPROVIDES_libcogl = "cogl-1.0"
 RCONFLICTS_libcogl = "cogl-1.0"
 RREPLACES_libcogl = "cogl-1.0"
 
+RDEPENDS_${PN}-dev = ""
+
 COMPATIBLE_HOST_armv4 = 'null'
-- 
2.22.0

Re: [PATCH] cogl-1.0.inc: set RDEPENDS_${PN}-dev = ""

[Kang Kai]

On 2019/7/17 下午3:48, Adrian Ratiu wrote:
> ${PN}-dev by default depends on ${PN} but in this case ${PN} is empty
> by design (FILES_${PN} = "") and is not created, leading to
> installation dependency errors like the following:
>
> Collected errors:
>   * Solver encountered 1 problem(s):
>   * Problem 1/1:
>   *   - nothing provides cogl-1.0 = 1.22.2-r0 needed by cogl-1.0-dev-1.22.2-r0.armv7at2hf-neon-imx
>   *
>   * Solution 1:
>   *   - do not ask to install a package providing cogl-1.0-dev


I think what you need is

ALLOW_EMPTY_${PN} = "1"


Regards,
Kai


>
> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
> ---
>   meta/recipes-graphics/cogl/cogl-1.0.inc | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-graphics/cogl/cogl-1.0.inc b/meta/recipes-graphics/cogl/cogl-1.0.inc
> index 3e392fa5ec..a388023a03 100644
> --- a/meta/recipes-graphics/cogl/cogl-1.0.inc
> +++ b/meta/recipes-graphics/cogl/cogl-1.0.inc
> @@ -75,4 +75,6 @@ RPROVIDES_libcogl = "cogl-1.0"
>   RCONFLICTS_libcogl = "cogl-1.0"
>   RREPLACES_libcogl = "cogl-1.0"
>   
> +RDEPENDS_${PN}-dev = ""
> +
>   COMPATIBLE_HOST_armv4 = 'null'


-- 
Kai Kang

Re: [PATCH] cogl-1.0.inc: set RDEPENDS_${PN}-dev = ""

[Adrian Ratiu]

On Wed, 17 Jul 2019, Kang Kai <Kai.Kang@windriver.com> wrote:
Hi

> On 2019/7/17 下午3:48, Adrian Ratiu wrote: 
>> ${PN}-dev by default depends on ${PN} but in this case ${PN} is 
>> empty by design (FILES_${PN} = "") and is not created, leading 
>> to installation dependency errors like the following: 
>> 
>> Collected errors: 
>>   * Solver encountered 1 problem(s): * Problem 1/1: *   - 
>>   nothing provides cogl-1.0 = 1.22.2-r0 needed by 
>>   cogl-1.0-dev-1.22.2-r0.armv7at2hf-neon-imx * * Solution 1: * 
>>   - do not ask to install a package providing cogl-1.0-dev 
>  
> I think what you need is 
> 
> ALLOW_EMPTY_${PN} = "1"

Why is creating and depending on an empty package better than not 
creating and depending on it at all?

From what I can see in other recipes both methods are used.

>
>
> Regards,
> Kai
>
>
>>
>> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
>> ---
>>   meta/recipes-graphics/cogl/cogl-1.0.inc | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-graphics/cogl/cogl-1.0.inc b/meta/recipes-graphics/cogl/cogl-1.0.inc
>> index 3e392fa5ec..a388023a03 100644
>> --- a/meta/recipes-graphics/cogl/cogl-1.0.inc
>> +++ b/meta/recipes-graphics/cogl/cogl-1.0.inc
>> @@ -75,4 +75,6 @@ RPROVIDES_libcogl = "cogl-1.0"
>>   RCONFLICTS_libcogl = "cogl-1.0"
>>   RREPLACES_libcogl = "cogl-1.0"
>>   
>> +RDEPENDS_${PN}-dev = ""
>> +
>>   COMPATIBLE_HOST_armv4 = 'null'
>
>
> -- 
> Kai Kang

Re: [PATCH] cogl-1.0.inc: set RDEPENDS_${PN}-dev = ""

[Kang Kai]

On 2019/7/17 下午6:46, Adrian Ratiu wrote:
> On Wed, 17 Jul 2019, Kang Kai <Kai.Kang@windriver.com> wrote:
> Hi
>
>> On 2019/7/17 下午3:48, Adrian Ratiu wrote:
>>> ${PN}-dev by default depends on ${PN} but in this case ${PN} is 
>>> empty by design (FILES_${PN} = "") and is not created, leading to 
>>> installation dependency errors like the following:
>>> Collected errors:   * Solver encountered 1 problem(s): * Problem 
>>> 1/1: *   -   nothing provides cogl-1.0 = 1.22.2-r0 needed by   
>>> cogl-1.0-dev-1.22.2-r0.armv7at2hf-neon-imx * * Solution 1: *   - do 
>>> not ask to install a package providing cogl-1.0-dev 
>>
>> I think what you need is
>> ALLOW_EMPTY_${PN} = "1"
>
> Why is creating and depending on an empty package better than not 
> creating and depending on it at all?

I thought it is conventional way to solve such dependency issue. After I 
grepped recipes, there are similar solutions there.
So fine with your patch.

Kai


>
>> From what I can see in other recipes both methods are used.
>
>>
>>
>> Regards,
>> Kai
>>
>>
>>>
>>> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
>>> ---
>>>   meta/recipes-graphics/cogl/cogl-1.0.inc | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/meta/recipes-graphics/cogl/cogl-1.0.inc 
>>> b/meta/recipes-graphics/cogl/cogl-1.0.inc
>>> index 3e392fa5ec..a388023a03 100644
>>> --- a/meta/recipes-graphics/cogl/cogl-1.0.inc
>>> +++ b/meta/recipes-graphics/cogl/cogl-1.0.inc
>>> @@ -75,4 +75,6 @@ RPROVIDES_libcogl = "cogl-1.0"
>>>   RCONFLICTS_libcogl = "cogl-1.0"
>>>   RREPLACES_libcogl = "cogl-1.0"
>>>   +RDEPENDS_${PN}-dev = ""
>>> +
>>>   COMPATIBLE_HOST_armv4 = 'null'
>>
>>
>> -- 
>> Kai Kang
>

-- 
Kai Kang