Patchwork [V2,2/2] libav: Update to v0.8.9

login
register
mail settings
Submitter Andrei Gherzan
Date Dec. 5, 2013, 9:01 a.m.
Message ID <1386234101-21411-2-git-send-email-andrei@gherzan.ro>
Download mbox | patch
Permalink /patch/62893/
State New
Headers show

Comments

Andrei Gherzan - Dec. 5, 2013, 9:01 a.m.
Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
---
 .../0001-configure-enable-pic-for-AArch64.patch                   | 0
 meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb}  | 8 +++-----
 2 files changed, 3 insertions(+), 5 deletions(-)
 rename meta/recipes-multimedia/libav/{libav-0.8.8 => libav-0.8.9}/0001-configure-enable-pic-for-AArch64.patch (100%)
 rename meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb} (69%)
Saul Wold - Dec. 5, 2013, 7:25 p.m.
Andrei:

I see RP already merged this, but I have a couple of comments


On 12/05/2013 01:01 AM, Andrei Gherzan wrote:
> Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
> ---
>   .../0001-configure-enable-pic-for-AArch64.patch                   | 0
>   meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb}  | 8 +++-----
>   2 files changed, 3 insertions(+), 5 deletions(-)
>   rename meta/recipes-multimedia/libav/{libav-0.8.8 => libav-0.8.9}/0001-configure-enable-pic-for-AArch64.patch (100%)
>   rename meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb} (69%)
>
> diff --git a/meta/recipes-multimedia/libav/libav-0.8.8/0001-configure-enable-pic-for-AArch64.patch b/meta/recipes-multimedia/libav/libav-0.8.9/0001-configure-enable-pic-for-AArch64.patch
> similarity index 100%
> rename from meta/recipes-multimedia/libav/libav-0.8.8/0001-configure-enable-pic-for-AArch64.patch
> rename to meta/recipes-multimedia/libav/libav-0.8.9/0001-configure-enable-pic-for-AArch64.patch

We are generally trying to get rid of versioned patch directory in favor 
of just a general recipe name (libav in this case).

> diff --git a/meta/recipes-multimedia/libav/libav_0.8.8.bb b/meta/recipes-multimedia/libav/libav_0.8.9.bb
> similarity index 69%
> rename from meta/recipes-multimedia/libav/libav_0.8.8.bb
> rename to meta/recipes-multimedia/libav/libav_0.8.9.bb
> index 9cb4cf2..065a6d2 100644
> --- a/meta/recipes-multimedia/libav/libav_0.8.8.bb
> +++ b/meta/recipes-multimedia/libav/libav_0.8.9.bb
> @@ -1,10 +1,10 @@
>   require libav.inc
>
> -SRC_URI = "http://libav.org/releases/libav-0.8.8.tar.xz \
> +SRC_URI = "http://libav.org/releases/libav-0.8.9.tar.xz \
why not convert this to ${BP} so it does not need to be edited in the 
future. BP = BPN-PV.

Sau!

>              file://0001-configure-enable-pic-for-AArch64.patch"
>
> -SRC_URI[md5sum] = "34b8f1279a04466386ed67731197efe3"
> -SRC_URI[sha256sum] = "e95cf618eb6239177a62c46f15e840c37e02e8308baf94912fc5910ff4aacbf2"
> +SRC_URI[md5sum] = "7c0e5743cd5c4d4b1bcafd02a2e82e09"
> +SRC_URI[sha256sum] = "4f5181164e4007d5b0013bb3cc5e3b7393ab71ac286319c714b7697e2784f77f"
>
>   LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>                       file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
> @@ -14,5 +14,3 @@ LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>   EXTRA_OECONF += " \
>       --enable-postproc \
>   "
> -
> -
>
Andrei Gherzan - Dec. 7, 2013, 6:01 p.m.
On Thu, Dec 5, 2013 at 9:25 PM, Saul Wold <sgw@linux.intel.com> wrote:

