On 11/11/2010, at 7:05 PM, [email protected] wrote:

> 
> +            String repo = System.getenv( "PROGRAMFILES" );

How portable is this? It might be better to have a configuration variable for 
the installation location, and set it to this if not specified?

> +            writer.write( addin.replaceAll( "\\$\\{localRepository\\}", 
> repo.replaceAll( "\\\\", "\\\\\\\\" ) ) );

Should this be a different variable name to be clearer? Is there any built-in 
technique that could be used to detect the location instead of the substitution?

- Brett

--
Brett Porter
[email protected]
http://brettporter.wordpress.com/




Reply via email to