On Sun, Feb 04, 2007 at 02:05:31PM +0600, Aleksey Demakov wrote:
> I do not think that there is much need for --enable-debug option
> as configure allows to specify whatever CFALGS you like in the
> command line:
> 
> ./configure CFLAGS="-g -O0"
> 
> However if there are people who like it this might be ok.

Hmm, --enable-debug seems to become standard for autotoolized programs,
and it gives "minor-skilled" users a quick way to build everything with
debug support, i.e. for using gdb. In this case I assume it's even more
important, because pnet has problems with optimization, so the
maintainers might have a precise rule what should be done to enable
debugging. And, because configure handles these switches down to the
inferior configures of libffi and libgc, these libraries might have a
different oppinion about what has to be done to enable debugging. So I
think it is a better solution to have it abstract.

> But it seems that your patch intervenes not only in the case when
> --enable-debug is provided but also when it does not,  That is it
> overrides the -O option that might be specified in the command
> line with -O2. I do not think this is any good as it goes against pretty
> common way of specifying CFLAGS that is incidentally also what
> I accustomed to.

Thanks for the catch, I'll re-check.

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
     Hannoversche Str. 2, 31134 Hildesheim, Germany
   Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9

_______________________________________________
Developers mailing list
[email protected]
http://dotgnu.org/mailman/listinfo/developers

Reply via email to