>
> Andrei:
>
> I see RP already merged this, but I have a couple of comments
>
>
>
> On 12/05/2013 01:01 AM, Andrei Gherzan wrote:
>
>> Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
>> ---
>>   .../0001-configure-enable-pic-for-AArch64.patch                   | 0
>>   meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb}  | 8
>> +++-----
>>   2 files changed, 3 insertions(+), 5 deletions(-)
>>   rename meta/recipes-multimedia/libav/{libav-0.8.8 =>
>> libav-0.8.9}/0001-configure-enable-pic-for-AArch64.patch (100%)
>>   rename meta/recipes-multimedia/libav/{libav_0.8.8.bb => libav_0.8.9.bb}
>> (69%)
>>
>> diff --git a/meta/recipes-multimedia/libav/libav-0.8.8/0001-
>> configure-enable-pic-for-AArch64.patch b/meta/recipes-multimedia/
>> libav/libav-0.8.9/0001-configure-enable-pic-for-AArch64.patch
>> similarity index 100%
>> rename from meta/recipes-multimedia/libav/libav-0.8.8/0001-configure-
>> enable-pic-for-AArch64.patch
>> rename to meta/recipes-multimedia/libav/libav-0.8.9/0001-configure-
>> enable-pic-for-AArch64.patch
>>
>
> We are generally trying to get rid of versioned patch directory in favor
> of just a general recipe name (libav in this case).
>
>
>  diff --git a/meta/recipes-multimedia/libav/libav_0.8.8.bbb/meta/recipes-multimedia/
>> libav/libav_0.8.9.bb
>> similarity index 69%
>> rename from meta/recipes-multimedia/libav/libav_0.8.8.bb
>> rename to meta/recipes-multimedia/libav/libav_0.8.9.bb
>> index 9cb4cf2..065a6d2 100644
>> --- a/meta/recipes-multimedia/libav/libav_0.8.8.bb
>> +++ b/meta/recipes-multimedia/libav/libav_0.8.9.bb
>> @@ -1,10 +1,10 @@
>>   require libav.inc
>>
>> -SRC_URI = "http://libav.org/releases/libav-0.8.8.tar.xz \
>> +SRC_URI = "http://libav.org/releases/libav-0.8.9.tar.xz \
>>
> why not convert this to ${BP} so it does not need to be edited in the
> future. BP = BPN-PV.
>
> Sau!


Pushing a patch to address these comments. Thanks.

Patch

diff --git a/meta/recipes-multimedia/libav/libav-0.8.8/0001-configure-enable-pic-for-AArch64.patch b/meta/recipes-multimedia/libav/libav-0.8.9/0001-configure-enable-pic-for-AArch64.patch
similarity index 100%
rename from meta/recipes-multimedia/libav/libav-0.8.8/0001-configure-enable-pic-for-AArch64.patch
rename to meta/recipes-multimedia/libav/libav-0.8.9/0001-configure-enable-pic-for-AArch64.patch
diff --git a/meta/recipes-multimedia/libav/libav_0.8.8.bb b/meta/recipes-multimedia/libav/libav_0.8.9.bb
similarity index 69%
rename from meta/recipes-multimedia/libav/libav_0.8.8.bb
rename to meta/recipes-multimedia/libav/libav_0.8.9.bb
index 9cb4cf2..065a6d2 100644
--- a/meta/recipes-multimedia/libav/libav_0.8.8.bb
+++ b/meta/recipes-multimedia/libav/libav_0.8.9.bb
@@ -1,10 +1,10 @@ 
 require libav.inc
 
-SRC_URI = "http://libav.org/releases/libav-0.8.8.tar.xz \
+SRC_URI = "http://libav.org/releases/libav-0.8.9.tar.xz \
            file://0001-configure-enable-pic-for-AArch64.patch"
 
-SRC_URI[md5sum] = "34b8f1279a04466386ed67731197efe3"
-SRC_URI[sha256sum] = "e95cf618eb6239177a62c46f15e840c37e02e8308baf94912fc5910ff4aacbf2"
+SRC_URI[md5sum] = "7c0e5743cd5c4d4b1bcafd02a2e82e09"
+SRC_URI[sha256sum] = "4f5181164e4007d5b0013bb3cc5e3b7393ab71ac286319c714b7697e2784f77f"
 
 LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
                     file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
@@ -14,5 +14,3 @@  LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
 EXTRA_OECONF += " \
     --enable-postproc \
 "
-
-