[PATCH 1/2] zeromq: Add zeromq socket library.

[PATCH 1/2] zeromq: Add zeromq socket library.

[Siddharth Heroor]

Signed-off-by: Siddharth Heroor <heroor@ti.com>
---
 recipes/zeromq/zeromq.inc |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)
 create mode 100644 recipes/zeromq/zeromq.inc

diff --git a/recipes/zeromq/zeromq.inc b/recipes/zeromq/zeromq.inc
new file mode 100644
index 0000000..b631d98
--- /dev/null
+++ b/recipes/zeromq/zeromq.inc
@@ -0,0 +1,10 @@
+DESCRIPTION = "Socket Library that acts as a concurrency framework"
+HOMEPAGE = "http://www.zeromq.org"
+SECTION = "libs"
+LICENSE = "LGPLv3" 
+
+PR = "r1"
+
+SRC_URI = "http://download.zeromq.org/zeromq-${PV}.tar.gz"
+
+inherit autotools
-- 
1.7.0.4

[PATCH 2/2] zeromq: Add version 2.1.6

[Siddharth Heroor]

Signed-off-by: Siddharth Heroor <heroor@ti.com>
---
 recipes/zeromq/zeromq_2.1.6.bb |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
 create mode 100644 recipes/zeromq/zeromq_2.1.6.bb

diff --git a/recipes/zeromq/zeromq_2.1.6.bb b/recipes/zeromq/zeromq_2.1.6.bb
new file mode 100644
index 0000000..f88f512
--- /dev/null
+++ b/recipes/zeromq/zeromq_2.1.6.bb
@@ -0,0 +1,4 @@
+require zeromq.inc
+
+SRC_URI[md5sum] = "77b9ca20e8f2426874022080f41d7b0b"
+SRC_URI[sha256sum] = "a1bbb352a9dbf34b18a3999b22a8db2d51e4a52508d098528d771811533f9fd0"
-- 
1.7.0.4

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Paul Menzel]

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

Am Sonntag, den 01.05.2011, 15:20 +0530 schrieb Siddharth Heroor:
> Signed-off-by: Siddharth Heroor <heroor@ti.com>

The email differs again from the From line. If you want the TI address
be shown, please add a from line to the beginning of the message with
the patch. I guess `git send-email` can also be configured this way.

> ---
>  recipes/zeromq/zeromq.inc |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/zeromq/zeromq.inc
> 
> diff --git a/recipes/zeromq/zeromq.inc b/recipes/zeromq/zeromq.inc
> new file mode 100644
> index 0000000..b631d98
> --- /dev/null
> +++ b/recipes/zeromq/zeromq.inc
> @@ -0,0 +1,10 @@
> +DESCRIPTION = "Socket Library that acts as a concurrency framework"
> +HOMEPAGE = "http://www.zeromq.org"
> +SECTION = "libs"
> +LICENSE = "LGPLv3" 
> +
> +PR = "r1"

In an `*.inc` file `INC_PR` is used [1]. Please also start with `r0`.

> +
> +SRC_URI = "http://download.zeromq.org/zeromq-${PV}.tar.gz"
> +
> +inherit autotools


Thanks,

Paul


[1] http://openembedded.org/index.php/Styleguide#PR_variables_with_recipes_that_use_INC_files

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 2/2] zeromq: Add version 2.1.6

[Paul Menzel]

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

Am Sonntag, den 01.05.2011, 15:20 +0530 schrieb Siddharth Heroor:

What distribution and machine did you use for testing?

> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> ---
>  recipes/zeromq/zeromq_2.1.6.bb |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/zeromq/zeromq_2.1.6.bb
> 
> diff --git a/recipes/zeromq/zeromq_2.1.6.bb b/recipes/zeromq/zeromq_2.1.6.bb
> new file mode 100644
> index 0000000..f88f512
> --- /dev/null
> +++ b/recipes/zeromq/zeromq_2.1.6.bb
> @@ -0,0 +1,4 @@
> +require zeromq.inc

Please add `PR = ${INC_PR}.0`.

> +
> +SRC_URI[md5sum] = "77b9ca20e8f2426874022080f41d7b0b"
> +SRC_URI[sha256sum] = "a1bbb352a9dbf34b18a3999b22a8db2d51e4a52508d098528d771811533f9fd0"

You can also squash your two patches (`git rebase -i HEAD~2`) since the
include file by itself cannot not be used.


