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 [EMAIL PROTECTED] PGP Key ID 36E7CD09 http://homepage.ruhr-uni-bochum.de/Marcus.Brinkmann/
