[PATCH v5] linuxptp: Add recipe for linux ptp

[PATCH v5] linuxptp: Add recipe for linux ptp

[Sam Nelson]

- linuxptp is an implementation of precision time protocol (PTP)

Signed-off-by: Sam Nelson <sam.nelson@ti.com>

---
Change from previous patch
* Fix issues with passing extra Cflags
---
---
 .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   23 ++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100755 meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb

diff --git a/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
new file mode 100755
index 0000000..d2e39f0
--- /dev/null
+++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
@@ -0,0 +1,23 @@
+DESCRIPTION = "This software is an implementation of the Precision Time Protocol (PTP) \
+according to IEEE standard 1588 for Linux"
+HOMEPAGE = "http://linuxptp.sourceforge.net/"
+SECTION = "console/network"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+PR = "r0"
+
+SRC_URI = "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
+SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
+SRC_URI[sha256sum] = "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad3085"
+
+inherit autotools 
+
+EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH} \
+		EXTRA_CFLAGS="-D_GNU_SOURCE -DHAVE_CLOCK_ADJTIME \
+		-DHAVE_ONESTEP_SYNC""
+
+do_install() {
+	install -d ${D}${bindir}/
+	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
+}
+
-- 
1.7.9.5

Re: [PATCH v5] linuxptp: Add recipe for linux ptp

[Denys Dmytriyenko]

On Sun, Mar 02, 2014 at 05:53:06PM -0500, Sam Nelson wrote:
> - linuxptp is an implementation of precision time protocol (PTP)

Ah, didn't see you sent v5 after v4, sorry...

> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> 
> ---
> Change from previous patch
> * Fix issues with passing extra Cflags

BTW, you can keep previous patch revisions here as well, just accumulate them 
and say something like "Changes in v5". That's basically the Changelog of the 
patch itself, it won't go into git commit message... But it's not a strict 
requirement - it's up to you whether to keep full history of changes in the 
patch, or just the latest one.

> ---
> ---
>  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   23 ++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>  create mode 100755 meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> 
> diff --git a/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> new file mode 100755
> index 0000000..d2e39f0
> --- /dev/null
> +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> @@ -0,0 +1,23 @@
> +DESCRIPTION = "This software is an implementation of the Precision Time Protocol (PTP) \
> +according to IEEE standard 1588 for Linux"
> +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> +SECTION = "console/network"
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +PR = "r0"
> +
> +SRC_URI = "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
> +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> +SRC_URI[sha256sum] = "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad3085"
> +
> +inherit autotools 
> +
> +EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} ARCH=${TARGET_ARCH} \
> +		EXTRA_CFLAGS="-D_GNU_SOURCE -DHAVE_CLOCK_ADJTIME \
> +		-DHAVE_ONESTEP_SYNC""

Are you sure you don't need to escape quotes in the middle? Or alternatively, 
you can use single quotes in the middle, inside outer double quotes...


> +
> +do_install() {
> +	install -d ${D}${bindir}/
> +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> +}
> +
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH v5] linuxptp: Add recipe for linux ptp

[Nelson, Sam]

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, March 03, 2014 10:03 PM
> To: Nelson, Sam
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH v5] linuxptp: Add recipe for linux ptp
> 
> On Sun, Mar 02, 2014 at 05:53:06PM -0500, Sam Nelson wrote:
> > - linuxptp is an implementation of precision time protocol (PTP)
> 
> Ah, didn't see you sent v5 after v4, sorry...
> 
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> >
> > ---
> > Change from previous patch
> > * Fix issues with passing extra Cflags
> 
> BTW, you can keep previous patch revisions here as well, just accumulate
> them
> and say something like "Changes in v5". That's basically the Changelog of the
> patch itself, it won't go into git commit message... But it's not a strict
> requirement - it's up to you whether to keep full history of changes in the
> patch, or just the latest one.
[Sam] Makes sense. I will try  to keep the full history.
> 
> > ---
> > ---
> >  .../recipes-connectivity/linuxptp/linuxptp_1.3.bb  |   23
> ++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >  create mode 100755 meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb
> >
> > diff --git a/meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb b/meta-arago-extras/recipes-
> connectivity/linuxptp/linuxptp_1.3.bb
> > new file mode 100755
> > index 0000000..d2e39f0
> > --- /dev/null
> > +++ b/meta-arago-extras/recipes-connectivity/linuxptp/linuxptp_1.3.bb
> > @@ -0,0 +1,23 @@
> > +DESCRIPTION = "This software is an implementation of the Precision Time
> Protocol (PTP) \
> > +according to IEEE standard 1588 for Linux"
> > +HOMEPAGE = "http://linuxptp.sourceforge.net/"
> > +SECTION = "console/network"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM =
> "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > +PR = "r0"
> > +
> > +SRC_URI =
> "http://iweb.dl.sourceforge.net/project/linuxptp/v${PV}/linuxptp-${PV}.tgz"
> > +SRC_URI[md5sum] = "664706a86fe1413391d4536080548501"
> > +SRC_URI[sha256sum] =
> "b1651186689ef06e56add506bd9bb55a700ca56e3f92ba71d08f3ad12fad30
> 85"
> > +
> > +inherit autotools
> > +
> > +EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX}
> ARCH=${TARGET_ARCH} \
> > +		EXTRA_CFLAGS="-D_GNU_SOURCE -
> DHAVE_CLOCK_ADJTIME \
> > +		-DHAVE_ONESTEP_SYNC""
> 
> Are you sure you don't need to escape quotes in the middle? Or
> alternatively,
> you can use single quotes in the middle, inside outer double quotes...
> 
[Sam] Adding the escape quotes results in error. I checked without the escape quotes the flags are passed correctly.
> 
> > +
> > +do_install() {
> > +	install -d ${D}${bindir}/
> > +	install -c -m 755 ${S}/ptp4l ${D}${bindir}/
> > +}
> > +
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago