Deleting temporary dir and config file, like harness -killTemp option (ignores -keepTemp)

including new attributes for round-trip testing

- test compile includeClassesDir attribute
- add pass|fail title prefix to run in one go

test source upgrades for new and old tests

Another strange incremental test, taken from ajcHarnessTests
and comparable with a known limitation case for injars,
this sometimes doesn't fail for me.  hmm.
(requires pending changes to sources)

new attribute for compile, includeClassesDir

moved incremental tests from failing (since they now pass)
cleaned out contents of failing that are now in ajcTests

changed suite spec from ajc-test+ to ajc-test* to allow empty suites
so that ajcTestsFailing can be empty without looking scary when run

handling Shadow.PreInitialization as a INITIALIZER_RELATION
rather than printing a warning message to System.err

this fixes the dirty output that was very visible when running the
harness with -emacssym enabled

fix to use cache for isPackage method (as indicated by previous !!! comment)
this fixes the two failing incremental test cases just added by Wes and now
moved to ajcTests as passing

Added persist member.

Backing out the change I made to add the output directory to the classpath.
This is wrong because we should resolve types wrt the cached classes in the weaver.
But if this change were left in, the test cases just added to ajcTestsFailing.xml pass.

two new incremental tests, each of which fails if run first but not otherwise (assuming the output directory is not added to the classpath).

comment documenting keyword "incremental-test"

Fixing Harness bug of adding the output directory to the classpath, which conceals a bug in the incremental compilation.
This change should have no effect on other tests, but I noticed two other failures during normal compiles caused by inability to resolve type names.

fixed testFileNodeFind that would always pass to be more strict
  and then fixed the file it was operating on to be correct

adding output dir/jar by default to classpath to permit resolving types in incremental compile
Might need to be more discriminating and/or to document.

- removed invalid test requiring -sourceroots in the argument list.  They might be in a config file.
- handling request  for fresh rebuild during incremental

updated with tests, etc.  Not working yet:
- junitModules.xml needs better setup with testable jars
- cvs unvalidated
- ajcTests unvalidated - see also ../build.xml script

incomplete update to support module jars rather than eclipse bin directories when running junit tests
the old way with bin directories still works fine.

permitting module builds without trimming, to have the option of using those jars rather than eclipse bin directories when running junit tests

changing after to after returning, with Jim's blessing.

fix for reported bug with invalid classpath entries (from the environment)
being reported as errors.  These are now just info messages.

initial draft of release build/test script.
CVS portion is untested.

fixed bug for inferring the kind of messages from the eclipse batch
compiler where == -1 was used instead of != -1

wrong handler reference

Minor updates for 1.1 rc2

moving to all-inclusive testing client for forking
(This might be wrong.. still verifying)

envp and snooping error stream in process controller

supporting a forked java run provisionally with system properties specifying fork, java home, java, and bootclasspath.
This should enable running the harness when targeting a VM that the harness or compiler can't run on,
but this does not detect failures correctly in some cases,
nor does it validate any of the system properties.

-keepTemp implementation: Suite run iterator calls clearCommand, which also calls validator.deleteTempFiles

-killTemp to delete temp directories when each test case completes

updated test-writing docs for AJDE fresh incremental builds.

new harness tests
- inc-compile "fresh" attribute and "same" tag
- compile "reuseCompiler" attribute

new attributes:
- compile reuseCompiler
- inc-compile fresh

new dependency on AJDE for ajde.CompileCommand implementation

remove !eclipse from aliases because it is the default
and it overrides specification of the AJDE compiler
(using -ajdeCompiler)

AJDE compiler, supports fresh incremental builds

- support for "same" tag, which makes no changes before incrementally compiling
- support  attribute for a fresh rebuild, currently implemented only by ...ajde.CompileCommand

API for Compiler to reuse command

- new reuseCompiler option (but see inc-compile fresh)
- support for -ajdeCompiler, like -eclipse and -ajc
- resequenced xml output - hmm

staging was not being written to XML.

I believe this file can/should be deleted as unused

comment, organizes imports (using 2, right?)

added harness tests (to JUnit test that is not included by default since it runs all the ajc tests)

creating a fresh AjState object whenever doing a batch build to make
sure we're doing a completely new build without confusion from previous
state

"organized imports" to eliminate compiler warnings

