[PATCH] db: Remove unneeded PR

[PATCH] db: Remove unneeded PR

[Aníbal Limón]

The PR is not need so remove it after upgrade to 6.0.35.

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta/recipes-support/db/db_6.0.35.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-support/db/db_6.0.35.bb b/meta/recipes-support/db/db_6.0.35.bb
index 32afbe8..ecb1bee 100644
--- a/meta/recipes-support/db/db_6.0.35.bb
+++ b/meta/recipes-support/db/db_6.0.35.bb
@@ -15,8 +15,6 @@ LICENSE = "AGPL-3.0"
 VIRTUAL_NAME ?= "virtual/db"
 RCONFLICTS_${PN} = "db3"
 
-PR = "r1"
-
 # Note, when upgraded to 6.1.x, a patch in RPM will need to be removed to activate db 6.1 support.
 SRC_URI = "http://distfiles.gentoo.org/distfiles/db-${PV}.tar.gz \
            file://arm-thumb-mutex_db5.patch \
-- 
2.1.4

Re: [PATCH] db: Remove unneeded PR

[Paul Eggleton]

On Mon, 03 Oct 2016 14:17:27 Aníbal Limón wrote:
> The PR is not need so remove it after upgrade to 6.0.35.
> 
> Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
> ---
>  meta/recipes-support/db/db_6.0.35.bb | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/meta/recipes-support/db/db_6.0.35.bb
> b/meta/recipes-support/db/db_6.0.35.bb index 32afbe8..ecb1bee 100644
> --- a/meta/recipes-support/db/db_6.0.35.bb
> +++ b/meta/recipes-support/db/db_6.0.35.bb
> @@ -15,8 +15,6 @@ LICENSE = "AGPL-3.0"
>  VIRTUAL_NAME ?= "virtual/db"
>  RCONFLICTS_${PN} = "db3"
> 
> -PR = "r1"
> -
>  # Note, when upgraded to 6.1.x, a patch in RPM will need to be removed to
> activate db 6.1 support. SRC_URI =
> "http://distfiles.gentoo.org/distfiles/db-${PV}.tar.gz \
> file://arm-thumb-mutex_db5.patch \

I think it's too late to do this unfortunately - it needs to be done at the 
same time as the upgrade.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH] db: Remove unneeded PR

[Aníbal Limón]

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



On 10/03/2016 02:25 PM, Paul Eggleton wrote:
> On Mon, 03 Oct 2016 14:17:27 Aníbal Limón wrote:
>> The PR is not need so remove it after upgrade to 6.0.35.
>>
>> Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
>> ---
>>  meta/recipes-support/db/db_6.0.35.bb | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/meta/recipes-support/db/db_6.0.35.bb
>> b/meta/recipes-support/db/db_6.0.35.bb index 32afbe8..ecb1bee 100644
>> --- a/meta/recipes-support/db/db_6.0.35.bb
>> +++ b/meta/recipes-support/db/db_6.0.35.bb
>> @@ -15,8 +15,6 @@ LICENSE = "AGPL-3.0"
>>  VIRTUAL_NAME ?= "virtual/db"
>>  RCONFLICTS_${PN} = "db3"
>>
>> -PR = "r1"
>> -
>>  # Note, when upgraded to 6.1.x, a patch in RPM will need to be removed to
>> activate db 6.1 support. SRC_URI =
>> "http://distfiles.gentoo.org/distfiles/db-${PV}.tar.gz \
>> file://arm-thumb-mutex_db5.patch \
> 
> I think it's too late to do this unfortunately - it needs to be done at the 
> same time as the upgrade.

I sent a PATCHv3 [1] removing the PR but unfortunately was not integrated.

[1]
http://lists.openembedded.org/pipermail/openembedded-core/2016-September/127027.html

Cheers,
	alimon

> 
> Cheers,
> Paul
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH] db: Remove unneeded PR

[Paul Eggleton]

On Mon, 03 Oct 2016 14:30:15 Aníbal Limón wrote:
> On 10/03/2016 02:25 PM, Paul Eggleton wrote:
> > On Mon, 03 Oct 2016 14:17:27 Aníbal Limón wrote:
> >> The PR is not need so remove it after upgrade to 6.0.35.
> >> 
> >> Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
> >> ---
> >> 
> >>  meta/recipes-support/db/db_6.0.35.bb | 2 --
> >>  1 file changed, 2 deletions(-)
> >> 
> >> diff --git a/meta/recipes-support/db/db_6.0.35.bb
> >> b/meta/recipes-support/db/db_6.0.35.bb index 32afbe8..ecb1bee 100644
> >> --- a/meta/recipes-support/db/db_6.0.35.bb
> >> +++ b/meta/recipes-support/db/db_6.0.35.bb
> >> @@ -15,8 +15,6 @@ LICENSE = "AGPL-3.0"
> >> 
> >>  VIRTUAL_NAME ?= "virtual/db"
> >>  RCONFLICTS_${PN} = "db3"
> >> 
> >> -PR = "r1"
> >> -
> >> 
> >>  # Note, when upgraded to 6.1.x, a patch in RPM will need to be removed
> >>  to
> >> 
> >> activate db 6.1 support. SRC_URI =
> >> "http://distfiles.gentoo.org/distfiles/db-${PV}.tar.gz \
> >> file://arm-thumb-mutex_db5.patch \
> > 
> > I think it's too late to do this unfortunately - it needs to be done at
> > the
> > same time as the upgrade.
> 
> I sent a PATCHv3 [1] removing the PR but unfortunately was not integrated.
> 
> [1]
> http://lists.openembedded.org/pipermail/openembedded-core/2016-September/127
> 027.html

Understood - for most other changes we'd want to have a follow-up patch to 
apply what's missing, but for PR it's still too late - the r1 value is already 
merged and packages could have been built from it. Decreasing PR now will make 
the version number go backwards which we want to avoid.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre