[Box Backup-dev] COMMIT r655 - box/chris/general/infrastructure

[Ben Summers]

On 19 Jul 2006, at 19:44, subversion at fluffy.co.uk wrote:

> Log:
> * makebuildenv.pl
> - Added lib/server to implicit includes to help tests find  
> ServerControl.h
>
>
> Modified: box/chris/general/infrastructure/makebuildenv.pl
> ===================================================================
> --- box/chris/general/infrastructure/makebuildenv.pl	2006-07-19  
> 18:43:52 UTC (rev 654)
> +++ box/chris/general/infrastructure/makebuildenv.pl	2006-07-19  
> 18:44:28 UTC (rev 655)
> @@ -40,7 +40,8 @@
>  my $windows_include_path = "";
>  if ($target_windows)
>  {
> -	$windows_include_path = "-I../../lib/win32 -I../../lib/common ";
> +	$windows_include_path = "-I../../lib/win32 -I ../../lib/common " .
> +		"-I../../lib/server ";
>  	push @implicit_deps, "lib/win32";


Er? Shouldn't you be using modules.txt for dependency info? Or use  
the @implicit_deps array?

Random dependencies going all over the place is nasty. Is there a  
cleaner way to design this?

Ben