On 16/05/2012, at 6:01 PM, phil swenson wrote:

> Dunno why I thought I needed "hasProperty"

Because doing project.profilingEnabled will throw an exception if that property 
hasn't been set. All of the example code uses hasProperty.

> But if I pass in -PprofilingEnabled=false, groovy would evaluate
> project.profilingEnabled as true because the property is a string set
> to "false"
> 
> In other words, in a conditional groovy evaluates "false" as true.
> 
> groovy> value = "false"
> groovy> if (value){
> groovy>  println "true"
> groovy> }
> true
> 
> I think I need to do this:
> if (project.properties.profilingEnabled == "true"){
>   runAEJvmArguments << "-agentlib:yjpagent"
> }

Correct, depending on how exact you need to be.

You could do:

if (Boolean.valueOf(project.properties.profilingEnabled)){
  runAEJvmArguments << "-agentlib:yjpagent"
}


-- 
Luke Daley
Principal Engineer, Gradleware 
http://gradleware.com


---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to