All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-networking][PATCH] lksctp-tools: make conpatible to autoconf-2.70
@ 2021-02-04 10:34 hongxu
  2021-02-04 17:25 ` Khem Raj
  0 siblings, 1 reply; 3+ messages in thread
From: hongxu @ 2021-02-04 10:34 UTC (permalink / raw)
  To: raj.khem; +Cc: openembedded-devel

...
./configure: line 16398: syntax error: unexpected end of file
...

Backport a commit from upstream to fix it

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 .../corosync/corosync_3.0.3.bb                |  1 +
 ....m4-make-conpatible-to-autoconf-2.70.patch | 89 +++++++++++++++++++
 2 files changed, 90 insertions(+)
 create mode 100644 meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch

diff --git a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
index c0531d386..7e0b5d5ca 100644
--- a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
+++ b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
@@ -8,6 +8,7 @@ SECTION = "base"
 inherit autotools pkgconfig systemd useradd
 
 SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz \
+           file://0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch \
            file://corosync.conf \
           "
 SRC_URI[sha256sum] = "20eb903eb984f6a728282c199825e442e8bba869acefd22390076ef3a33a4ded"
diff --git a/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
new file mode 100644
index 000000000..484af4e0f
--- /dev/null
+++ b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
@@ -0,0 +1,89 @@
+From d6d7130f0a2e3b81880fca29966e42c1b2be40a7 Mon Sep 17 00:00:00 2001
+From: Sergei Trofimovich <slyfox@gentoo.org>
+Date: Fri, 8 Jan 2021 22:22:52 +0000
+Subject: [PATCH] m4/sctp.m4: make conpatible to autoconf-2.70
+
+On recently released `autoconf-2.70` generated `./configure`
+fails as:
+
+```
+$ ./configure
+...
+checking for struct sctp_event_subscribe.sctp_stream_reset_event... yes
+checking for gcc options needed to detect all undeclared functions... none needed
+./configure: line 16464: syntax error: unexpected end of file
+```
+
+This happens becuase new autoconf generates less whitespace:
+
+```
+{
+if ...
+...
+fi}
+```
+
+It requires at least whitespace between `fi` and `}`.
+
+As input already has newlines the change just drops extra `{}`.
+
+Tested on `autoconf-2.69` and `autoconf-2.70`.
+
+Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
+Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
+
+Upstream-Status: Backport [git://github.com/sctp/lksctp-tools.git]
+Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
+
+---
+ m4/sctp.m4 | 12 ++++++------
+ 1 file changed, 6 insertions(+), 6 deletions(-)
+
+diff --git a/m4/sctp.m4 b/m4/sctp.m4
+index 6593517..94527a6 100644
+--- a/m4/sctp.m4
++++ b/m4/sctp.m4
+@@ -6,7 +6,7 @@
+ 
+ # Macros to assist on probing kernel features
+ #   Probes if a type is defined
+-AC_DEFUN([LKSCTP_CHECK_TYPE], [{
++AC_DEFUN([LKSCTP_CHECK_TYPE], [
+ AC_CHECK_TYPE([$1],
+ 	AC_DEFINE([$2], 1,
+ 		  [Define if $1 is present.])
+@@ -22,10 +22,10 @@ AC_CHECK_TYPE([$1],
+ #ifdef HAVE_LINUX_SCTP_H
+ # include <linux/sctp.h>
+ #endif
+-])}])
++])])
+ 
+ #   Probes if a struct has a given member
+-AC_DEFUN([LKSCTP_CHECK_MEMBER], [{
++AC_DEFUN([LKSCTP_CHECK_MEMBER], [
+ AC_CHECK_MEMBER([$1],
+ 	AC_DEFINE([$2], 1,
+ 		  [Define if $1 is present.])
+@@ -41,10 +41,10 @@ AC_CHECK_MEMBER([$1],
+ #ifdef HAVE_LINUX_SCTP_H
+ # include <linux/sctp.h>
+ #endif
+-])}])
++])])
+ 
+ #   Probes if a declaration is present
+-AC_DEFUN([LKSCTP_CHECK_DECL], [{
++AC_DEFUN([LKSCTP_CHECK_DECL], [
+ AC_CHECK_DECL([$1],
+ 	AC_DEFINE([$2], 1,
+ 		  [Define if $1 is present.])
+@@ -60,4 +60,4 @@ AC_CHECK_DECL([$1],
+ #ifdef HAVE_LINUX_SCTP_H
+ # include <linux/sctp.h>
+ #endif
+-])}])
++])])
+-- 
+2.27.0
+
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [meta-networking][PATCH] lksctp-tools: make conpatible to autoconf-2.70
  2021-02-04 10:34 [meta-networking][PATCH] lksctp-tools: make conpatible to autoconf-2.70 hongxu
@ 2021-02-04 17:25 ` Khem Raj
  2021-02-05  5:56   ` hongxu
  0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2021-02-04 17:25 UTC (permalink / raw)
  To: Hongxu Jia; +Cc: openembedded-devel



On 2/4/21 2:34 AM, Hongxu Jia wrote:
> ...
> ./configure: line 16398: syntax error: unexpected end of file
> ...
> 
> Backport a commit from upstream to fix it
> 
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>   .../corosync/corosync_3.0.3.bb                |  1 +
>   ....m4-make-conpatible-to-autoconf-2.70.patch | 89 +++++++++++++++++++
>   2 files changed, 90 insertions(+)
>   create mode 100644 meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
> 
> diff --git a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
> index c0531d386..7e0b5d5ca 100644
> --- a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
> +++ b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb

should this be 
meta-networking/recipes-support/lksctp-tools/lksctp-tools_1.0.18.bb 
instead ?

> @@ -8,6 +8,7 @@ SECTION = "base"
>   inherit autotools pkgconfig systemd useradd
>   
>   SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz \
> +           file://0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch \
>              file://corosync.conf \
>             "
>   SRC_URI[sha256sum] = "20eb903eb984f6a728282c199825e442e8bba869acefd22390076ef3a33a4ded"
> diff --git a/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
> new file mode 100644
> index 000000000..484af4e0f
> --- /dev/null
> +++ b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
> @@ -0,0 +1,89 @@
> +From d6d7130f0a2e3b81880fca29966e42c1b2be40a7 Mon Sep 17 00:00:00 2001
> +From: Sergei Trofimovich <slyfox@gentoo.org>
> +Date: Fri, 8 Jan 2021 22:22:52 +0000
> +Subject: [PATCH] m4/sctp.m4: make conpatible to autoconf-2.70
> +
> +On recently released `autoconf-2.70` generated `./configure`
> +fails as:
> +
> +```
> +$ ./configure
> +...
> +checking for struct sctp_event_subscribe.sctp_stream_reset_event... yes
> +checking for gcc options needed to detect all undeclared functions... none needed
> +./configure: line 16464: syntax error: unexpected end of file
> +```
> +
> +This happens becuase new autoconf generates less whitespace:
> +
> +```
> +{
> +if ...
> +...
> +fi}
> +```
> +
> +It requires at least whitespace between `fi` and `}`.
> +
> +As input already has newlines the change just drops extra `{}`.
> +
> +Tested on `autoconf-2.69` and `autoconf-2.70`.
> +
> +Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
> +Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
> +
> +Upstream-Status: Backport [git://github.com/sctp/lksctp-tools.git]
> +Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> +
> +---
> + m4/sctp.m4 | 12 ++++++------
> + 1 file changed, 6 insertions(+), 6 deletions(-)
> +
> +diff --git a/m4/sctp.m4 b/m4/sctp.m4
> +index 6593517..94527a6 100644
> +--- a/m4/sctp.m4
> ++++ b/m4/sctp.m4
> +@@ -6,7 +6,7 @@
> +
> + # Macros to assist on probing kernel features
> + #   Probes if a type is defined
> +-AC_DEFUN([LKSCTP_CHECK_TYPE], [{
> ++AC_DEFUN([LKSCTP_CHECK_TYPE], [
> + AC_CHECK_TYPE([$1],
> + 	AC_DEFINE([$2], 1,
> + 		  [Define if $1 is present.])
> +@@ -22,10 +22,10 @@ AC_CHECK_TYPE([$1],
> + #ifdef HAVE_LINUX_SCTP_H
> + # include <linux/sctp.h>
> + #endif
> +-])}])
> ++])])
> +
> + #   Probes if a struct has a given member
> +-AC_DEFUN([LKSCTP_CHECK_MEMBER], [{
> ++AC_DEFUN([LKSCTP_CHECK_MEMBER], [
> + AC_CHECK_MEMBER([$1],
> + 	AC_DEFINE([$2], 1,
> + 		  [Define if $1 is present.])
> +@@ -41,10 +41,10 @@ AC_CHECK_MEMBER([$1],
> + #ifdef HAVE_LINUX_SCTP_H
> + # include <linux/sctp.h>
> + #endif
> +-])}])
> ++])])
> +
> + #   Probes if a declaration is present
> +-AC_DEFUN([LKSCTP_CHECK_DECL], [{
> ++AC_DEFUN([LKSCTP_CHECK_DECL], [
> + AC_CHECK_DECL([$1],
> + 	AC_DEFINE([$2], 1,
> + 		  [Define if $1 is present.])
> +@@ -60,4 +60,4 @@ AC_CHECK_DECL([$1],
> + #ifdef HAVE_LINUX_SCTP_H
> + # include <linux/sctp.h>
> + #endif
> +-])}])
> ++])])
> +--
> +2.27.0
> +
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [meta-networking][PATCH] lksctp-tools: make conpatible to autoconf-2.70
  2021-02-04 17:25 ` Khem Raj
