| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
Review by phaller (not urgent).
|
|
|
|
|
|
| |
Partest support for sbt (with failed only and specific files only
options) Support for triggered execution
|
|
|
|
|
|
| |
Partest task will go into verbose debug mode when ant's debug flag is
set.
|
| |
|
|
|
|
|
|
|
|
| |
Changed the script runner mechanism to alchemize from AST atoms rather
than generating wrapper source, and fixed script position reporting.
This patch does not include a discussed change to mark some positions as
synthetic. Closes #3119, #3121. Review by milessabin.
|
|
|
|
|
| |
Removed more than 3400 svn '$Id' keywords and related junk.
|
|
|
|
|
|
|
|
| |
Remove the sourcepath prefix from the logs and only then do dos2unix
paths changing (otherwise #3054 fails). It is a bit ugly but it is only
done for buildmanager (paul was doing it anyway in his reverted partes).
That should really fix the windows build. No review.
|
| |
|
|
|
|
|
|
|
|
| |
changes necessary to plug it back in while preserving everything which
has happened since then in tests and such, but we should be the lookout
for overreversion. Review by phaller (but as a formality, I don't think
it requires direct review.)
|
|
|
|
|
|
| |
Rolled back a subset of partest to use StreamAppender and avoid
io.Process. No review.
|
|
|
|
|
|
|
| |
Reverted change that made partest fail with 'bin/javac: no such file'
when JAVA_HOME was not set (instead of using the javac on the PATH).
Reviewed by extempore, so no review.
|
|
|
|
|
|
|
|
|
| |
Since pickled data moved into annotations ShowPickled has been
confusedly scratching its head. Made tools/showPickled work again. In
the process created a simple interface for creating command line tools
for the (majority of) commands which would just like to specify a
handful of options. No review.
|
|
|
|
|
|
|
|
|
|
| |
Handled action items (1) and (4) of the phaller/cunei/rytz agenda
for a less regressive partest, to the extent possible without some
accompanying sbaz changes. Unnecessary sanity check which failed on sane
but non-trunk filesystem layouts removed. Files with custom command
sequences explicitly qualify the path to the filenames listed in the
.cmds file. Review by phaller.
|
|
|
|
|
|
| |
Degeneralized some incomplete generalization of the diff machinery.
Improved the summary output on failures. No review.
|
|
|
|
|
|
| |
partest --update-check to create the check file if none exists as well
as updating a pre-existing one. No review.
|
|
|
|
|
|
| |
way equals was being handled, and hammered it out. New ==/## tests which
covers all types and values. Review by odersky.
|
|
|
|
|
|
|
|
|
|
|
|
| |
temporary if considered clutter.
ant test.partest
ant test.partest-opt
They run some recently troublesome partest tests with a low timeout.
Logged some more exceptions where the compiler has been dying. Review by
phaller.
|
|
|
|
|
|
| |
Another attempt at avoiding partest hang ups when the compiler crashes.
Review by extempore.
|
|
|
|
|
|
|
|
| |
Correct oversight where neg tests didn't demand a checkfile. More trying
to get timeout/interrupted logic so we know when things die but also
don't impede interrupts. Folded precondition logic into test sequence
logic, it's plenty general to cover it. Closes #3282, review by phaller.
|
|
|
|
|
| |
Reverted r21500 until all test categories work again.
|
|
|
|
|
|
|
|
| |
Restored partest behavior to write log files with normalized paths. For
new tests the check file can be copied from the log file. This makes the
behavior of --show-diff exactly the same as diffing the log and check
file. Review by extempore.
|
| |
|
|
|
|
|
| |
Attempting to unblock partest by handling AssertionErrors. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
automatically update all the checkfiles with whatever output the tests
are producing. I suppose it goes without saying that this would be easy
to misuse, so let the record show that I accept no responsibility when
someone overwrites all the checkfiles in trunk with their shopping list
and checks it in.
Recommended usage: See some tests fail. Run
partest --failed --show-diff
Verify that all output differs in the expected fashion. Then run
partest --failed --update-check
Mission accomplished. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Introduces scala.tools.cmd providing command line tool infrastructure.
For a quick look at what can be done, see
scala.tools.cmd.Demo
For a more involved, potentially eye-straining look, see
scala.tools.partest.PartestSpec
To experience it through the eyes of Joe Partest User, run
test/partest
Review by community.
|
|
|
|
|
| |
partest it really will use quick as the build dir. No review.
|
|
|
|
|
|
|
|
|
| |
Fixed another partest feature I'd managed to break at the very last
minute. When a test is too slow finishing, there will be messages
identifying the test. It defaults to 90 seconds before the first warning
because I know some machines are slow, but it'd be nice if that was more
like 30. No review.
|
|
|
|
|
|
| |
A couple more bits of partest I discovered weren't doing their jobs.
Some of my classiest messages were going unheard! No review.
|
|
|
|
|
| |
Noticed a bug with test obj dirs not getting deleted. No review.
|
|
|
|
|
|
|
|
| |
Fix for the partest task to fail the build when a test fails, and fixes
for 2/3 of the quietly failing tests. I'm not quite sure what to do
about the view ones, it doesn't look like a simple rename is going to
cut it, so: review by odersky.
|
|
|
|
|
|
|
|
|
|
| |
it they can be my guest (reviewbot: review by community!) More
realistically: more than likely I have unwittingly altered or impaired
some piece of functionality used by someone somewhere. Please alert me
if this is the case and I will remedy it. I have to call it at this
point as the best interests of 2.8 cannot be served by me nursing this
patch along any further.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While working on partest discovered that CompilerCommand ignores half
its constructor arguments and a couple dozen places blithely pass it
those arguments as if they're being used. Then there were setups like
this:
class OfflineCompilerCommand(
arguments: List[String],
settings: Settings,
error: String => Unit,
interactive: Boolean)
extends CompilerCommand(arguments, new Settings(error), error, false)
Hey offline compiler command, why throw away the perfectly good settings
you were given? Ever heard 'reduce, reuse, recycle'? How did you ever
work... or do you? No review.
|
|
|
|
|
|
|
|
|
| |
method on Iterator called collect which I had to remove, because if
the method is overloaded it puts a bullet in the type inference, an
intolerable result for a function which takes a partial function as its
argument. I don't think there's much chance of confusion, but I put a
migration warning on collect just in case. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Scala signature is generated as an annotation (that is accessible
through Java reflection).
- compiler generates all pickled Scala signatures as annotations to class files.
- compiler can read class files with signature as annotations or old-style signatures as attributes.
- Scalap has also been updated to new signatures (contributed by Ilya Sergey: thanks a lot).
- FJBG updated to allow entering constant pool strings as byte arrays.
- ByteCodecs decode method returns the length of the decoded array.
Review by ilyas. Already mostly reviewed by odersky.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Went ahead and implemented classpaths as described in email to
scala-internals on the theory that at this point I must know what I'm
doing.
** PUBLIC SERVICE ANNOUNCEMENT **
If your code of whatever kind stopped working with this commit (most
likely the error is something like "object scala not found") you can get
it working again with either of:
passing -usejavacp on the command line
set system property "scala.usejavacp" to "true"
Either of these will alert scala that you want the java application
classpath to be utilized by scala as well.
Review by community.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Leveraged -Xmigration to burn off some warts which arose in the new
collections. Warnings put in place for behavioral changes, allowing the
following. 1) Buffers: create new collections on
++ and -- like all the other collections. 2) Maps: eliminated
never-shipped redundant method valuesIterable and supplied these
return types:
def keys: Iterable[A]
def keysIterator: Iterator[A]
def values: Iterable[B]
def valuesIterator: Iterator[B]
def keySet: Set[A]
I concluded that keys should return Iterable because keySet also exists
on Map, and is not solely in the province of Maps even if we wanted to
change it: it's defined on Sorted and also appears in some Sets. So it
seems sensible to have keySet return a Set and keys return the more
general type.
Closes #3089, #3145. Review by odersky.
|
|
|
|
|
|
| |
...and a line from partest I didn't notice the absence of which would
break the build. No review.
|
|
|
|
|
|
|
|
|
|
| |
tests you have placed in the test/debug directories - critically
for those of us stuck debugging ant, this lets one run a small
selection of tests by way of ant instead of the console runner.
(Sorry about the empty .gitignore files, but one of git's quirks is that
it won't acknowledge the existence of an empty directory.) No review.
|
|
|
|
|
|
| |
Changed partest ant task not to use reflection, instead using the path
with which scala was invoked. No review.
|
|
|
|
|
|
| |
More progress toward immutable Settings, and various cleanups
encountered along the way. No review.
|
|
|
|
|
|
| |
Still working my way through all the classpath manipulations in partest.
No review.
|
|
|
|
|
|
|
|
|
| |
test with the string "Manifest" in the source file, you may now do:
./partest --grep Manifest
No review.
|
|
|
|
|
|
| |
Removed the symlinks between scalacheck jars to satisfy windows. Tweaked
partest to accomodate. No review.
|
|
|
|
|
|
|
|
| |
what must be legacy scalatest.* properties to partest.*, boldly assuming
that the fact that partest is pretty much unusable outside of scalac
means there are no users outside of scalac who might be disrupted by
eliminating old property names. Review by community.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removed the partest restriction that individual files must be in the
same set. Haven't you always wanted to do this? Now you can. Review by
phaller.
% ./partest `ack --type=scala -l HashSet | head -6`
Testing individual files
testing: [...]/files/jvm/serialization.scala [ OK ]
testing: [...]/files/jvm/t1600.scala [ OK ]
Testing individual files
testing: [...]/files/pos/collections.scala [ OK ]
testing: [...]/files/pos/t2183.scala [ OK ]
Testing individual files
testing: [...]/files/run/bug1074.scala [ OK ]
testing: [...]/files/run/bug2512.scala [ OK ]
|
|
|
|
|
|
| |
More partest cleanups, and putting back a couple lines in build.xml
which were left a little too commented out. No review.
|
|
|
|
|
|
| |
Working on making the bootstrap process transparent and consistent. And
removed a bunch of what is now cruft in partest. No review.
|
|
|
|
|
|
| |
timings on all the individual tests and prints it sorted by glacialness.
Review by community.
|
|
|
|
|
|
|
|
|
| |
not being used. Updated partest with a --scalacheck option. Added
scalacheck tests to the ant build target. Still struggling with
ant/partest/classpaths so it's not on by default yet, but at least
./partest --scalacheck works. We... will... use... scalacheck. And we
will like it! No review.
|