You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2007/01/30 03:21:25 UTC
svn commit: r501296 -
/geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy
Author: jdillon
Date: Mon Jan 29 18:21:24 2007
New Revision: 501296
URL: http://svn.apache.org/viewvc?view=rev&rev=501296
Log:
Show iter map in order, not sorted which is not right
Modified:
geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy
Modified: geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy
URL: http://svn.apache.org/viewvc/geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy?view=diff&rev=501296&r1=501295&r2=501296
==============================================================================
--- geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy (original)
+++ geronimo/sandbox/build-support/harness/trunk/lib/buildharness/commands/iteration.groovy Mon Jan 29 18:21:24 2007
@@ -52,8 +52,7 @@
props.load(file.newInputStream())
// Sanity check mapping, make sure we have a complete set
- int max = props.keySet().size()
- for (i in 1 .. max) {
+ for (i in 1 .. props.size()) {
assert props.containsKey(i as String) : "Missing mapping for iteration: ${i}"
}
@@ -105,8 +104,10 @@
this.mapping = loadMapping(mappingFile)
println 'Iteration Map:'
- mapping.keySet().sort().each {
- println " ${it}=" + mapping[it]
+
+ // Dump the map contents in iteration order
+ for (i in 1 .. mapping.size()) {
+ println " ${i}=" + mapping[i as String]
}
// Make sure its a valid key in the map