[PATCH] extern-inline: Pull __header_inline definition

2021-02-09 Thread Roman Bolshakov
* m4/extern-inline.m4 (gl_EXTERN_INLINE): __header_inline is defined in but the header is not included and as result _GL_EXTERN_INLINE doesn't work as intended on macOS. Signed-off-by: Roman Bolshakov --- Hi, The issue prevents -O1/-O0 compilation of libtasn1 on macOS: https://gitlab.com

Re: [PATCH v3 0/2] Fix syntax-check on macOS/FreeBSD

2018-12-20 Thread Roman Bolshakov
On Thu, Dec 13, 2018 at 06:34:51PM +0300, Roman Bolshakov wrote: > Hello, > > There was an issue with syntax-check on FreeBSD reported a few years > ago: > https://www.redhat.com/archives/libvir-list/2015-August/msg00758.html > http://lists.gnu.org/archive/html/bug-gnulib/201

[PATCH v3 2/2] maint.mk: Replace grep with $(GREP)

2018-12-13 Thread Roman Bolshakov
grep we don't get the messages in syntax-check: prohibit_diagnostic_without_format grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression Signed-off-by: Roman Bolshakov

[PATCH v3 0/2] Fix syntax-check on macOS/FreeBSD

2018-12-13 Thread Roman Bolshakov
_ARG_MAX - removed "-n" flag in xargs invocations - added /dev/null as an extra file for uniform grep output regardless of number of files. The file isn't added if grep invocation has "-L" flag - added dependency on AC_PROG_GREP to modules/maintainer-makefile -- Best regards,

[PATCH v3 1/2] maint.mk: Split long argument lists

2018-12-13 Thread Roman Bolshakov
ds default value for "-s" flag (max-chars), xargs will feed less arguments than max-args. Signed-off-by: Roman Bolshakov --- top/maint.mk | 135 --- 1 file changed, 86 insertions(+), 49 deletions(-) diff --git a/top/maint.mk b/top/main

Re: [PATCH v2 1/2] maint.mk: Split long argument lists

2018-12-13 Thread Roman Bolshakov
On Wed, Dec 12, 2018 at 07:42:44PM -0600, Eric Blake wrote: > On 12/12/18 3:13 PM, Roman Bolshakov wrote: > > > > > @@ -845,7 +853,10 @@ sc_prohibit_always-defined_macros: > > > > case $$(echo all: | grep -l -f - Makefile) in Makefile);; *) > >

Re: [PATCH v2 1/2] maint.mk: Split long argument lists

2018-12-12 Thread Roman Bolshakov
On Tue, Dec 04, 2018 at 04:10:08PM -0600, Eric Blake wrote: > On 12/3/18 9:00 AM, Roman Bolshakov wrote: > > $(VC_LIST_EXCEPT) is usually expanded into arguments for a command. > > When a project contains too many, some operating systems can't pass all > > the arguments becau

Re: [PATCH v2 0/2] Fix syntax-check on macOS/FreeBSD

2018-12-12 Thread Roman Bolshakov
On Tue, Dec 04, 2018 at 01:44:42AM +0100, Bruno Haible wrote: > Roman Bolshakov wrote: > > - added /dev/null as an extra file for uniform grep output regardless of > > number of files. The file isn't added if grep invocation has "-L" flag > > Good point. And

Re: [PATCH v2 2/2] maint.mk: Replace grep with $(GREP)

2018-12-05 Thread Roman Bolshakov
On Tue, Dec 04, 2018 at 04:14:04PM -0600, Eric Blake wrote: > On 12/3/18 9:00 AM, Roman Bolshakov wrote: > > : Filter by file name; \ > > if test -n "$$in_files"; then \ > > - fi

[PATCH v2 1/2] maint.mk: Split long argument lists

2018-12-03 Thread Roman Bolshakov
ds default value for "-s" flag (max-chars), xargs will feed less arguments than max-args. Signed-off-by: Roman Bolshakov --- top/maint.mk | 53 +--- 1 file changed, 34 insertions(+), 19 deletions(-) diff --git a/top/maint.mk b/top/main

[PATCH v2 2/2] maint.mk: Replace grep with $(GREP)

2018-12-03 Thread Roman Bolshakov
grep we don't get the messages in syntax-check: prohibit_diagnostic_without_format grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression Signed-off-by: Roman Bolshakov

[PATCH v2 0/2] Fix syntax-check on macOS/FreeBSD

