Hi, Thanks, Volker for backporting this to JDK11 and thanks Andrew for reviewing it.
> On 30/12/2019 20:18, Volker Simonis wrote: > > Hi, > > I'd like to downport the support for Visual Studio Code project > > creation to 11u. I think we will have to support 11u for quite some > > years and it makes sense to have as good as possible tool support in > > 11u as well. > > I just came across this proposal in the process of backporting > JDK-8232748: "Build static versions of certain JDK libraries". The only > hunk of that changeset which isn't a simple context difference is that > which applies to JdkNativeCompilation.gmk and the FindLib & > FindStaticLib rules it introduces. > > I was going to just drop this hunk, as I didn't see any reason to > backport JDK-8210459, but then I saw your existing RFR referenced on the > bug. If JDK-8210459 is to be backported, I thus need this in place > before posting JDK-8232748 for review. Thanks for waiting, Andrew. I'll approve JDK-8210459 now in order to get it pushed to unblock your work on JDK-8210459. > > Finally, I've added an additional fix to this last change, whch fixes > > the make tests (i.e. "make test-make"). These tests are currently > > broken in jdk11u. They have been broken by JDK-8212028 (which has > > already been downported to jdk11u) and fixed in jdk 12 as part of > > "8210958: Rename "make run-test" to "make test"" (which hasn't been > > downported yet and probably won't be donwported at all). Because the > > fix is trivial (that's why it was done as part of 8210958 without an > > extra bug ID) and because I wanted to run "make test-make" in jdk11u > > as well to verify my changes, I've decided to downport this fix as > > part of 8223678: > > > > diff -r 8599975f5b33 test/make/TestMakeBase.gmk > > --- a/test/make/TestMakeBase.gmk Tue Feb 12 08:40:43 2019 +0100 > > +++ b/test/make/TestMakeBase.gmk Mon Dec 23 22:10:46 2019 +0100 > > KWBASE := > APA=banan;GURKA=tomat;COUNT=1%202%203%204%205;SUM=1+2+3+4+5 > ;MANY_WORDS=I > > have the best words. > > > > $(eval $(call ParseKeywordVariable, KWBASE, \ > > - KEYWORDS := APA GURKA SUM, \ > > + SINGLE_KEYWORDS := APA GURKA SUM, \ > > STRING_KEYWORDS := COUNT MANY_WORDS, \ > > )) > > > > @@ -364,7 +376,7 @@ > > KWBASE_WEIRD := ;;APA=banan;;;GURKA=apelsin;APA=skansen;; > > > > $(eval $(call ParseKeywordVariable, KWBASE_WEIRD, \ > > - KEYWORDS := APA GURKA SUM, \ > > + SINGLE_KEYWORDS := APA GURKA SUM, \ > > STRING_KEYWORDS := COUNT, \ > > )) > > > > I agree it doesn't make sense to backport JDK-8210958, as that's a major > change to the test infrastructure. However, I think this warrants its > own 11u bug rather than being smuggled in under JDK-8223678. Otherwise, > you're just mirroring the same problem that was created by mixing this > in with JDK-8210958. An independent bug can describe the breakage and > link to the original issue that caused it. Hm, I think Volker's way of adding this trivial correction to the backport of 8223678 is okay. But, of course a separate bug could also be a good way to document this fix. Maybe we need a tiebreaker here? 😊 So I'll approve the series of backports now, except for JDK-8223678. Cheers Christoph
