Re: [oe] [meta-oe][PATCH] inotify-tools: add recipe
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
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
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