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.