2018-12-03 Thread Roman Bolshakov
ency on AC_PROG_GREP to modules/maintainer-makefile -- Best regards, Roman Roman Bolshakov (2): maint.mk: Split long argument lists maint.mk: Replace grep with $(GREP) modules/maintainer-makefile | 1 + top/maint.mk| 145 2 files ch

Re: [PATCH 2/2] maint.mk: Replace grep with $(GREP)

2018-12-03 Thread Roman Bolshakov
On Sun, Dec 02, 2018 at 01:31:03PM +0100, Bruno Haible wrote: > Roman Bolshakov wrote: > > I'm quite new to gnulib but thanks to Eric and your comments that should do > > it: > > > > diff --git a/modules/maintainer-makefile b/modules/maintainer-makefile > >

Re: [PATCH 1/2] maint.mk: Split long argument lists

2018-12-03 Thread Roman Bolshakov
On Sun, Dec 02, 2018 at 01:26:52PM +0100, Bruno Haible wrote: > Roman Bolshakov wrote: > > But then we will need to correct calculation of VC_ARG_MAX. We can take > > formulae from [2]: > > expr `getconf ARG_MAX` - `env|wc -c` - `env|egrep '^[^ ]+='|wc -l` \* 4 - >

Re: [PATCH 1/2] maint.mk: Split long argument lists

2018-12-03 Thread Roman Bolshakov
On Mon, Dec 03, 2018 at 08:16:22AM +0100, Bernhard Voelker wrote: > On 11/30/18 12:14 PM, Roman Bolshakov wrote: > > May I ask you to review what way we should go with ARG_MAX? > > > > I'm okay with both ways whether it's: > > * computing effective argument length an

Re: [PATCH 1/2] maint.mk: Split long argument lists

2018-11-30 Thread Roman Bolshakov
Hi Bruno, May I ask you to review what way we should go with ARG_MAX? I'm okay with both ways whether it's: * computing effective argument length and passing it to "-s" option; * or exploiting behaviour of GNU/BSD xargs and specifying "-n" beyond the limit. Thank you, Roman

Re: [PATCH 2/2] maint.mk: Replace grep with $(GREP)

2018-11-28 Thread Roman Bolshakov
On Tue, Nov 27, 2018 at 07:47:12PM +0100, Bruno Haible wrote: > Eric Blake wrote: > > > In other words, can you guarantee that $(GREP) will never expand to empty? > > > > If we change gl_INIT() to AC_REQUIRE([AC_PROG_GREP]), then that should > > be sufficient to ensure $(GREP) is set by the time

Re: [PATCH 1/2] maint.mk: Split long argument lists

2018-11-28 Thread Roman Bolshakov
On Tue, Nov 27, 2018 at 07:40:24PM +0100, Bruno Haible wrote: > Roman Bolshakov wrote: > > if test -n "$$files"; then > > \ > > if test -n "$$prohibit"; then

Re: [PATCH 1/2] maint.mk: Split long argument lists

2018-11-28 Thread Roman Bolshakov
On Tue, Nov 27, 2018 at 07:19:43PM +0100, Bruno Haible wrote: > Hi, > > > The workaround is to split argument list into chunks that operating > > system can process. "getconf ARG_MAX" is used to determine size of the > > chunk. > > Two questions on this: > > 1) People say that 'getconf ARG_MAX'

[PATCH 1/2] maint.mk: Split long argument lists

2018-11-27 Thread Roman Bolshakov
5d5f55e.60...@redhat.com Signed-off-by: Roman Bolshakov --- top/maint.mk | 55 ++-- 1 file changed, 36 insertions(+), 19 deletions(-) diff --git a/top/maint.mk b/top/maint.mk index 4889ebacc..c4f21f947 100644 --- a/top/maint.mk +++ b/top/maint.mk @@ -

[PATCH 2/2] maint.mk: Replace grep with $(GREP)

2018-11-27 Thread Roman Bolshakov
grep we don't get the messages in syntax-check: prohibit_diagnostic_without_format grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression grep: empty (sub)expression Signed-off-by: Roman Bolshakov

[PATCH 0/2] Fix syntax-check on macOS/FreeBSD

2018-11-27 Thread Roman Bolshakov
changes on libvirt side I can run make syntax-check on macOS. I wasn't entirely sure on indentation in the long commands. I did my best, sorry if something is wrong. -- Best regards, Roman Roman Bolshakov (2): maint.mk: Split long argument lists maint.mk: Replace grep with $(GREP) top