[oe,meta-filesystems] xorriso: Add xorriso v1.4.0

Message ID 1433170089-9030-1-git-send-email-naresh.bhat@linaro.org
State New
Headers show

Commit Message

naresh.bhat@linaro.org June 1, 2015, 2:48 p.m.
Add xorriso v1.4.0 based on the discussions
http://patchwork.openembedded.org/patch/91143/
http://lists.openembedded.org/pipermail/openembedded-devel/2015-April/100981.html
We have tested this package with
Montavista CGE product - http://www.mvista.com/ and
luvOS distribution - https://01.org/linux-uefi-validation build.
Inherit the pkgconfig to avoid the below error,
..
...
| disabled use of O_DIRECT with input of -as cdrskin
| luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
configure: line 14352: syntax error near unexpected token `LIBCDIO,'
| luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
configure: line 14352: `    PKG_CHECK_MODULES(LIBCDIO, libcdio >= $LIBCDIO_REQUIRED)'
...
....

Signed-off-by: Naresh Bhat <naresh.bhat@linaro.org>
Signed-off-by: Jagadeesh Krishnanjanappa <jkrishnanjanappa@mvista.com>
Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
---
 .../recipes-utils/xorriso/xorriso_1.4.0.bb         |   30 ++++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb

Comments

naresh.bhat@linaro.org June 2, 2015, 9:35 a.m. | #1
Hi Khem Raj,

On 1 June 2015 at 23:40, Khem Raj <raj.khem@gmail.com> wrote:
> On Mon, Jun 1, 2015 at 7:48 AM, Naresh Bhat <naresh.bhat@linaro.org> wrote:
>> Add xorriso v1.4.0 based on the discussions
>> http://patchwork.openembedded.org/patch/91143/
>> http://lists.openembedded.org/pipermail/openembedded-devel/2015-April/100981.html
>> We have tested this package with
>> Montavista CGE product - http://www.mvista.com/ and
>> luvOS distribution - https://01.org/linux-uefi-validation build.
>> Inherit the pkgconfig to avoid the below error,
>> ..
>> ...
>> | disabled use of O_DIRECT with input of -as cdrskin
>> | luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
>> configure: line 14352: syntax error near unexpected token `LIBCDIO,'
>> | luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
>> configure: line 14352: `    PKG_CHECK_MODULES(LIBCDIO, libcdio >= $LIBCDIO_REQUIRED)'
>> ...
>> ....
>>
>> Signed-off-by: Naresh Bhat <naresh.bhat@linaro.org>
>> Signed-off-by: Jagadeesh Krishnanjanappa <jkrishnanjanappa@mvista.com>
>> Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
>> ---
>>  .../recipes-utils/xorriso/xorriso_1.4.0.bb         |   30 ++++++++++++++++++++
>>  1 file changed, 30 insertions(+)
>>  create mode 100644 meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>>
>> diff --git a/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>> new file mode 100644
>> index 0000000..2f81f0d
>> --- /dev/null
>> +++ b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>> @@ -0,0 +1,30 @@
>> +DESCRIPTIONS = "xorriso copies file objects from POSIX compliant filesystems \
>> +into Rock Ridge enhanced ISO 9660 filesystems and allows session-wise \
>> +manipulation of such filesystems"
>> +
>> +LICENSE = "GPLv3"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>> +
>> +# with, without, depends, rdepends
>> +#
>> +PACKAGECONFIG ??= "acl attr zlib bzip2 readline"
>> +PACKAGECONFIG[acl] = "--enable-libacl,--disable-libacl,acl,"
>> +PACKAGECONFIG[attr] = "--enable-xattr,--disable-xattr,attr,"
>> +PACKAGECONFIG[zlib] = "--enable-zlib,--disable-zlib,zlib,"
>> +PACKAGECONFIG[bzip2] = "--enable-libbz2,--disable-libbz2,bzip2,"
>> +PACKAGECONFIG[readline] = "--enable-libreadline,--disable-libreadline,readline,"
>> +
>> +RDEPENDS_${PN} = "tk"
>> +
>> +inherit autotools-brokensep pkgconfig
>
> would be good if it did not need brokensep but probably its ok to have
> it done as a follow up.
I understand.  If you are ok with it.  We want to take it as a follow
up activity.
>
>> +
>> +SRC_URI = "http://www.gnu.org/software/${BPN}/${BPN}-${PV}.tar.gz"
>> +
>> +do_configure_prepend () {
>> +    touch NEWS
>> +}
>> +
>> +BBCLASSEXTEND = "native"
>> +
>> +SRC_URI[md5sum] = "ec79fd2736b8da76e7a870e27cadf6fa"
>> +SRC_URI[sha256sum] = "0bd1e085015b28c24f57697d6def2fe84517967dc417554c0c3ccf1685ed0e56"
>> --
>> 1.7.9.5
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
naresh.bhat@linaro.org June 3, 2015, 7:25 a.m. | #2
Hi Andre McCurdy,

Thank you very much for doing review.  I have considered all your
suggestions and you can find the re-worked patch
http://lists.openembedded.org/pipermail/openembedded-devel/2015-June/101744.html

