Re: [oe] [meta-oe][PATCH] inotify-tools: add recipe

2016-10-17 Thread Andrea Galbusera
Hi Khem,

On Mon, Oct 17, 2016 at 11:46 PM, Khem Raj  wrote:

> On Oct 17, 2016 2:30 PM, "Andrea Galbusera"  wrote:
> >
> > inotify-tools is a C library and a set of command-line programs for Linux
> > providing a simple interface to inotify.
> >
> > Signed-off-by: Andrea Galbusera 
> > ---
> >  .../inotifywait-fix-compile-error-with-GCC-6.patch | 45
> ++
> >  .../inotify-tools/inotify-tools_git.bb | 23 +++
> >  2 files changed, 68 insertions(+)
> >  create mode 100644
> meta-oe/recipes-support/inotify-tools/inotify-tools/
> inotifywait-fix-compile-error-with-GCC-6.patch
> >  create mode 100644 meta-oe/recipes-support/inotify-tools/
> inotify-tools_git.bb
> >
> > diff --git
> a/meta-oe/recipes-support/inotify-tools/inotify-tools/
> inotifywait-fix-compile-error-with-GCC-6.patch
> b/meta-oe/recipes-support/inotify-tools/inotify-tools/
> inotifywait-fix-compile-error-with-GCC-6.patch
> > new file mode 100644
> > index 000..ba042a2
> > --- /dev/null
> > +++
> b/meta-oe/recipes-support/inotify-tools/inotify-tools/
> inotifywait-fix-compile-error-with-GCC-6.patch
> > @@ -0,0 +1,45 @@
> > +From 7affb288d6c0726e7b1ebc317a878927b6ef0d02 Mon Sep 17 00:00:00 2001
> > +From: Andrea Galbusera 
> > +Date: Tue, 13 Sep 2016 08:10:29 +0200
> > +Subject: [PATCH] inotifywait: fix compile error with GCC 6
> > +
> > +Fails to compile with misleading-indentation error
> > +
> > +| src/inotifywait.c: In function 'output_event_csv':
> > +| src/inotifywait.c:126:5: error: this 'if' clause does not guard...
> [-Werror=misleading-indentation]
> > +|  if (filename != NULL)
> > +|  ^~
> > +| src/inotifywait.c:129:2: note: ...this statement, but the latter is
> misleadingly indented as if it is guarded by the 'if'
> > +|   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask ) )
> );
> > +|   ^~
> > +| cc1: all warnings being treated as errors
> > +
> > +Fix indentation to resolve.
> > +
> > +Upstream-Status: Submitted [
> https://github.com/rvoicilas/inotify-tools/pull/66]
> > +
> > +Signed-off-by: Andrea Galbusera 
> > +---
> > + src/inotifywait.c | 6 +++---
> > + 1 file changed, 3 insertions(+), 3 deletions(-)
> > +
> > +diff --git a/src/inotifywait.c b/src/inotifywait.c
> > +index c5ce5e3..404a85b 100644
> > +--- a/src/inotifywait.c
> >  b/src/inotifywait.c
> > +@@ -122,9 +122,9 @@ void validate_format( char * fmt ) {
> > +
> > +
> > + void output_event_csv( struct inotify_event * event ) {
> > +-char *filename =
> csv_escape(inotifytools_filename_from_wd(event->wd));
> > +-if (filename != NULL)
> > +-printf("%s,", filename);
> > ++  char *filename =
> csv_escape(inotifytools_filename_from_wd(event->wd));
> > ++  if (filename != NULL)
> > ++  printf("%s,", filename);
> > +
> > +   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask
> ) ) );
> > +   if ( event->len > 0 )
> > +--
> > +1.9.1
> > +
> > diff --git a/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
> b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
> > new file mode 100644
> > index 000..a8ef557
> > --- /dev/null
> > +++ b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
> > @@ -0,0 +1,23 @@
> > +SUMMARY = "Command line tools and C library providing a simple interface
> to inotify"
> > +AUTHOR = "Rohan McGovern "
> > +HOMEPAGE = "http://wiki.github.com/rvoicilas/inotify-tools";
> > +SECTION = "console/devel"
> > +LICENSE = "GPL-2.0"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=ac6c26e52aea428ee7f56dc2c56424
> c6"
> > +
> > +SRCREV = "1df9af4d6cd0f4af4b1b19254bcf056aed4ae395"
> > +PV = "3.14+git${SRCPV}"
> > +
> > +SRC_URI = "git://github.com/rvoicilas/${BPN} \
> > +   file://inotifywait-fix-compile-error-with-GCC-6.patch \
> > +  "
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +inherit autotools
> > +
> > +EXTRA_OECONF = "--disable-doxygen"
> > +
> > +PACKAGES =+ "libinotifytools"
> >
> Why is this needed ?


