I could revert the deprecations (though they were specifically added at your request ;) ).
That said, those changed when the old IE versions were removed, and I didn't see the showcase start to emit those errors then. Even with the JDT upgrade, I still don't see them for the showcase: https://github.com/niloc132/gwt/runs/6124099314?check_suite_focus=true#step:5:3295 The error also does not appear to happen when running from ant, at least not in the showcase sample. At a glance I can't see anything explicitly disabling deprecation warnings in that code. The warning is coming from JsniReferenceResolver. The linked log above is emitting warnings, but clearly not one for the code you see, even though the showcase references those types. Showcase's CwRichText explicitly uses the RichText widget. However, both maven and gradle projects do correctly include the warnings, if you happen to include those gwt modules in your project. That does make the iteration time a bit longer to test, but it appears we can suppress the warnings on a per-usage basis. I'll propose a patch (and upload a new build to test) as soon as I'm able. On Friday, April 22, 2022 at 9:12:58 AM UTC-5 Thomas Broyer wrote: > I tried changing only the version (from com.google.gwt:gwt:2.8.2 to > com.google.gwt:2.10.0-new-groupid-2). > Looking at a dependency graph (./gradlew dependencies), I can see the > com.google.gwt dependencies properly "relocated" to their org.gwtproject > equivalent 🎉 > Java compilation passes without warning or error, same for JVM tests (I > don't have much that use GWT though), and even GWT tests! 🎉 (I only have a > couple of them though) > GWT compilation emits a few unexpected warnings though: > [WARN] Warnings in > 'jar:file:/home/redacted/.gradle/caches/modules-2/files-2.1/org.gwtproject/gwt-user/2.10.0-new-groupid-2/10fcf6c7213db2fc2c71a7731c582b6d8f9a4739/gwt-user-2.10.0-new-groupid-2.jar!/com/google/gwt/user/client/impl/DOMImplMozilla.java' > [WARN] Line 30: Referencing deprecated class > 'com.google.gwt.user.client.impl.DOMImplStandard' > [WARN] Line 30: Referencing deprecated class > 'com.google.gwt.user.client.impl.DOMImplStandard' > [WARN] Line 57: Referencing deprecated class > 'com.google.gwt.user.client.impl.DOMImplStandard' > [WARN] Warnings in > 'jar:file:/home/redacted/.gradle/caches/modules-2/files-2.1/org.gwtproject/gwt-user/2.10.0-new-groupid-2/10fcf6c7213db2fc2c71a7731c582b6d8f9a4739/gwt-user-2.10.0-new-groupid-2.jar!/com/google/gwt/user/client/ui/impl/RichTextAreaImplMozilla.java' > [WARN] Line 40: Referencing deprecated class > 'com.google.gwt.user.client.ui.impl.RichTextAreaImplStandard' > [WARN] Line 68: Referencing deprecated class > 'com.google.gwt.user.client.ui.impl.RichTextAreaImplStandard' > > and later fails with errors in a (very old) third-party dependency, > related to generics and wildcards (see below); but those errors are already > there with GWT 2.9.0 actually, and if I remove -failOnError then it > compiles OK! 🎉 > > Result of the compilation runs OK in Chrome (haven't tested others, and > haven't run extensive tests either, so I can't say there are no > regressions), and dev mode runs OK as well (after I remove -failOnError). > This is using Activities/Places, UiBinder, Editors and RequestFactory > extensively. Fwiw, it's also using elemental2-dom 1.0.0-RC1 (I probably > should have updated), and GIN 2.1.2 (with Guice 4.2.2). > > I also tried replacing the BOM dependency with org.gwtproject:gwt (without > changing the actual dependencies, so still using com.google.gwt:gwt-user), > works the same. > > I would prefer if the deprecation warnings above were suppressed before > the release, but it otherwise LGTM. At least the relocation works as > expected (from a Gradle point of view) > > Thanks a lot Colin for the hard work. > > Tracing compile failure path for type > 'org.waveprotocol.wave.model.supplement.WaveletBasedSupplement' > [ERROR] Errors in > 'jar:file:/home/redacted/.gradle/caches/modules-2/files-2.1/org.waveprotocol.waveinabox/waveinabox-model/redacted!/org/waveprotocol/wave/model/supplement/WaveletBasedSupplement.java' > [ERROR] Line 948: The method > createWaveletSeenVersion(DocumentEventRouter<? super E,E,?>) in the type > WaveletBasedSupplement is not applicable for the arguments > (DocumentEventRouter<N,capture#88-of ?,capture#90-of ? extends N>) > [ERROR] Line 943: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#85-of ?,capture#86-of ?>) > [ERROR] Line 953: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#91-of ?,capture#92-of ?>) > [ERROR] Line 918: The method createMuted(DocumentEventRouter<? > super E,E,?>) in the type WaveletBasedSupplement is not applicable for the > arguments (DocumentEventRouter<N,capture#70-of ?,capture#72-of ? extends N>) > [ERROR] Line 933: The method > createPendingNotification(DocumentEventRouter<? super E,E,?>) in the type > WaveletBasedSupplement is not applicable for the arguments > (DocumentEventRouter<N,capture#79-of ?,capture#81-of ? extends N>) > [ERROR] Line 938: The method > createWaveletReadState(DocumentEventRouter<? super E,E,?>, > ObservablePrimitiveSupplement.Listener) in the type WaveletBasedSupplement > is not applicable for the arguments (DocumentEventRouter<N,capture#82-of > ?,capture#84-of ? extends N>, ObservablePrimitiveSupplement.Listener) > [ERROR] Line 958: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#94-of ?,capture#95-of ?>) > [ERROR] Line 923: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#73-of ?,capture#74-of ?>) > [ERROR] Line 913: The method createFolders(DocumentEventRouter<? > super E,E,?>) in the type WaveletBasedSupplement is not applicable for the > arguments (DocumentEventRouter<N,capture#67-of ?,capture#69-of ? extends N>) > [ERROR] Line 913: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#67-of ?,capture#68-of ?>) > [ERROR] Line 938: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#82-of ?,capture#83-of ?>) > [ERROR] Line 933: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#79-of ?,capture#80-of ?>) > [ERROR] Line 943: The method > createWaveletCollapsedState(DocumentEventRouter<? super E,E,?>, > ObservablePrimitiveSupplement.Listener) in the type WaveletBasedSupplement > is not applicable for the arguments (DocumentEventRouter<N,capture#85-of > ?,capture#87-of ? extends N>, ObservablePrimitiveSupplement.Listener) > [ERROR] Line 948: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#88-of ?,capture#89-of ?>) > [ERROR] Line 923: The method createCleared(DocumentEventRouter<? > super E,E,?>) in the type WaveletBasedSupplement is not applicable for the > arguments (DocumentEventRouter<N,capture#73-of ?,capture#75-of ? extends N>) > [ERROR] Line 958: The method > createAbuseStore(DocumentEventRouter<? super E,E,?>) in the type > WaveletBasedSupplement is not applicable for the arguments > (DocumentEventRouter<N,capture#94-of ?,capture#96-of ? extends N>) > [ERROR] Line 953: The method > createWaveletNotifiedVersion(DocumentEventRouter<? super E,E,?>) in the > type WaveletBasedSupplement is not applicable for the arguments > (DocumentEventRouter<N,capture#91-of ?,capture#93-of ? extends N>) > [ERROR] Line 918: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#70-of ?,capture#71-of ?>) > [ERROR] Line 963: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#97-of ?,capture#98-of ?>) > [ERROR] Line 928: The method > create(ObservableMutableDocument<N,E,?>) in the type > DefaultDocumentEventRouter is not applicable for the arguments > (ObservableMutableDocument<N,capture#76-of ?,capture#77-of ?>) > [ERROR] Line 963: The method > createGadgetStatesDoc(DocumentEventRouter<? super E,E,?>, > ObservablePrimitiveSupplement.Listener) in the type WaveletBasedSupplement > is not applicable for the arguments (DocumentEventRouter<N,capture#97-of > ?,capture#99-of ? extends N>, ObservablePrimitiveSupplement.Listener) > [ERROR] Line 928: The method > createWaveletArchiveState(DocumentEventRouter<? super E,E,?>) in the type > WaveletBasedSupplement is not applicable for the arguments > (DocumentEventRouter<N,capture#76-of ?,capture#78-of ? extends N>) > > > On Fri, Apr 22, 2022 at 1:56 AM Colin Alworth <[email protected]> wrote: > >> TL;DR: If you have the capability to do so, now would be an excellent >> time to help us test GWT in anticipation of a release, especially around >> the groupId change we're going to make. >> >> -- >> >> We think that we're one merge away from being ready for a GWT 2.10 >> release, so I'm starting the release process a bit early, since this last >> commit involves changing GWT's groupId away from com.google and to >> org.gwtproject. >> >> To that end, I have a maven repo with the maven changes along wit all of >> the other changes in the GWT 2.10 series. The repo's URL is >> https://repo.vertispan.com/gwt-groupid-migration-test/, and the only GWT >> version that exists there is "2.10.0-new-groupid-2". >> >> Based on earlier work (such as in the >> https://groups.google.com/g/google-web-toolkit-contributors/c/L2RMqglOEXo/m/44BeZKeBCQAJ >> >> thread), this should allow projects to transition from com.google.gwt to >> org.gwtproject by adding the org.gwtproject:gwt:pom to their project, and >> then specify gwt-user etc, and automatically manage the version of gwt used >> by dependencies. It should also be possible to just use the old groupid for >> this release, but later releases will not have that option. >> >> Please note that *packages are not changing *as part of this transition, >> only groupIds. >> >> Rough release notes: >> * Updated htmlunit and jetty to more recent versions >> * Dropped support for IE8/9/10 >> * Dropped support for Java 7 >> * Support long classpaths by using CLASSPATH env vars to run child >> permutation workers >> * Many enhancements to emulation APIs and generated code >> >> Please reply to this thread or email me directly with any >> results/surprises/questions. >> >> -- >> You received this message because you are subscribed to the Google Groups >> "GWT Contributors" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected]. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/google-web-toolkit-contributors/dad1685b-9ad0-4a1a-88f4-dd0332d7b91dn%40googlegroups.com >> >> <https://groups.google.com/d/msgid/google-web-toolkit-contributors/dad1685b-9ad0-4a1a-88f4-dd0332d7b91dn%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> > > > -- > Thomas Broyer > /tɔ.ma.bʁwa.je/ <http://xn--nna.ma.xn--bwa-xxb.je/> > -- You received this message because you are subscribed to the Google Groups "GWT Contributors" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/69a483d4-5233-44e2-850d-e7abc2498ac9n%40googlegroups.com.
