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

[Andrew Haley]

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.