The upstream project provides a thin C library and a couple of tools,
inotify{watch,wait}, built on to top of it. This was intended as an
improvement to the default packaging, to allow using and installing the
library for your own developments. The approach is inspired by i.e.
curl/libcurl recipe.


>




> > +FILES_libinotifytools = "${libdir}/lib*.so.*"
> > --
> > 1.9.1
> >
> > --
> > ___
> > 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
>
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] inotify-tools: add recipe

2016-10-17 Thread Khem Raj
On Oct 17, 2016 2:30 PM, "Andrea Galbusera"  wrote:
>
> inotify-tools is a C library and a set of command-line programs for Linux
> providing a simple interface to inotify.
>
> Signed-off-by: Andrea Galbusera 
> ---
>  .../inotifywait-fix-compile-error-with-GCC-6.patch | 45
++
>  .../inotify-tools/inotify-tools_git.bb | 23 +++
>  2 files changed, 68 insertions(+)
>  create mode 100644
meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
>  create mode 100644 meta-oe/recipes-support/inotify-tools/
inotify-tools_git.bb
>
> diff --git
a/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
b/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
> new file mode 100644
> index 000..ba042a2
> --- /dev/null
> +++
b/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
> @@ -0,0 +1,45 @@
> +From 7affb288d6c0726e7b1ebc317a878927b6ef0d02 Mon Sep 17 00:00:00 2001
> +From: Andrea Galbusera 
> +Date: Tue, 13 Sep 2016 08:10:29 +0200
> +Subject: [PATCH] inotifywait: fix compile error with GCC 6
> +
> +Fails to compile with misleading-indentation error
> +
> +| src/inotifywait.c: In function 'output_event_csv':
> +| src/inotifywait.c:126:5: error: this 'if' clause does not guard...
[-Werror=misleading-indentation]
> +|  if (filename != NULL)
> +|  ^~
> +| src/inotifywait.c:129:2: note: ...this statement, but the latter is
misleadingly indented as if it is guarded by the 'if'
> +|   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask ) )
);
> +|   ^~
> +| cc1: all warnings being treated as errors
> +
> +Fix indentation to resolve.
> +
> +Upstream-Status: Submitted [
https://github.com/rvoicilas/inotify-tools/pull/66]
> +
> +Signed-off-by: Andrea Galbusera 
> +---
> + src/inotifywait.c | 6 +++---
> + 1 file changed, 3 insertions(+), 3 deletions(-)
> +
> +diff --git a/src/inotifywait.c b/src/inotifywait.c
> +index c5ce5e3..404a85b 100644
> +--- a/src/inotifywait.c
>  b/src/inotifywait.c
> +@@ -122,9 +122,9 @@ void validate_format( char * fmt ) {
> +
> +
> + void output_event_csv( struct inotify_event * event ) {
> +-char *filename =
csv_escape(inotifytools_filename_from_wd(event->wd));
> +-if (filename != NULL)
> +-printf("%s,", filename);
> ++  char *filename =
csv_escape(inotifytools_filename_from_wd(event->wd));
> ++  if (filename != NULL)
> ++  printf("%s,", filename);
> +
> +   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask
) ) );
> +   if ( event->len > 0 )
> +--
> +1.9.1
> +
> diff --git a/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
> new file mode 100644
> index 000..a8ef557
> --- /dev/null
> +++ b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
> @@ -0,0 +1,23 @@
> +SUMMARY = "Command line tools and C library providing a simple interface
to inotify"
> +AUTHOR = "Rohan McGovern "
> +HOMEPAGE = "http://wiki.github.com/rvoicilas/inotify-tools";
> +SECTION = "console/devel"
> +LICENSE = "GPL-2.0"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=ac6c26e52aea428ee7f56dc2c56424c6"
> +
> +SRCREV = "1df9af4d6cd0f4af4b1b19254bcf056aed4ae395"
> +PV = "3.14+git${SRCPV}"
> +
> +SRC_URI = "git://github.com/rvoicilas/${BPN} \
> +   file://inotifywait-fix-compile-error-with-GCC-6.patch \
> +  "
> +
> +S = "${WORKDIR}/git"
> +
> +inherit autotools
> +
> +EXTRA_OECONF = "--disable-doxygen"
> +
> +PACKAGES =+ "libinotifytools"
>
Why is this needed ?

