[vsipl++] PATCH: installation / packaging

Mark Mitchell mark at codesourcery.com
Wed Jan 4 04:57:25 UTC 2006


Stefan Seefeld wrote:
> The appended patch adds or completes the following targets:

Wow, very cool.

>  ########################################################################
> +# Variables
> +########################################################################

I would still call this "Internal Variables", just to be a little clearer.

> +# Call this target explicitely to copy documentation back into the 

explicitly

> +# source directory.
> +ifneq '$(srcdir)' '.'

Would "ifneq $(srcdir) $(objdir)" be clearer and/or more robust?

Other than that, it looks great to me.

-- 
Mark Mitchell
CodeSourcery
mark at codesourcery.com
(650) 331-3385 x713



More information about the vsipl++ mailing list