Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 11 May 2020 13:31:10 +0200
From:      Hans Petter Selasky <hps@selasky.org>
To:        Jan Beich <jbeich@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r534893 - head/multimedia/webcamd
Message-ID:  <eddb95ce-5085-e0be-7c3a-d60fb912eb98@selasky.org>
In-Reply-To: <tv0m-g1q4-wny@FreeBSD.org>
References:  <202005110834.04B8YTMU056795@repo.freebsd.org> <tv0m-g1q4-wny@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2020-05-11 13:17, Jan Beich wrote:
> Hans Petter Selasky <hselasky@FreeBSD.org> writes:
> 
>> Author: hselasky
>> Date: Mon May 11 08:34:28 2020
>> New Revision: 534893
>> URL: https://svnweb.freebsd.org/changeset/ports/534893
>>
>> Log:
>>    Only set COMPAT32 option when building 64-bit platforms.
>>    
>>    PR:		245698
>>    Approved by:	pi (implicit)
> [...]
>> -.if exists(/usr/lib32/libcuse.so)
>> +.include <bsd.port.options.mk>
>> +
>> +.if ${ARCH} == amd64 || ${ARCH} == aarch64
>>   OPTIONS_DEFAULT+=	COMPAT32
>>   .endif
> 
> Looks pointless. Not only it misses other 64-bit archs like powerpc64 but
> also doesn't address bug 245698 where user accidentally enabled COMPAT option.

Hi Jan,

The COMPAT32 option was enabled by error because the user tried to build 
both 32-bit and 64-bit world and ports from the same jail.

The COMPAT32 option is mostly relevant for platforms that have lots of 
binary applications in 32-bit mode, like wine-i386-devel, and is not 
that relevant for power64 nor sparc64, at the moment.

It would be better if the ports tree could set a variable by default if 
we are building for a 64-bit platform!

.if ${ARCH64} != ""

.endif

--HPS

> 
> Try the following instead
> 
> Index: multimedia/webcamd/Makefile
> ===================================================================
> --- multimedia/webcamd/Makefile	(revision 534907)
> +++ multimedia/webcamd/Makefile	(working copy)
> @@ -20,15 +20,13 @@ USE_RC_SUBR=	webcamd
>   SUB_FILES=	webcamd.conf
>   
>   OPTIONS_DEFINE=	COMPAT32 DEBUG HAL VT_CLIENT VT_SERVER DVB INPUT KEYBOARD MOUSE RADIO WEBCAM
> -OPTIONS_DEFAULT=DVB HAL INPUT RADIO WEBCAM
> +OPTIONS_DEFAULT=COMPAT32 DVB HAL INPUT RADIO WEBCAM
>   
>   COMPAT32_DESC=	Enable 32-bit compatibility support
>   COMPAT32_MAKE_ARGS=	HAVE_COMPAT32="YES"
>   
> -.include <bsd.port.options.mk>
> -
> -.if ${ARCH} == amd64 || ${ARCH} == aarch64
> -OPTIONS_DEFAULT+=	COMPAT32
> +.if !exists(/usr/lib32/libcuse.so)
> +OPTIONS_EXCLUDE+=	COMPAT32
>   .endif
>   
>   HAL_DESC=	Build with HAL and DBUS support
> 




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?eddb95ce-5085-e0be-7c3a-d60fb912eb98>