Recently, HEAD has had incidents where checked-in code would not even compile but was not detected by runtests.sh or tinderbox This seems to be because runtests.sh cannot presume that everyone has optional modules installed. Runtests needs an option (which tinderbox should use) that assures it that all the optional module dependencies are satisfied and it should check all of the modules.
Created attachment 96573 [details] [diff] [review] Patch to t/Support/Files.pm to selectively exclude files from tests Patch causes runtests.sh to test files with module dependencies if the module exists rather than unconditionally ignoring them.
Created attachment 96601 [details] [diff] [review] Revised patch - do it neatly
Comment on attachment 96601 [details] [diff] [review] Revised patch - do it neatly the patch shows signs of 2 and 4 space indentation, it seems the file also has 8 space indentation :-) at least make your work consistent (esp w/ modeline), but feel free to fix the entire file.
Checking in t/Support/Files.pm; /cvsroot/mozilla/webtools/bugzilla/t/Support/Files.pm,v <-- Files.pm new revision: 1.11; previous revision: 1.10 done