That works for me. It will only increase the number of files if platforms have bugs, but it will make for easier maintenance.

We'll do the same in DRLVM too.

geir

Ivanov, Alexey A wrote:
Hi everyone,

Recently test exclude lists were removed from build.xml of the corresponding 
module, and there were added *six* files with excluded tests. These files 
contain almost the same list of files. The lists are identical for swing 
module. I found 2 differences for awt module (there are still about 50 files 
names listed in every of the exclude lists).

Why can't we use one 'exclude.all' file to exclude tests which fail on every 
platform? It's an obvious optimization.
I've tested the approach of using several exclude list files on build.xml of 
swing module. It works just fine.

Your comments?

Regards,
Alexey.


----- build.xml patch --------
Index: build.xml
===================================================================
--- build.xml   (revision 478584)
+++ build.xml   (working copy)
@@ -186,6 +186,7 @@

                 <fileset dir="${hy.swing.src.test.api}/java/common">
                     <include name="**/*Test*.java"/>
+                       <excludesfile name="./make/exclude.all" />
                     <excludesfile name="${exclude.file}" />
                 </fileset>
             </batchtest>
------------------------------


--
Alexey A. Ivanov
Intel Enterprise Solutions Software Division

Reply via email to