@ 2021-02-05  5:56   ` hongxu
  0 siblings, 0 replies; 3+ messages in thread
From: hongxu @ 2021-02-05  5:56 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembedded-devel

On 2/5/21 1:25 AM, Khem Raj wrote:
> [Please note: This e-mail is from an EXTERNAL e-mail address]
>
> On 2/4/21 2:34 AM, Hongxu Jia wrote:
>> ...
>> ./configure: line 16398: syntax error: unexpected end of file
>> ...
>>
>> Backport a commit from upstream to fix it
>>
>> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>> ---
>>   .../corosync/corosync_3.0.3.bb                |  1 +
>>   ....m4-make-conpatible-to-autoconf-2.70.patch | 89 +++++++++++++++++++
>>   2 files changed, 90 insertions(+)
>>   create mode 100644 
>> meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
>>
>> diff --git 
>> a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb 
>> b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
>> index c0531d386..7e0b5d5ca 100644
>> --- a/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
>> +++ b/meta-networking/recipes-extended/corosync/corosync_3.0.3.bb
>
> should this be
> meta-networking/recipes-support/lksctp-tools/lksctp-tools_1.0.18.bb
> instead ?
>
Sorry, my mistake, V2 incoming

//Hongxu

>> @@ -8,6 +8,7 @@ SECTION = "base"
>>   inherit autotools pkgconfig systemd useradd
>>
>>   SRC_URI = 
>> "https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz 
>> \
>> + file://0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch \
>>              file://corosync.conf \
>>             "
>>   SRC_URI[sha256sum] = 
>> "20eb903eb984f6a728282c199825e442e8bba869acefd22390076ef3a33a4ded"
>> diff --git 
>> a/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch 
>> b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch 
>>
>> new file mode 100644
>> index 000000000..484af4e0f
>> --- /dev/null
>> +++ 
>> b/meta-networking/recipes-support/lksctp-tools/lksctp-tools/0001-m4-sctp.m4-make-conpatible-to-autoconf-2.70.patch
>> @@ -0,0 +1,89 @@
>> +From d6d7130f0a2e3b81880fca29966e42c1b2be40a7 Mon Sep 17 00:00:00 2001
>> +From: Sergei Trofimovich <slyfox@gentoo.org>
>> +Date: Fri, 8 Jan 2021 22:22:52 +0000
>> +Subject: [PATCH] m4/sctp.m4: make conpatible to autoconf-2.70
>> +
>> +On recently released `autoconf-2.70` generated `./configure`
>> +fails as:
>> +
>> +```
>> +$ ./configure
>> +...
>> +checking for struct sctp_event_subscribe.sctp_stream_reset_event... yes
>> +checking for gcc options needed to detect all undeclared 
>> functions... none needed
>> +./configure: line 16464: syntax error: unexpected end of file
>> +```
>> +
>> +This happens becuase new autoconf generates less whitespace:
>> +
>> +```
>> +{
>> +if ...
>> +...
>> +fi}
>> +```
>> +
>> +It requires at least whitespace between `fi` and `}`.
>> +
>> +As input already has newlines the change just drops extra `{}`.
>> +
>> +Tested on `autoconf-2.69` and `autoconf-2.70`.
>> +
>> +Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
>> +Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
>> +
>> +Upstream-Status: Backport [git://github.com/sctp/lksctp-tools.git]
>> +Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
>> +
>> +---
>> + m4/sctp.m4 | 12 ++++++------
>> + 1 file changed, 6 insertions(+), 6 deletions(-)
>> +
>> +diff --git a/m4/sctp.m4 b/m4/sctp.m4
>> +index 6593517..94527a6 100644
>> +--- a/m4/sctp.m4
>> ++++ b/m4/sctp.m4
>> +@@ -6,7 +6,7 @@
>> +
>> + # Macros to assist on probing kernel features
>> + #   Probes if a type is defined
>> +-AC_DEFUN([LKSCTP_CHECK_TYPE], [{
>> ++AC_DEFUN([LKSCTP_CHECK_TYPE], [
>> + AC_CHECK_TYPE([$1],
>> +     AC_DEFINE([$2], 1,
>> +               [Define if $1 is present.])
>> +@@ -22,10 +22,10 @@ AC_CHECK_TYPE([$1],
>> + #ifdef HAVE_LINUX_SCTP_H
>> + # include <linux/sctp.h>
>> + #endif
>> +-])}])
>> ++])])
>> +
>> + #   Probes if a struct has a given member
>> +-AC_DEFUN([LKSCTP_CHECK_MEMBER], [{
>> ++AC_DEFUN([LKSCTP_CHECK_MEMBER], [
>> + AC_CHECK_MEMBER([$1],
>> +     AC_DEFINE([$2], 1,
>> +               [Define if $1 is present.])
>> +@@ -41,10 +41,10 @@ AC_CHECK_MEMBER([$1],
>> + #ifdef HAVE_LINUX_SCTP_H
>> + # include <linux/sctp.h>
>> + #endif
>> +-])}])
>> ++])])
>> +
>> + #   Probes if a declaration is present
>> +-AC_DEFUN([LKSCTP_CHECK_DECL], [{
>> ++AC_DEFUN([LKSCTP_CHECK_DECL], [
>> + AC_CHECK_DECL([$1],
>> +     AC_DEFINE([$2], 1,
>> +               [Define if $1 is present.])
>> +@@ -60,4 +60,4 @@ AC_CHECK_DECL([$1],
>> + #ifdef HAVE_LINUX_SCTP_H
>> + # include <linux/sctp.h>
>> + #endif
>> +-])}])
>> ++])])
>> +--
>> +2.27.0
>> +
>>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-02-05  5:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-04 10:34 [meta-networking][PATCH] lksctp-tools: make conpatible to autoconf-2.70 hongxu
2021-02-04 17:25 ` Khem Raj
2021-02-05  5:56   ` hongxu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.