Fwd: hg: jdk7/build/langtools: 6829575: 100028: Debug information is incomplete or missing

[Kelly O'Hair]

Yes, I think you are right. Sorry, I think I sent you on this dead goose
chase. :^(

If this is in the jaxp and jaxws repositories, we need to fix those too.

-kto

Andrew Haley wrote:
> Kelly O'Hair wrote:
>> My fault. I reviewed this and it didn't dawn on me what this did
>> until your email.
>>
>> Yes, the default production builds with javac should be source,lines.
>> Sorry, I had native code on the brain and was thinking 'debug off' :^(
> 
> How about if I do
> 
> ifeq ($(DEBUG_CLASSFILES), true)
>    ANT_OPTIONS += -Djavac.debug=true
>    ANT_OPTIONS += -Djavac.debuglevel=source,lines,vars
> endif
> 
> instead of
> 
> ifeq ($(DEBUG_CLASSFILES), true)
>   ANT_OPTIONS += -Djavac.debug=true
>   ANT_OPTIONS += -Djavac.debuglevel=source,lines,vars
>  else
>   ANT_OPTIONS += -Djavac.debug=false
>   ANT_OPTIONS += -Djavac.debuglevel=
> endif
> 
> I think that answers Jonathan's objection...
> 
> Andrew.