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 <svenj...@gmx.de> 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)