> +FILES_libinotifytools = "${libdir}/lib*.so.*"
> --
> 1.9.1
>
> --
> ___
> 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


[oe] [meta-oe][PATCH] inotify-tools: add recipe

2016-10-17 Thread Andrea Galbusera
inotify-tools is a C library and a set of command-line programs for Linux
providing a simple interface to inotify.

Signed-off-by: Andrea Galbusera 
---
 .../inotifywait-fix-compile-error-with-GCC-6.patch | 45 ++
 .../inotify-tools/inotify-tools_git.bb | 23 +++
 2 files changed, 68 insertions(+)
 create mode 100644 
meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
 create mode 100644 meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb

diff --git 
a/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
 
b/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
new file mode 100644
index 000..ba042a2
--- /dev/null
+++ 
b/meta-oe/recipes-support/inotify-tools/inotify-tools/inotifywait-fix-compile-error-with-GCC-6.patch
@@ -0,0 +1,45 @@
+From 7affb288d6c0726e7b1ebc317a878927b6ef0d02 Mon Sep 17 00:00:00 2001
+From: Andrea Galbusera 
+Date: Tue, 13 Sep 2016 08:10:29 +0200
+Subject: [PATCH] inotifywait: fix compile error with GCC 6
+
+Fails to compile with misleading-indentation error
+
+| src/inotifywait.c: In function 'output_event_csv':
+| src/inotifywait.c:126:5: error: this 'if' clause does not guard... 
[-Werror=misleading-indentation]
+|  if (filename != NULL)
+|  ^~
+| src/inotifywait.c:129:2: note: ...this statement, but the latter is 
misleadingly indented as if it is guarded by the 'if'
+|   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask ) ) );
+|   ^~
+| cc1: all warnings being treated as errors
+
+Fix indentation to resolve.
+
+Upstream-Status: Submitted [https://github.com/rvoicilas/inotify-tools/pull/66]
+
+Signed-off-by: Andrea Galbusera 
+---
+ src/inotifywait.c | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/src/inotifywait.c b/src/inotifywait.c
+index c5ce5e3..404a85b 100644
+--- a/src/inotifywait.c
 b/src/inotifywait.c
+@@ -122,9 +122,9 @@ void validate_format( char * fmt ) {
+ 
+ 
+ void output_event_csv( struct inotify_event * event ) {
+-char *filename = csv_escape(inotifytools_filename_from_wd(event->wd));
+-if (filename != NULL)
+-printf("%s,", filename);
++  char *filename = csv_escape(inotifytools_filename_from_wd(event->wd));
++  if (filename != NULL)
++  printf("%s,", filename);
+ 
+   printf("%s,", csv_escape( inotifytools_event_to_str( event->mask ) ) );
+   if ( event->len > 0 )
+-- 
+1.9.1
+
diff --git a/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb 
b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
new file mode 100644
index 000..a8ef557
--- /dev/null
+++ b/meta-oe/recipes-support/inotify-tools/inotify-tools_git.bb
@@ -0,0 +1,23 @@
+SUMMARY = "Command line tools and C library providing a simple interface to 
inotify"
+AUTHOR = "Rohan McGovern "
+HOMEPAGE = "http://wiki.github.com/rvoicilas/inotify-tools";
+SECTION = "console/devel"
+LICENSE = "GPL-2.0"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ac6c26e52aea428ee7f56dc2c56424c6"
+
+SRCREV = "1df9af4d6cd0f4af4b1b19254bcf056aed4ae395"
+PV = "3.14+git${SRCPV}"
+
+SRC_URI = "git://github.com/rvoicilas/${BPN} \
+   file://inotifywait-fix-compile-error-with-GCC-6.patch \
+  "
+
+S = "${WORKDIR}/git"
+
+inherit autotools
+
+EXTRA_OECONF = "--disable-doxygen"
+
+PACKAGES =+ "libinotifytools"
+
+FILES_libinotifytools = "${libdir}/lib*.so.*"
-- 
1.9.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel