From: Rafael L. <lab...@ps...> - 2003-01-30 14:31:26
|
* Alan W. Irwin <ir...@be...> [2003-01-29 18:37]: > On Tue, 28 Jan 2003, Rafael Laboissiere wrote: > > > 3) include/plConfig.h.in: > > > > Removed all #undef PLD_*. since they are already in plDevs.h.in. > > This part of your patch I don't quite understand. This file is currently > generated by autoheader so it does not exist in CVS. Are you saying we > should take the latest autoheader-generated version, put it under CVS > control, and then make those recommended changes to get rid of the device > stuff (as well as never running autoheader again under any circumstances)? See my last reply to Maurice's message regarding the use of config.h.in. Yes, part of my proposal is to put plConfig.h.in under CVS control. This file would contain the user's space #defines, while config.h.in would be autoheader-generated (since only the first file given to AC_CONFIG_HEADERS is touched by autoheader). > If we run autoheader ever again after this file is edited, then I believe > the device stuff will just come back again. This happen in the current implementation because I did a poor implementation of header file processing in Autotools. With the new proposed scheme things will be much better. -- Rafael |