- special-casing -help to avoid printing usage twice
- guarding usage against loading eclipse message,
  which it did if called before some instance loaded the
  bundle (which should probably be loaded in a
  static initializer)

adding ant script to build AspectJ sources using AspectJ as a test
listed test in release-testing.txt
using spaces to avoid tab differences in release-testing.txt

keyboard mnemonics and accelerators

ensuring release-checklist.txt supercedes

  tests/readme-release-testing.txt

and adding some optional tests

updating library per binary-compatible change in feb to make build check pass

stateful config file chooser now starts with launch dir and
returns to directory of last-opened file, per convention.

added ant tasks to the devguide as a chapter (rather than refentry),
converted ajbrowser refentry to chapter (to get subsection TOC and better pagination)
updated ajbrowser slightly, and
updated index page to link accordingly.

NOTE: links to ajc changed
from devguide/rn01re01.html
   to devguide/ajc-ref.html

Leaving ajc as refentry is awkward but seems right since
it's a command-line tool.

corrected unterminated bold mark

added top-level xref target

implement binary-incompatible API for detecting whether any warnings exist, to enable/disable message display.

use binary-incompatible API for detecting whether any warnings exist, to enable/disable message display.

binary-incompatible API for detecting whether any warnings exist, to enable/disable message display.
(should move to MessageHandler-based model)

comment re: canonicalized path

NPE guard

comment re: List children supporting ListIterator().remove()

comment regarding canonicalizing path for input

fix test - NPE Guard, canonicalize file input

NPE guard - client of getSourceLocation only uses files - enforcing that.
(ajcTests.xml -emacssym passes)

addressing (Andy's) NPE - null source location is permitted for packages, so I checked all uses for NPE.
Also cleaned up the code a bit.  Should be semantics-preserving.

redoing root to be kind [java file || config file] --
 was java file kind for config files.

comment re: using shadow file rather than enclosing node.

minimal update to test script to test running, remove debugger tests.
Seems out of date; may try to enhance ajbrowser doc as basic ajde test script

out of the run loop - now direct from TopFrame to Ajde

- added run-in-other-vm button/menu,
  unscattered run and build invocations
- nb: key binding broken, might need run-other icon

support both same- and other-vm runs using new using new LangUtil API's for running processes

added LangUtil.ProcessController for external processes

- disabled logging by default
- added tests for FileUtil.Pipe

added FileUtil.Pipe

accept "." as base dir in resolving files

fix and better tests for
Bugzilla Bug 37152
   java.lang.VerifyError:

updated with requirement to use -incremental and sourceroots for inc-compile tests

passing incremental tests from testing-drivers/testdata/harness/incremental

converted to sourceroots, removed one test case

fix for
Bugzilla Bug 37152
   java.lang.VerifyError:

This fix needs a little clean-up and testing before closing out

more progress messages for ajde

added test that deprecation warnings are not shown unless asked for
modifying option setting to make that test pass

Updated expected bytecode to match change for  fix for issue of
Class.forName differences between IBM and SUN's JDKs

minimal run-in-same-vm feature for ajbrowser

- permit empty input to split
- split classpath
- check for array arguments

updated tests for [method|execution] binary join point detection.
added narrower knownLimitation test

removing 1.4 API

expanded coverage and fix for bug involving around advice with either
assert or <name>.class in the body

the fix for now is just to not inline such advice, but in the future these
should be transformed to inlinable constructs

fix for
Bugzilla Bug 37020
   wrong line for method execution join point
(but points to first executable line rather than method declaration)

and

Bugzilla Bug 37021
   source path wrong in declare warning on binary files

Expected fix for issue of Class.forName differences
between IBM and SUN's JDKs -- not yet tested on IBMs JDK to confirm

Added easy to understand test for issue of Class.forName differences
between IBM and SUN's JDKs

Moved a nit-picking part of SourceLocationWithExpr to its own test that
is listed as a known limitation.

cleaner, but no semantic changes

removed printlns

not using unimplemented buildStructure() api

no change, just nicer feedback

mainly changed setup to avoid reading any existing user properties, which was  causing build configuration tests to fail now that all options are significant

identify source of log message for convenience

- IMessage-based compiler messages
- better toString for logging message lists

- IMessage-based compiler messages
include the stack trace as tooltip text if there
is an associated exception