On Fri, Nov 10, 2017 at 05:08:53PM +0200, Alberto Garcia wrote:
> On Fri, Nov 10, 2017 at 03:49:58PM +0100, Helmut Grohne wrote:
> > ifeq ($(DEB_HOST_ARCH_BITS),32)
> >     LDFLAGS += -Wl,--no-keep-memory
> >  endif
> >  
> > -ifeq ($(DEB_BUILD_ARCH),alpha)
> > +ifeq ($(DEB_HOST_ARCH),alpha)
> >     LDFLAGS += -Wl,--no-relax
> >  endif
> 
> Wait a minute, why do you change this to DEB_HOST_ARCH here? Shouldn't
> we be using the DEB_BUILD_* variables in these two cases?

I think you are confusing build and host again.

If you check DEB_BUILD_ARCH, then you will pass -Wl,--no-relax when
building for amd64 on alpha. Since --no-relax is machine-dependent, that
seems wrong to me. In general, changing the build architecture should
not affect the resulting binary packages. Passing different flags for
different build architectures often breaks that.

Helmut

Reply via email to