Hi,

On Sun, 2005-04-10 at 12:17 -0700, [EMAIL PROTECTED] wrote:
> On Apr 10, 2005, at 12:31 AM, Matthijs Mohlmann wrote:
> > On Sat, 2005-04-09 at 22:42 -0700, [EMAIL PROTECTED] wrote:
> >> On Apr 9, 2005, at 8:26 AM, Matthijs Mohlmann wrote:
> >>> Hi
> >>>
> >>> I've reviewed the patch and fixed the number of problems you noticed.
> >>> Can you try to build it with the new patch i provide in this mail ?
> >> Thanks! Now it works rad like it should! 100% : )
> >>
> >> Tried long names, names with dots, & multiple files - all good : )
> > Very good.
> 
> Well - only thing it doesn't do is include single files, instead of 
> whole directories -
> 
> This attached version of your patch does include both single files & 
> whole directories -
> 
> It's simply extra -
> 
> 
>       file(val.c_str(), false);
> 
Nice, but my intention was to include a full directory with
configuration items. You can use this to include just a file but maybe
then it's more nice to add a feature that only includes a file with a
configuration parameter in the main config.

> 
> No doubt there's a more sophisticated solution, but I tested this & it 
> works - what do you think?
> 
> Many thanks!
> 
Also many thanks for creating this. :)

I like it when people give feedback, many thanks :)

> Jack
> 

Regards,

Matthijs Mohlmann

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to