threads / patch / 50951

patchMakefile: use fileno macro work around on AIX

Subject: [PATCH] Makefile: use fileno macro work around on AIX

## tl;dr

2 messages between Apr 18, 2019 and Apr 19, 2019. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

CHIGOT, CLEMENT· Apr 18, 2019, 08:55 UTC · lore

Declare FILENO_IS_A_MACRO on AIX

On AIX, fileno(fp) is a macro and need to use the work around already made for BSD's.

Signed-off-by: Clément Chigot <clement.chigot@atos.net>
---
 config.mak.uname | 1 +
 1 file changed, 1 insertion(+)

Show changes to config.mak.uname +1 −0

diff --git a/config.mak.uname b/config.mak.uname
index 41e85fab1c..86cbe47627 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -269,6 +269,7 @@ ifeq ($(uname_S),AIX)
 	INTERNAL_QSORT = UnfortunatelyYes
 	NEEDS_LIBICONV = YesPlease
 	BASIC_CFLAGS += -D_LARGE_FILES
+	FILENO_IS_A_MACRO = UnfortunatelyYes
 	ifeq ($(shell expr "$(uname_V)" : '[1234]'),1)
 		NO_PTHREADS = YesPlease
 	else

-- 
2.17.1

Junio C Hamano· Apr 19, 2019, 05:01 UTC · re: CHIGOT, CLEMENT · lore

Re: [PATCH] Makefile: use fileno macro work around on AIX

"CHIGOT, CLEMENT" <clement.chigot@atos.net> writes:

Show 8 quoted lines

> Declare FILENO_IS_A_MACRO on AIX
>
> On AIX, fileno(fp) is a macro and need to use the work around already made for BSD's. 
>
> Signed-off-by: Clément Chigot <clement.chigot@atos.net>
> ---
>  config.mak.uname | 1 +
>  1 file changed, 1 insertion(+)

I do not have an AIX box, so I'll take this as-is.

Thanks.

Show 13 quoted lines

>
> diff --git a/config.mak.uname b/config.mak.uname
> index 41e85fab1c..86cbe47627 100644
> --- a/config.mak.uname
> +++ b/config.mak.uname
> @@ -269,6 +269,7 @@ ifeq ($(uname_S),AIX)
>  	INTERNAL_QSORT = UnfortunatelyYes
>  	NEEDS_LIBICONV = YesPlease
>  	BASIC_CFLAGS += -D_LARGE_FILES
> +	FILENO_IS_A_MACRO = UnfortunatelyYes
>  	ifeq ($(shell expr "$(uname_V)" : '[1234]'),1)
>  		NO_PTHREADS = YesPlease
>  	else