Thanks,

Paul

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Siddharth Heroor]

On Sun, May 1, 2011 at 3:38 PM, Paul Menzel
<paulepanter@users.sourceforge.net> wrote:
> Am Sonntag, den 01.05.2011, 15:20 +0530 schrieb Siddharth Heroor:
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>
> The email differs again from the From line. If you want the TI address
> be shown, please add a from line to the beginning of the message with
> the patch. I guess `git send-email` can also be configured this way.

I was using gmail to send out the email. I'll change the signoff line.
Its not strictly from TI.
Is it ok if I sign off my gmail account from non-TI stuff?

>
>> ---
>>  recipes/zeromq/zeromq.inc |   10 ++++++++++
>>  1 files changed, 10 insertions(+), 0 deletions(-)
>>  create mode 100644 recipes/zeromq/zeromq.inc
>>
>> diff --git a/recipes/zeromq/zeromq.inc b/recipes/zeromq/zeromq.inc
>> new file mode 100644
>> index 0000000..b631d98
>> --- /dev/null
>> +++ b/recipes/zeromq/zeromq.inc
>> @@ -0,0 +1,10 @@
>> +DESCRIPTION = "Socket Library that acts as a concurrency framework"
>> +HOMEPAGE = "http://www.zeromq.org"
>> +SECTION = "libs"
>> +LICENSE = "LGPLv3"
>> +
>> +PR = "r1"
>
> In an `*.inc` file `INC_PR` is used [1]. Please also start with `r0`.
>
>> +
>> +SRC_URI = "http://download.zeromq.org/zeromq-${PV}.tar.gz"
>> +
>> +inherit autotools
>
>
> Thanks,
>
> Paul
>
>
> [1] http://openembedded.org/index.php/Styleguide#PR_variables_with_recipes_that_use_INC_files
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>



-- 
Sid

Re: [PATCH 2/2] zeromq: Add version 2.1.6

[Siddharth Heroor]

On Sun, May 1, 2011 at 3:40 PM, Paul Menzel
<paulepanter@users.sourceforge.net> wrote:
> Am Sonntag, den 01.05.2011, 15:20 +0530 schrieb Siddharth Heroor:
>
> What distribution and machine did you use for testing?
I built for Angstrom using Koen's angstrom setup scripts. I built for
angstrom-2008.1 and MACHINE c6a816x-evm (a TI device). I some basic
tests
on the same board.

>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>> ---
>>  recipes/zeromq/zeromq_2.1.6.bb |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>  create mode 100644 recipes/zeromq/zeromq_2.1.6.bb
>>
>> diff --git a/recipes/zeromq/zeromq_2.1.6.bb b/recipes/zeromq/zeromq_2.1.6.bb
>> new file mode 100644
>> index 0000000..f88f512
>> --- /dev/null
>> +++ b/recipes/zeromq/zeromq_2.1.6.bb
>> @@ -0,0 +1,4 @@
>> +require zeromq.inc
>
> Please add `PR = ${INC_PR}.0`.
>
>> +
>> +SRC_URI[md5sum] = "77b9ca20e8f2426874022080f41d7b0b"
>> +SRC_URI[sha256sum] = "a1bbb352a9dbf34b18a3999b22a8db2d51e4a52508d098528d771811533f9fd0"
>
> You can also squash your two patches (`git rebase -i HEAD~2`) since the
> include file by itself cannot not be used.
>
>
> Thanks,
>
> Paul
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>



-- 
Sid

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Petr Štetiar]

Siddharth Heroor <heroor@gmail.com> [2011-05-01 15:20:48]:

Hi,

> +PR = "r1"
> +
> +SRC_URI = "http://download.zeromq.org/zeromq-${PV}.tar.gz"
> +
> +inherit autotools

for version 2.1.1 I've needed also the following:

	DEPENDS += "util-linux-ng"

	do_configure_prepend() {
        	./autogen.sh
	}

It's not needed anymore?

-- ynezz

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Siddharth Heroor]

