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

- IMessage-based compiler messages

- using IMessage-based API's for displaying messages
- if a source line message contains a throwable
and the user double-clicks, this displays the error dialog
that permits them to copy the trace.  That should mean that
multiple exceptions can be in the task list.

support shift-key to rebuild-all in incremental mode

slight rewording of error dialog, and correct URL.

- making default filename static (access by tests)
- closing streams to permit better test cleanup

incremental flag support

static utility for detecting shift key down - should move

binary-incompatible change to IMessage-based API (using ISourceLocation anyway)

using new IMessage-based API for sourceline tasks

- incremental support
- buildStructure test API now builds everything
- removed unused code from pre-IMessage days

- supporting incremental
- handling new fast-fail config errors
- better error display, no messages lost

Implemented new options merging:
- can specify any local config file options

- project properties and build options
  now can contain any options and
  override local values only if not set

- have method to fixup the options for IDE's
  to enforce their invariants, but this needs
  to delegate to a new adapted API.

- added incremental checkbox
- broadened size and meaning of "non-standard compiler options" to "other compiler options"

added flag for incremental mode

binary-incompatible API change to add methods for building fresh when in incremental mode

Making genBuildConfig static so it can be re-used from AJDE.

added support for adopting global configuration values (without overwriting local values)

refactored batch and incremental into one method to make it easier to see the differences and do a fix.

test case can't expect usage message any more

fixed two tests to expect addition abort message for usage,
and commented out one test because it does not add the file correctly (to a matching task)

Usage is now emitted as an abort message,  so AjcTask now ignores usage when determining whether to throw a BuildException after getting an abort message.  This fixes 3 tests.

error messages for undefined ${module.name}

fix test title

With the restructure of the org.eclipse.jdt.core project, it no
longer has a bin dir. Updated the classpath to include
jdtcore-for-aspectj.jar instead.

track internal API changes

using Kind.isSameOrLessThan(..) rather than direct compare for inferring streams

semantics-preserving - still only warn if aspectjrt.jar not found

boolean hasSources() to validate after parsing

error-handling cleaned up
- usage printed once by caller
- any arg mistake is an error
    (i.e., not ignoring/continuing)
    (does not include list entries like classpath)
- refactored commands to share common error-handling code