Hi Brett,

----- "Brett Porter" <[email protected]> wrote:

> 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?


Revised in r1036410


> 
> > +            writer.write( addin.replaceAll(
> "\\$\\{localRepository\\}", repo.replaceAll( "\\\\", "\\\\\\\\" ) )
> );
> 
> Should this be a different variable name to be clearer? Is there any


Revised in r1036410


> built-in technique that could be used to detect the location instead
> of the substitution?


I'm kinda confused on what built-in technique you meant here. Do you have any 
suggestion?

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


Thanks,

Reply via email to