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.

Reply via email to