If you're asking me, I would say no. Leaving it the way it is lessens the learning curve for people with Groovy and Java skills. Adding an API for something that already exists would just mean one more thing for the user to learn. It's just one guy's opinion though. Keep the API minimal.
-- Hamlet D'Arcy [email protected] > -----Original Message----- > From: Les Hazlewood [mailto:[email protected]] > Sent: Monday, December 29, 2008 12:12 PM > To: [email protected] > Subject: Re: [gradle-user] Gradle easy access to environment variables > > Beautiful, thanks :) I just didn't know if there was an > alias for System.getenv() already built in to Gradle. This > would be kinda nice to do out of the box: > > env['JAVA_HOME'] > > Do you think this is a good idea? Should I open a Jira issue > for an improvement if so? > > > On Mon, Dec 29, 2008 at 1:07 PM, D'Arcy, Hamlet B > <[email protected]> wrote: > > > I think you'd use the standard Groovy way, right? > > println System.getenv()['JAVA_HOME'] > > Should work within the script. It works in mine. > > -- > Hamlet D'Arcy > [email protected] > > > > -----Original Message----- > > From: Les Hazlewood [mailto:[email protected]] > > Sent: Monday, December 29, 2008 11:58 AM > > To: [email protected] > > Subject: [gradle-user] Gradle easy access to > environment variables > > > > It is not readily apparent to me in the User Guide (Section > > 4.3) how to quickly reference environment variables > in Gradle scripts. > > > > I do not want to create environment variables > > ORG_GRADLE_PROJECT_propName since I'm trying to support an > > existing build where environment variables are already set on > > many machines. > > > > So far, I found I can do the following: > > > > createTask("init", overwrite: true) { task -> > > ant.property(environment: "env"); > > println ant.antProject.properties["env.MY_ENV_VAR"] > > } > > > > However, that seems a little hacky. I was assuming that > > there was a nicer way to do this in Gradle automatically, for > > example, envProps.MY_ENV_VAR. > > > > Can someone please clarify? > > > > Thanks, > > > > Les > > > > > > > > --------------------------------------------------------------------- > To unsubscribe from this list, please visit: > > http://xircles.codehaus.org/manage_email > > > > > > --------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email
