> -----Original Message----- > From: Stefan Bodewig [mailto:[EMAIL PROTECTED] > Sent: Wednesday, 6 December 2000 23:32 > To: [EMAIL PROTECTED] > Subject: Re: Expansion in includes > > > Conor MacNeill <[EMAIL PROTECTED]> wrote: > > > if gr is not defined ${gr} is not expanded. Actually I think it > > should throw a BuildException. I'm going to make that change. > > Stop, we already had that (as well as pronting null there). > > The current behavior is due to popular user request, although we > should at least log a warning IMHO.
Well, throwing an exception actually feels right to me, but the current behaviour *IS* required due to properties files and the possibility of property forward references. OK, I'll change it to log a warning. Hmm, that requires the project object to be passed in. Maybe replaceProperties should be a Project method and not a static method in ProjectHelper. Cheers Conor
