From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <legion@altlinux.ru>
Date: Wed, 18 Aug 2021 19:36:30 +0200
From: Alexey Gladkov <legion@altlinux.ru>
To: Michael Shigorin <mike@altlinux.org>
Message-ID: <20210818173630.hphy4uaq5zy3cfxs@example.org>
References: <20210818162115.2813079-1-mike@altlinux.org>
 <20210818162115.2813079-3-mike@altlinux.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20210818162115.2813079-3-mike@altlinux.org>
Cc: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH 2/2] [e2k] ftbfs workaround (disable TLS)
X-BeenThere: make-initrd@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: make-initrd@lists.altlinux.org
List-Id: <make-initrd.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/make-initrd>
List-Post: <mailto:make-initrd@lists.altlinux.org>
List-Help: <mailto:make-initrd-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Aug 2021 17:36:32 -0000
Archived-At: <http://lore.altlinux.org/make-initrd/20210818173630.hphy4uaq5zy3cfxs@example.org/>
List-Archive: <http://lore.altlinux.org/make-initrd/>

On Wed, Aug 18, 2021 at 07:21:15PM +0300, Michael Shigorin wrote:
> The problem looks like this:
> 
> lcc: "/usr/include/bits/byteswap.h", line 105: error #1348: declaration hides
>           variable "__x" (declared at line 706 of
>           "/usr/src/RPM/BUILD/make-initrd-2.20.1/external/busybox/upstream/networking/tls.c")
>           [-Werror=shadow=global]
>         ({ register __uint64_t __v, __x = (x);                                \
>                                     ^
>  in expansion of macro "__bswap_64" at line 37 of "/usr/include/byteswap.h"
>  in expansion of macro "bswap_64" at line 198 of
>            "/usr/src/RPM/BUILD/make-initrd-2.20.1/external/busybox/upstream/include/platform.h"
>  in expansion of macro "bb_bswap_64" at line 214 of
>            "/usr/src/RPM/BUILD/make-initrd-2.20.1/external/busybox/upstream/include/platform.h"
>  in expansion of macro "SWAP_BE64" at line 706 of
>            "/usr/src/RPM/BUILD/make-initrd-2.20.1/external/busybox/upstream/networking/tls.c"
>  in expansion of macro "SWAP_BE64" at line 706 of
>            "/usr/src/RPM/BUILD/make-initrd-2.20.1/external/busybox/upstream/networking/tls.c"
>         tls->write_seq64_be = SWAP_BE64(1 + SWAP_BE64(tls->write_seq64_be));
>                               ^
> 
> We think it's improper for toolchain to define
> quite common single-character symbols like this,
> thus I've filed mcst bug #6061.

Это что, шутка ?

> ---
>  make-initrd.spec | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/make-initrd.spec b/make-initrd.spec
> index bc48e97..907d3ed 100644
> --- a/make-initrd.spec
> +++ b/make-initrd.spec
> @@ -204,6 +204,11 @@ Feature adds smart card daemon and smart card utilities.
>  
>  %prep
>  %setup
> +%ifarch %e2k
> +# avoid ftbfs due to __v shadowing in /usr/include/bits/byteswap.h (mcst#6061)
> +sed -i -r 's,^CONFIG_(TLS|SSL_CLIENT|FEATURE_WGET_HTTPS)=y,CONFIG_\1=n,' \
> +	external/busybox/config
> +%endif

Миш, это неправильно и я не могу с этим согласиться даже с %ifarch.

Выключать tls только из-за глючнсти /usr/include/bits/byteswap.h в
какой-то закрытой реализации libc я не буду. Почините byteswap.h.

-- 
Rgrds, legion