Re: [PATCH] ipvsadm: allow to redefine AR (libipvs)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



	Hello,

On Sat, 5 Feb 2022, Sergey V. Lobanov wrote:

> libipvs Makefile uses `ar` to build archive. On cross-compile
> cross-platfrom build it fails due system ar might be incompatible
> with the objects generated by $(CC). It happens if build ipvsadm
> on macos using GCC Toolchain (Linux target)
> 
> This patch allows to redefine `ar` using AR var in libipvs Makefile
> 
> Signed-off-by: Sergey V. Lobanov <sergey@xxxxxxxxxx>

	Looks good to me, thanks!

Acked-by: Julian Anastasov <ja@xxxxxx>

> ---
>  libipvs/Makefile | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libipvs/Makefile b/libipvs/Makefile
> index f845c8b..7e02d4d 100644
> --- a/libipvs/Makefile
> +++ b/libipvs/Makefile
> @@ -1,5 +1,6 @@
>  # Makefile for libipvs
>  
> +AR		= ar
>  CC		= gcc
>  CFLAGS		= -Wall -Wunused -Wstrict-prototypes -g -fPIC
>  ifneq (0,$(HAVE_NL))
> @@ -30,7 +31,7 @@ SHARED_LIB	= libipvs.so
>  all:		$(STATIC_LIB) $(SHARED_LIB)
>  
>  $(STATIC_LIB):	libipvs.o ip_vs_nl_policy.o
> -		ar rv $@ $^
> +		$(AR) rv $@ $^
>  
>  $(SHARED_LIB):	libipvs.o ip_vs_nl_policy.o
>  		$(CC) -shared -Wl,-soname,$@ -o $@ $^
> -- 
> 2.32.0 (Apple Git-132)

Regards

--
Julian Anastasov <ja@xxxxxx>




[Index of Archives]     [Linux Filesystem Devel]     [Linux NFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]     [X.Org]

  Powered by Linux