<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Hi Allon,<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I think I found some strange Checkstyle related problems on master.<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Engine build with (GWT compilation enabled) works OK.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Next, trying to start GWT debugger:<br><br>$ make gwt-debug DEBUG_MODULE=webadmin \<br> DEV_EXTRA_BUILD_FLAGS_GWT_DEFAULTS="-Dgwt.userAgent=gecko1_8,safari" \<br> DEV_EXTRA_BUILD_FLAGS="-Dgwt.logLevel=INFO -Dgwt.locale=en_US -Dgwt.compiler.localWorkers=1" \<br> DEV_BUILD_GWT_SUPER_DEV_MODE=1<br><br>maven-checkstyle-plugin:check execution fails on<br><br> frontend/webadmin/modules/gwt-extension/src/main/java/org/ovirt/engine/ui/uioverrides/org/ovirt/engine/core/compat/FormatterDotnet.java<br> can't parse argument number: (\\d)\\: For input string: "(\\d)\\"<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">the class isn't used, removed it, retry. Now it fails on:<br><br> frontend/webadmin/modules/gwt-extension/src/main/java/org/ovirt/engine/ui/uioverrides/org/slf4j/Logger.java<br> can't parse argument number: For input string: ""<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I guess it's a bug in our NON-NLS check? But why doesn't the problem occur during Engine build?<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I'm thinking about disabling Checkstyle for gwt-extension module, as it contains custom GWT RPC serializers and GWT class overrides, and maybe the file path src/main/java/org/ovirt/engine/ui/uioverrides/here/goes/actual/pkg is confusing the Checkstyle now.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks,<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Vojtech<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 22, 2017 at 10:33 PM, Allon Mureinik <span dir="ltr"><<a href="mailto:amureini@redhat.com" target="_blank">amureini@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-family:monospace,monospace">Hi all,</div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace">As per [1], I've just merged a series of patches that upgrades the oVirt engine to use the latest maven-checkstyle-plugin and checkstyle packages.</div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace">Please note that the newer checkstyle is a tad stricter than the old one we used to have (read: it contains several fixes for bugs where the old checkstyle was supposed to find issues but missed them).</div><div style="font-family:monospace,monospace">I also took the opportunity and added a couple of new checks that enforce rules we were de-facto adhering to anyway.</div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace">If any problems come up, please let me know.</div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace">-Your friendly neighborhood cleanup dude</div><div style="font-family:monospace,monospace"><br></div><div style="font-family:monospace,monospace">[1] <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1433408" target="_blank">https://bugzilla.redhat.<wbr>com/show_bug.cgi?id=1433408</a></div></div>
<br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div></div>