- better logging, temp dir, deletion...
- using build.config system property

Fix for failure building with useEclipseCompiles testing-drivers initially (or any module that only indirectly depended on jdt.core, which has no sources)
- better logging
- checking zip results - returns false positive for empty zips
- compile(..) API change for adopting classes

Also, copyright clearly wrong - file created last fall.

Fix for failure building with useEclipseCompiles testing-drivers initially (or any module that only indirectly depended on jdt.core, which has no sources)
- better logging
- checking zip results - returns false positive for empty zips
- compile(..) API change for adopting classes

delete, close, and isEmpty

fixing 2 bugs disclosed by running ajcTests.xml with -ajctaskCompiler: (1) compliance set incorrectly (-1.3 instead of 1.4) and (2) warn:... form was ignored.
These would break only when running from a Javac compiler adapter and using <compilerarg>

better logging

fixing 2 bugs disclosed by running ajcTests.xml with -ajctaskCompiler: (1) compliance set incorrectly (-1.3 instead of 1.4) and (2) warn:... form was ignored.
These would break only when running from a Javac compiler adapter and using <compilerarg>

updating harness documentation for new features

tests for new feature to select tests by title

suite files now constrained to be readable to avoid confusion with -ajctestTitle.. referents.

adding harness feature to select tests based on their title (description), reading from a comma-delimited list or a file containing PASS|FAIL lines (from -traceTestsMin), optionally restricted to FAIL.

updating build.jar to bootstrap changes to permit ant.jar in a testing module

no real changes, just better docs and an (untested) "all" target.

-noExit to avoid stopping a wrapping ant script

moved a command-line-driven test of classpath variants here

Better BuildModule tests following ant.jar updates.
These now build with assemble-all off and on, and
tests were updated to check by running classes.

To support the new testing ICommand wrapper for AjcTask, permit ant.jar in a testing module (ant is omitted from non-testing modules because it is normally deployed separately in ant/lib/ant.jar).
This does not affect regular module or product builds.

documenting -ajdeCompiler and -ajctaskCompiler selectors

added taskdefs to classpath to support AjcTaskCompileCommand

Tests for AjcTaskCompileCommmand adapter

ICommand adapter for Ant taskdef is simple for the non-incremental case and ugly otherwise.

supporting -ajctaskCompiler option with AjcTaskCompileCommand

awful downcast for AjcTaskCompileCommand - should be remedied after 1.1 with ICommand interface extension

updated implementation for -ajdeCompiler.

replaced reference to null ide properties adapter (defined only in testsrc)
and created logging proxy to verify API's used.
Unfortunately also formatted code, so it looks like there are more changes than there are.

updates mainly to support maven  but also to support new ICommand wrapper for AjcTask:
- findToolsJar now accepts versioned jars
- supporting a forkclasspath to permit user  to explicitly set
- upgrades for testing
  - so multiple iterations work,
    - not altering cmd in execute()
    - only fixing up outjar once
    - reset() complete
  - so incremental compile tests work,
    - passing quit through to main

- when forking on windows, use javaw rather than java
  if possible to avoid popping up a console window

more/better junit tests for AjcTask.java fork when possible

setting classpath for out-of-eclipse testing

fix for Bugzilla Bug 37758
   Weaving rt.jar results in stack overflow

test for Bugzilla Bug 37758
   Weaving rt.jar results in stack overflow

forcing target to be 1.1 in case someone builds using 1.4.

flagging with comments that 1.1 options are unimplemented

forcing target to be 1.1 in case someone builds using 1.4.

@testcase PR#37898 advice on handler join points should not throw unpermitted checked exceptions

added paragraph for call site implementation

changed jars produced by BcWeaverJarMaker to increase coverage of tests

better error message for trying declare parents on Object

expanded test coverage

fix for NPE revealed by BcWeaverJarMaker test code

added test for Bugzilla Bug 37304
   public static fields being ignored
this test is passing in 1.1rc2

fix and test Bugzilla Bug 37739
   Unexpected Xlint:unresolvableMember warning with withincode

- typo in top-level task
- incorrect pattern to copy all junit reports

- tense, xref usage corrections
- structure model caveat

More message-visiting methods, used only by harness now.

defensively supplying ISourceLocation.NO_FILE when there is no file in the result.
This is needed for file-based message matching to not fail when no file is specified,
and might provide a more sensible error message if the file is used.
This only replaces a NPE or invalid (empty) filename strings.

updating for installer temp dir fix when File.getTempFile() fails.

absolutizing temp dir path.
When File.getTempFile() fails, workaround temp dir
is relative, which messes up the zip resource copying.

build-testing-jars target for release testing setup

clean-jars target

Testing now seems to be working (tested incrementally)
CVS and entire-run still not verified - takes long on my machine.

variable renaming, verbosity - not much significant

- now supports both eclipse bin dirs (default)
  and ant-built test jars
- added incomplete test-all-at-once
  (currently runs each from its module directory)

raising issue of reporting source location for constructor execution as starting with the first initializer.  Possibly a known limitation.

- correcting message specification of existing tests
  using file and text comparison
- adding new command-line error tests (and keyword)

test data upgrades for adding new command-line error tests

two bad-input tests for the harness

- adding badInput attribute to compiler
- adding message kinds (abort, fail, info)
- message line no longer required

stricter test run comparisons

- new data (IMessageHolder rather than lists), same algorithms
- using medium-strenght message comparison that includes soft
  comparison of file and text
- unexpected info messages not marked as error
- expected FAIL+ messages suppressed to avoid false negative

- supporting badInput for all IAjcRun
- using IMessageHolder rather than lists

- enforcing -incremental => staging
- permitting bad input using badInput="true"
  (though still checking during staging)
- permitting -XserializableAspects
- supporting new AjcMessageHandler interfaces
- supporting parent badInput in XML

- supporting parent badInput in XML
- supporting new AjcMessageHandler interfaces

- supporting parent badInput in XML

- absolutizing tempdir path if necessary
  (bug in path calculations otherwise when
   using relative path to work around failure
   of File.getTempFile())

- revamped message comparators,
  mostly adding medium-level and consolidating
  common code

- new filter to remove arbitrary elements from diffs
  (used to not report unexpected info messages in diffs)

- using easier list view of messages

patch mentioned on the list for detecting errs in incremental command line

misspelled local invalid option in negative test

Fix for taskdef bug 37576
http://bugs.eclipse.org/bugs/show_bug.cgi?id=37576

No functionality change.
This just makes explicit that the browser reads user preferences
rather than relying on the default value supplied by the default constructor.

Disabling test reads from user preferences store.
This was discussed on the list before rc2.

Fixed relative links to examples. This time did it properly.

Fixed broken links to bug db and IBM sites.

Fixed links to Eclipse bugs database and to mailing lists.

Fixed link to Eclipse bugs database (was dev.eclipse.org/bugs now bugs.eclipse.org/bugs)

Fixed relative links to docs (was docs/index.html now doc/index.html)
and examples (was eaxamples now doc/examples).

Fixed link to Eclipse bugs database (was dev.eclipse.org/bugs now bugs.eclipse.org/bugs)

old dominates form produces an error, not a warning

link updates for progguide id filenames

fix for Bugzilla Bug 37325
   Weaver fails with NPE for very large source files

test for Bugzilla Bug 37325
   Weaver fails with NPE for very large source files

includeClassesDir

includeClassesDir

- platform-independent relative path
- compile-time checks for new attributes read

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