Re: [QAntenna-devel] Bug closed!

[Michael B. <mi...@ba...>]

Joop Stakenborg schrieb:
> Op zondag 26-08-2007 om 18:32 uur [tijdzone +0200], schreef Michael
> Barjenbruch:
> 
>> Well, that patch is just for Debian. But you could do it the same way
>> for others, too. Do you know if other distors (e.g. Fedora) have options
>> like nostrip? A work around for other distros would be doing
>> "DEB_BUILD_OPTIONS=nostrip gamke" instead of just "gamke".
>>
> 
> Michael proposed to use the following snippet in src/src.pro:
> 
>     DEB_BUILD = $$system(echo \$DEB_BUILD_OPTIONS)
>     contains(DEB_BUILD, nostrip) {
>         QMAKE_STRIP=:
>     }
> 
> I have checked the Debian package with the QMAKE_STRIP variable, qmake
> seems to ignore it completely. There are several bug reports about this
> on the web. I will always end up with a line in the Makefile saying:
> 
> -strip "$(INSTALL_ROOT)/usr/bin/$(QMAKE_TARGET)"
> 
> So, qmake is buggy.

Which version of qmake do you use? I tried the latest version of Etch
and it worked. Maybe you have to delete the Makefile before running
qmake or something like that.

> However, I have fixed the Debian package now, by doing the following:
> 
> INSTALL_PROGRAM = install
> ifeq (,$(findstring nostrip,$(DEB_BUILD_OPTIONS)))
>         INSTALL_PROGRAM += -s
> endif
> 
> and a bit further down:
> 
> $(INSTALL_PROGRAM) bin/qantenna $(CURDIR)/debian/qantenna/usr/bin
> 
> This seems to work okay.
> Michael's patch can be deleted from SVN if needed.

Ok, I'll do so.

> Another thing:
> src/Makefile contains lines like:
> -strip $(INSTALL_ROOT)/usr/share/doc/qantenna/examples/EX1.nec
> Of course we can never strip text files... ;-)

Well, those files were set as executable. Maybe that was the reason.