Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH V2][ 1/2] libav: Add v9.10
@ 2013-12-05  9:01 Andrei Gherzan
  2013-12-05  9:01 ` [PATCH V2][ 2/2] libav: Update to v0.8.9 Andrei Gherzan
  2013-12-05 19:53 ` [PATCH V2][ 1/2] libav: Add v9.10 Saul Wold
  0 siblings, 2 replies; 5+ messages in thread
From: Andrei Gherzan @ 2013-12-05  9:01 UTC (permalink / raw)
  To: openembedded-core

This version includes libavresample needed for packages like xbmc.
To use this version add:
PREFERRED_VERSION_libav = "9.10".
Removed git version as being an older version of a 9.X release.

Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
---
 .../libav/{libav_git.bb => libav_9.10.bb}               | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)
 rename meta/recipes-multimedia/libav/{libav_git.bb => libav_9.10.bb} (65%)

diff --git a/meta/recipes-multimedia/libav/libav_git.bb b/meta/recipes-multimedia/libav/libav_9.10.bb
similarity index 65%
rename from meta/recipes-multimedia/libav/libav_git.bb
rename to meta/recipes-multimedia/libav/libav_9.10.bb
index 951baa9..a76cc9c 100644
--- a/meta/recipes-multimedia/libav/libav_git.bb
+++ b/meta/recipes-multimedia/libav/libav_9.10.bb
@@ -1,16 +1,13 @@
 require libav.inc
 
-LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
-                    file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
-                    file://COPYING.LGPLv2.1;md5=bd7a443320af8c812e4c18d1b79df004 \
-                    file://COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
+SRC_URI = "http://libav.org/releases/libav-9.10.tar.xz"
 
-PV = "9.8+git${SRCPV}"
+SRC_URI[md5sum] = "4b49d1d8734815ca095bb0600d41c5cb"
+SRC_URI[sha256sum] = "83dbe640779da1c1e8d8836c99575f57f749d18407494ebca451578afee9300d"
 
 DEFAULT_PREFERENCE = "-1"
 
-SRCREV = "9aaca159bd220582c698f13d081a455f398c9975"
-SRC_URI = "git://git.libav.org/libav.git"
-
-S = "${WORKDIR}/git"
-
+LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
+                    file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
+                    file://COPYING.LGPLv2.1;md5=bd7a443320af8c812e4c18d1b79df004 \
+                    file://COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
-- 
1.8.1.4



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

* [PATCH V2][ 2/2] libav: Update to v0.8.9
  2013-12-05  9:01 [PATCH V2][ 1/2] libav: Add v9.10 Andrei Gherzan
@ 2013-12-05  9:01 ` Andrei Gherzan
  2013-12-05 19:25   ` Saul Wold
  2013-12-05 19:53 ` [PATCH V2][ 1/2] libav: Add v9.10 Saul Wold
  1 sibling, 1 reply; 5+ messages in thread
From: Andrei Gherzan @ 2013-12-05  9:01 UTC (permalink / raw)
  To: openembedded-core

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
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 \
 "
-
-
-- 
1.8.1.4



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

* Re: [PATCH V2][ 2/2] libav: Update to v0.8.9
  2013-12-05  9:01 ` [PATCH V2][ 2/2] libav: Update to v0.8.9 Andrei Gherzan
@ 2013-12-05 19:25   ` Saul Wold
  2013-12-07 18:01     ` Andrei Gherzan
  0 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2013-12-05 19:25 UTC (permalink / raw)
  To: Andrei Gherzan, openembedded-core


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 \
>   "
> -
> -
>


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

* Re: [PATCH V2][ 1/2] libav: Add v9.10
  2013-12-05  9:01 [PATCH V2][ 1/2] libav: Add v9.10 Andrei Gherzan
  2013-12-05  9:01 ` [PATCH V2][ 2/2] libav: Update to v0.8.9 Andrei Gherzan
@ 2013-12-05 19:53 ` Saul Wold
  1 sibling, 0 replies; 5+ messages in thread
From: Saul Wold @ 2013-12-05 19:53 UTC (permalink / raw)
  To: Andrei Gherzan, openembedded-core

On 12/05/2013 01:01 AM, Andrei Gherzan wrote:
> This version includes libavresample needed for packages like xbmc.
> To use this version add:
> PREFERRED_VERSION_libav = "9.10".
> Removed git version as being an older version of a 9.X release.
>
> Signed-off-by: Andrei Gherzan <andrei@gherzan.ro>
> ---
>   .../libav/{libav_git.bb => libav_9.10.bb}               | 17 +++++++----------
>   1 file changed, 7 insertions(+), 10 deletions(-)
>   rename meta/recipes-multimedia/libav/{libav_git.bb => libav_9.10.bb} (65%)
>
> diff --git a/meta/recipes-multimedia/libav/libav_git.bb b/meta/recipes-multimedia/libav/libav_9.10.bb
> similarity index 65%
> rename from meta/recipes-multimedia/libav/libav_git.bb
> rename to meta/recipes-multimedia/libav/libav_9.10.bb
> index 951baa9..a76cc9c 100644
> --- a/meta/recipes-multimedia/libav/libav_git.bb
> +++ b/meta/recipes-multimedia/libav/libav_9.10.bb
> @@ -1,16 +1,13 @@
>   require libav.inc
>
> -LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> -                    file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
> -                    file://COPYING.LGPLv2.1;md5=bd7a443320af8c812e4c18d1b79df004 \
> -                    file://COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
> +SRC_URI = "http://libav.org/releases/libav-9.10.tar.xz"
>
why not use ${BP} here instead, so we do not need to edit it for version 
changes?


> -PV = "9.8+git${SRCPV}"
> +SRC_URI[md5sum] = "4b49d1d8734815ca095bb0600d41c5cb"
> +SRC_URI[sha256sum] = "83dbe640779da1c1e8d8836c99575f57f749d18407494ebca451578afee9300d"
>
>   DEFAULT_PREFERENCE = "-1"
>
> -SRCREV = "9aaca159bd220582c698f13d081a455f398c9975"
> -SRC_URI = "git://git.libav.org/libav.git"
> -
> -S = "${WORKDIR}/git"
> -
> +LIC_FILES_CHKSUM = "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> +                    file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
> +                    file://COPYING.LGPLv2.1;md5=bd7a443320af8c812e4c18d1b79df004 \
> +                    file://COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02"
>
LIC_FILES_CHKSUM should be at the beginning before the SRC_URI

Thanks
	Sau!



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

* Re: [PATCH V2][ 2/2] libav: Update to v0.8.9
  2013-12-05 19:25   ` Saul Wold
@ 2013-12-07 18:01     ` Andrei Gherzan
  0 siblings, 0 replies; 5+ messages in thread
From: Andrei Gherzan @ 2013-12-07 18:01 UTC (permalink / raw)
  To: Saul Wold; +Cc: openembedded

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

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.

-- 
*ag*

[-- Attachment #2: Type: text/html, Size: 3693 bytes --]

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

end of thread, other threads:[~2013-12-07 18:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-05  9:01 [PATCH V2][ 1/2] libav: Add v9.10 Andrei Gherzan
2013-12-05  9:01 ` [PATCH V2][ 2/2] libav: Update to v0.8.9 Andrei Gherzan
2013-12-05 19:25   ` Saul Wold
2013-12-07 18:01     ` Andrei Gherzan
2013-12-05 19:53 ` [PATCH V2][ 1/2] libav: Add v9.10 Saul Wold

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