[GitHub] [cxf] rotty3000 commented on pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
rotty3000 commented on PR #946: URL: https://github.com/apache/cxf/pull/946#issuecomment-1123765114 @reta done! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscri

[GitHub] [cxf] reta commented on pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
reta commented on PR #946: URL: https://github.com/apache/cxf/pull/946#issuecomment-1123854355 > @reta done! Could see a few things: system.properties: ``` org.ops4j.pax.url.mvn.localRepository=/home/jenkins/workspace/CXF/CXF-JDK11-PR/.repository ``` /home/jenki

[GitHub] [cxf] reta commented on pull request #912: CXF-8371: Support Jakarta EE 9.1+

2022-05-11 Thread GitBox
reta commented on PR #912: URL: https://github.com/apache/cxf/pull/912#issuecomment-1123932696 > > > @reta I meant when can we merge the CXF-8371 to master . And we probably need plan some Alpha or Milestone release in the jakarta 4.x CXF and make CI work , then we don 't need to maintain t

[GitHub] [cxf] reta commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870500275 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

[GitHub] [cxf] rotty3000 commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
rotty3000 commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870506358 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

[GitHub] [cxf] reta commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870585002 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

[GitHub] [cxf] reta commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870590870 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

[GitHub] [cxf] rotty3000 commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
rotty3000 commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870649461 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

[GitHub] [cxf] reta commented on a diff in pull request #946: CXF-8700: Eliminate the `java.compiler` dependency from cxf-core

2022-05-11 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r870650350 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -62,7 +63,7 @@ public Option[] getConfig() {

Jakarta build hangs on test

2022-05-11 Thread Andriy Redko
Hey Jim, After this commit [2] the Jakarta builds are started to hang [2] (and eventually fail). I have not checked the reason but I could confirm that reverting these changes locally fixes the test, it does not hang anymore (with changes it hangs). Could you please check what is going on there

Re: Jakarta build hangs on test

2022-05-11 Thread Jim Ma
Hi Andriy, I'll have a look today and get back to you later. Thanks, Jim On Thu, May 12, 2022 at 8:26 AM Andriy Redko wrote: > Hey Jim, > > After this commit [2] the Jakarta builds are started to hang [2] (and > eventually fail). I have > not checked the reason but I could confirm that revertin