On Sun, May 1, 2011 at 11:00 PM, Petr Štetiar <ynezz@true.cz> wrote:
> Siddharth Heroor <heroor@gmail.com> [2011-05-01 15:20:48]:
>
> Hi,
>
>> +PR = "r1"
>> +
>> +SRC_URI = "http://download.zeromq.org/zeromq-${PV}.tar.gz"
>> +
>> +inherit autotools
>
> for version 2.1.1 I've needed also the following:
>
>        DEPENDS += "util-linux-ng"
>
>        do_configure_prepend() {
>                ./autogen.sh
>        }
>
> It's not needed anymore?
Is the older recipe available somewhere? I didn't have to do this for
the latest version.

-- 
Sid

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Petr Štetiar]

Siddharth Heroor <heroor@gmail.com> [2011-05-03 22:18:03]:

> Is the older recipe available somewhere? I didn't have to do this for the
> latest version.

Yes, it's in patchwork[1]. I should've pushed it few weeks ago, but didn't had
time to rebase and test it again yet. Hopefully I'll do it during this
weekend.

AFAIK, libuuid from util-linux-ng is build/run dependency, so it's needed.

1. http://patches.openembedded.org/patch/1885

-- ynezz

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Siddharth Heroor]

On Fri, May 6, 2011 at 2:32 PM, Petr Štetiar <ynezz@true.cz> wrote:
> Siddharth Heroor <heroor@gmail.com> [2011-05-03 22:18:03]:
>
>> Is the older recipe available somewhere? I didn't have to do this for the
>> latest version.
>
> Yes, it's in patchwork[1]. I should've pushed it few weeks ago, but didn't had
> time to rebase and test it again yet. Hopefully I'll do it during this
> weekend.
Ok. I'll wait for your patch. I'm facing some issues with testing my
patch and so I'll
withdraw it.

--
Sid

>
> AFAIK, libuuid from util-linux-ng is build/run dependency, so it's needed.
>
> 1. http://patches.openembedded.org/patch/1885
>
> -- ynezz
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>



-- 
Sid

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Petr Štetiar]

Siddharth Heroor <heroor@gmail.com> [2011-05-06 15:39:23]:

> Ok. I'll wait for your patch. I'm facing some issues with testing my patch
> and so I'll withdraw it.

It's in[1], please test it and give some feedback. I'm using it with
lua-zeromq for some time already on arm9, so I would be suprised if there are
some issues. Thanks.

1. http://git.openembedded.org/cgit.cgi/openembedded/commit/?id=b8e90091aca8777092aa410fbfbb5c5d8cc7cbcb

-- ynezz

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Siddharth Heroor]

On Sun, May 8, 2011 at 11:39 PM, Petr Štetiar <ynezz@true.cz> wrote:
> Siddharth Heroor <heroor@gmail.com> [2011-05-06 15:39:23]:
>
>> Ok. I'll wait for your patch. I'm facing some issues with testing my patch
>> and so I'll withdraw it.
>
> It's in[1], please test it and give some feedback. I'm using it with
> lua-zeromq for some time already on arm9, so I would be suprised if there are
> some issues. Thanks.
>
> 1. http://git.openembedded.org/cgit.cgi/openembedded/commit/?id=b8e90091aca8777092aa410fbfbb5c5d8cc7cbcb

I tried to build it and it fails. The error is as follows -

ERROR: Logfile of failure stored in:
/swcoe/sdk/sid/arago/build-oe/angstrom-setup-scripts/build/tmp-angstrom_2010_x/work/armv7a-angstrom-linux-gnueabi/zeromq-2.1.6-r0/temp/log.do_configure.10701
Log data follows:
| + do_configure
| + ./autogen.sh
| autogen.sh: error: could not find libtool.  libtool is required to
run autogen.sh.
NOTE: package zeromq-2.1.6-r0: task do_configure: Failed

My build configuration is

Build Configuration:
BB_VERSION        = "1.12.0"
METADATA_BRANCH   = "master"
METADATA_REVISION = "980407c"
TARGET_ARCH       = "arm"
TARGET_OS         = "linux-gnueabi"
MACHINE           = "c6a816x-evm"
DISTRO            = "angstrom"
DISTRO_VERSION    = "v2011.05"
TARGET_FPU        = "hard"

By removing do_configure_prepend() from the recipe, I am able to build zeromq.
Why do we need to run autogen.sh to regenerate the configure script?
Won't the shipped configure work?

Also, shouldn't most the recipe be in an .inc file and the checksums
in a version.bb?
>
> -- ynezz
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>



-- 
Sid

Re: [PATCH 1/2] zeromq: Add zeromq socket library.

[Petr Štetiar]