On 3 June 2015 at 05:05, Andre McCurdy <armccurdy@gmail.com> wrote:
> On Mon, Jun 1, 2015 at 7:48 AM, Naresh Bhat <naresh.bhat@linaro.org> wrote:
>> Add xorriso v1.4.0 based on the discussions
>> http://patchwork.openembedded.org/patch/91143/
>> http://lists.openembedded.org/pipermail/openembedded-devel/2015-April/100981.html
>> We have tested this package with
>> Montavista CGE product - http://www.mvista.com/ and
>> luvOS distribution - https://01.org/linux-uefi-validation build.
>> Inherit the pkgconfig to avoid the below error,
>> ..
>> ...
>> | disabled use of O_DIRECT with input of -as cdrskin
>> | luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
>> configure: line 14352: syntax error near unexpected token `LIBCDIO,'
>> | luv-yocto/build/tmp/work/aarch64-oe-linux/xorriso/1.4.0-r0/xorriso-1.4.0/
>> configure: line 14352: `    PKG_CHECK_MODULES(LIBCDIO, libcdio >= $LIBCDIO_REQUIRED)'
>> ...
>> ....
>>
>> Signed-off-by: Naresh Bhat <naresh.bhat@linaro.org>
>> Signed-off-by: Jagadeesh Krishnanjanappa <jkrishnanjanappa@mvista.com>
>> Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
>> ---
>>  .../recipes-utils/xorriso/xorriso_1.4.0.bb         |   30 ++++++++++++++++++++
>>  1 file changed, 30 insertions(+)
>>  create mode 100644 meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>>
>> diff --git a/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>> new file mode 100644
>> index 0000000..2f81f0d
>> --- /dev/null
>> +++ b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
>> @@ -0,0 +1,30 @@
>> +DESCRIPTIONS = "xorriso copies file objects from POSIX compliant filesystems \
>> +into Rock Ridge enhanced ISO 9660 filesystems and allows session-wise \
>> +manipulation of such filesystems"
>
> That should be DESCRIPTION, not DESCRIPTIONS.
Done.
>
>> +LICENSE = "GPLv3"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>> +
>> +# with, without, depends, rdepends
>> +#
>
> Drop these comments.
Thanks, I have removed it.
>
>> +PACKAGECONFIG ??= "acl attr zlib bzip2 readline"
>> +PACKAGECONFIG[acl] = "--enable-libacl,--disable-libacl,acl,"
>> +PACKAGECONFIG[attr] = "--enable-xattr,--disable-xattr,attr,"
>> +PACKAGECONFIG[zlib] = "--enable-zlib,--disable-zlib,zlib,"
>> +PACKAGECONFIG[bzip2] = "--enable-libbz2,--disable-libbz2,bzip2,"
>> +PACKAGECONFIG[readline] = "--enable-libreadline,--disable-libreadline,readline,"
>> +
>> +RDEPENDS_${PN} = "tk"
>
> tk depends on the x11 distro feature, so perhaps xorriso should do too?
Yes, You are right.  I did the necessary changes.
>
>> +inherit autotools-brokensep pkgconfig
>> +
>> +SRC_URI = "http://www.gnu.org/software/${BPN}/${BPN}-${PV}.tar.gz"
>
> Move nearer to the top of the recipe, after LIC_FILES_CHKSUM.
Done.
>
> See http://www.openembedded.org/wiki/Styleguide
Thanks I have looked into it.
>
>> +do_configure_prepend () {
>> +    touch NEWS
>> +}
>> +
>> +BBCLASSEXTEND = "native"
>> +
>> +SRC_URI[md5sum] = "ec79fd2736b8da76e7a870e27cadf6fa"
>> +SRC_URI[sha256sum] = "0bd1e085015b28c24f57697d6def2fe84517967dc417554c0c3ccf1685ed0e56"
>
> Move nearer to the top of the recipe, just after SRC_URI.
Done.
>
>> --
>> 1.7.9.5
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Patch

diff --git a/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
new file mode 100644
index 0000000..2f81f0d
--- /dev/null
+++ b/meta-filesystems/recipes-utils/xorriso/xorriso_1.4.0.bb
@@ -0,0 +1,30 @@ 
+DESCRIPTIONS = "xorriso copies file objects from POSIX compliant filesystems \
+into Rock Ridge enhanced ISO 9660 filesystems and allows session-wise \
+manipulation of such filesystems"
+
+LICENSE = "GPLv3"
+LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
+
+# with, without, depends, rdepends
+#
+PACKAGECONFIG ??= "acl attr zlib bzip2 readline"
+PACKAGECONFIG[acl] = "--enable-libacl,--disable-libacl,acl,"
+PACKAGECONFIG[attr] = "--enable-xattr,--disable-xattr,attr,"
+PACKAGECONFIG[zlib] = "--enable-zlib,--disable-zlib,zlib,"
+PACKAGECONFIG[bzip2] = "--enable-libbz2,--disable-libbz2,bzip2,"
+PACKAGECONFIG[readline] = "--enable-libreadline,--disable-libreadline,readline,"
+
+RDEPENDS_${PN} = "tk"
+
+inherit autotools-brokensep pkgconfig
+
+SRC_URI = "http://www.gnu.org/software/${BPN}/${BPN}-${PV}.tar.gz"
+
+do_configure_prepend () {
+    touch NEWS
+}
+
+BBCLASSEXTEND = "native"
+
+SRC_URI[md5sum] = "ec79fd2736b8da76e7a870e27cadf6fa"
+SRC_URI[sha256sum] = "0bd1e085015b28c24f57697d6def2fe84517967dc417554c0c3ccf1685ed0e56"