commit 8b896546f94de12d2823958dff3e450d4efc1912 Author: Eric Ayers Date: Fri Jun 12 13:38:03 2015 -0400 Make sure arg to logRaw and log are only eval'ed once. Stu pointed this out in https://rbcommons.com/s/twitter/r/2315/ after I submitted the PR. The parameter `message` is passed by name, and so the argument might get evaluated in isFiltered() and in the logging function itself. Also got rid of a deprecation warning for `ConsoleLogger.stdout` Testing Done: Patched in locally. Bugs closed: 1655 Reviewed at https://rbcommons.com/s/twitter/r/2338/ src/scala/org/pantsbuild/zinc/Util.scala | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) commit 4aa6204d1eade46d050db219ef28103994eefcab Author: Eric Ayers Date: Wed Jun 3 15:34:08 2015 -0400 Add -log-filter option to the zinc task Add the -log-filter option to the zinc task to perform the same function as -Tnowarnregex for the Compiler class used under jmake Testing Done: Tested by putting my updated zinc jar into place in BUILD.tools and updating pants.ini with: ``` +[compile.zinc-java] +enabled: True + +args: [ + '-fork-java', + '-log-filter', '.*X500.*', + '-java-only', '-C-encoding', '-CUTF-8', + ] ``` Then compiling the simple hello world target as modified in https://github.com/pantsbuild/pants/issues/1612. The lines containing the string `X500` were omitted from the output. Bugs closed: 1612, 1631 Reviewed at https://rbcommons.com/s/twitter/r/2315/ src/scala/org/pantsbuild/zinc/Main.scala | 3 ++- src/scala/org/pantsbuild/zinc/Settings.scala | 7 +++++-- src/scala/org/pantsbuild/zinc/Util.scala | 18 +++++++++++++++--- 3 files changed, 22 insertions(+), 6 deletions(-)