Siddharth Heroor <heroor@gmail.com> [2011-05-10 15:04:04]:

> I tried to build it and it fails. The error is as follows -
> 
> ERROR: Logfile of failure stored in:
> /swcoe/sdk/sid/arago/build-oe/angstrom-setup-scripts/build/tmp-angstrom_2010_x/work/armv7a-angstrom-linux-gnueabi/zeromq-2.1.6-r0/temp/log.do_configure.10701
> Log data follows:
> | + do_configure
> | + ./autogen.sh
> | autogen.sh: error: could not find libtool.  libtool is required to
> run autogen.sh.
> NOTE: package zeromq-2.1.6-r0: task do_configure: Failed

Hm sorry, but this is strange. I don't know why it shoudln't find libtool...
Log here:

	+ cd /opt/devel/oe/ts72xx-test/tmp-uclibc-angstrom-2010.x/work/armv4t-angstrom-linux-uclibceabi/zeromq-2.1.6-r0/zeromq-2.1.6
	+ do_configure
	+ ./autogen.sh
	autoreconf: Entering directory `.'
	autoreconf: configure.in: not using Gettext
	autoreconf: running: aclocal -I config -I config --force -I config
	...

> By removing do_configure_prepend() from the recipe, I am able to build zeromq.
> Why do we need to run autogen.sh to regenerate the configure script?
> Won't the shipped configure work?

It apparently wasn't working quite correctly in some scenario, that's why it's
there. Maybe automake or something was complaining about the shipped version
of the macros? Sorry, I don't remember it correctly now, but it was tested by
Craig also, so I've pushed it as it is.

Anyone with the better knowledge of the autofoo stuff, who could shed some
light on this?

> Also, shouldn't most the recipe be in an .inc file and the checksums
> in a version.bb?

AFAIK, there's no such guideline. If you do plan to add for example package
which would track the git version of zeromq, then yes, it would be good idea
to split it and reuse the common parts.

-- ynezz

zeromq: task configure fails with `autogen.sh: error: could not find libtool.` (was: [PATCH 1/2] zeromq: Add zeromq socket library.)

[Paul Menzel]

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

Dear Petr, dear Siddharth,


Am Dienstag, den 10.05.2011, 22:35 +0200 schrieb Petr Štetiar:
> Siddharth Heroor <heroor@gmail.com> [2011-05-10 15:04:04]:
> 
> > I tried to build it and it fails. The error is as follows -
> > 
> > ERROR: Logfile of failure stored in:
> > /swcoe/sdk/sid/arago/build-oe/angstrom-setup-scripts/build/tmp-angstrom_2010_x/work/armv7a-angstrom-linux-gnueabi/zeromq-2.1.6-r0/temp/log.do_configure.10701
> > Log data follows:
> > | + do_configure
> > | + ./autogen.sh
> > | autogen.sh: error: could not find libtool.  libtool is required to run autogen.sh.
> > NOTE: package zeromq-2.1.6-r0: task do_configure: Failed

I do get the same error as Siddharth using `angstrom-2010.x` for
`MACHINE = "beagleboard"`.

> Hm sorry, but this is strange. I don't know why it shoudln't find libtool...
> Log here:
> 
> 	+ cd /opt/devel/oe/ts72xx-test/tmp-uclibc-angstrom-2010.x/work/armv4t-angstrom-linux-uclibceabi/zeromq-2.1.6-r0/zeromq-2.1.6
> 	+ do_configure
> 	+ ./autogen.sh
> 	autoreconf: Entering directory `.'
> 	autoreconf: configure.in: not using Gettext
> 	autoreconf: running: aclocal -I config -I config --force -I config
> 	...

Are you sure it is not using `libtool` installed on your build system?

> > By removing do_configure_prepend() from the recipe, I am able to build zeromq.
> > Why do we need to run autogen.sh to regenerate the configure script?
> > Won't the shipped configure work?
> 
> It apparently wasn't working quite correctly in some scenario, that's why it's
> there. Maybe automake or something was complaining about the shipped version
> of the macros? Sorry, I don't remember it correctly now, but it was tested by
> Craig also, so I've pushed it as it is.
> 
> Anyone with the better knowledge of the autofoo stuff, who could shed some
> light on this?

Maybe Ilya who submitted a patch to upgrade the recipe to 2.1.6 has an
idea?

[…]


Thanks,

Paul

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]