Control: tags -1 + patch
On 2016-11-10 22:07 +0100, Sven Joachim wrote:
> On 2016-11-10 21:41 +0100, Milan Zamazal wrote:
>
>> Thanks for the report. I changed the default build architecture to
>> general-64, I guess that's a good default these days.
>
> I haven't looked at the stockfish source, but I can imagine that this is
> not going to improve the situation on 32-bit arches. If the config
> machinery cannot find out the pointer size by itself, dpkg-architecture
> provides DEB_HOST_ARCH_BITS for you.
Inspecting the build system I found out that there is no config
machinery, just a Makefile. The attached patch fixes the problem for
the usual build in Debian, although you might want to fix some other
code paths as well (e.g. if someone builds with COMP=clang).
Cheers,
Sven
Description: Don't add -m32/-m64 to CXXFLAGS
Adding those is not only unnecessary, but actually breaks the build on
many architectures, since g++ does not recognize them when built without
multilib support.
Author: Sven Joachim <[email protected]>
Bug-Debian: https://bugs.debian.org/843922
Last-Update: 2016-11-11
---
src/Makefile | 2 --
1 file changed, 2 deletions(-)
--- a/src/Makefile
+++ b/src/Makefile
@@ -156,8 +156,6 @@ ifeq ($(COMP),gcc)
ifeq ($(OS),Android)
CXXFLAGS += -m$(bits)
endif
- else
- CXXFLAGS += -m$(bits)
endif
ifneq ($(KERNEL),Darwin)
