Bug#43787: well, here it is: alternate proposal (was: changed title...)
On Fri, Sep 10, 1999 at 09:56:24AM -0400, Ben Collins wrote:
> I don't mind this proposal as long as it satisfies everyone. I'll redo mine
> to match Marcus's suggested one if I don't hear any complaints.
Thanks for your positive response. I hope people speak up if they have
complaints, so they can (hopefully) be addressed.
> Something like this for the options?:
>
> ------------------
> CFLAGS := -O2
> INSTALL := install -s
>
> ifneq (,$(findstring $(DEB_BUILD_OPTIONS),debug))
Note that this does not work, because findstring does search the first
string in the second. So you have to use
ifneq (,$(findstring debug,$(DEB_BUILD_OPTIONS)))
> CFLAGS += -g
> ifneq (,$(findstring $(DEB_BUILD_OPTIONS),nostrip))
Same here of course.
> INSTALL := install
> endif
> endif
> ------------------
>
> I set it up this way, because I don't think a `nostrip' is pertinent unless
> building with `debug' also. In this way "DEB_BUILD_OPTIONS=nostrip" will not
> do anything, but "DEB_BUILD_OPTIONS='debug nostrip'" will have the desired
> affect.
Well, I thought about it but didn't saw a reason not to support a single
nostrip (without debug). However, I don't really care either way.
(I don't think it is a bug to strip if only "nostrip" is given, on the oher
hand, I can't think of a good reason to do that, too.) i would like to hear
other peoples opinion on that.
Thanks,
Marcus
--
`Rhubarb is no Egyptian god.' Debian http://www.debian.org Check Key server
Marcus Brinkmann GNU http://www.gnu.org for public PGP Key
Marcus.Brinkmann@ruhr-uni-bochum.de PGP Key ID 36E7CD09
http://homepage.ruhr-uni-bochum.de/Marcus.Brinkmann/
Reply to: