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

2022-05-12 Thread GitBox
ffang commented on PR #912: URL: https://github.com/apache/cxf/pull/912#issuecomment-1125593877 > > @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

[GitHub] [cxf] reta merged pull request #948: CXF-8703: PAX Maven URL does not take local Maven repository into account

2022-05-12 Thread GitBox
reta merged PR #948: URL: https://github.com/apache/cxf/pull/948 -- 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 unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org For

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

2022-05-12 Thread GitBox
rotty3000 commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r871681147 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -59,10 +60,10 @@ 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-12 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r871674254 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -59,10 +60,10 @@ public Option[] getConfig() {

Re: Jakarta build hangs on test

2022-05-12 Thread Andrey Redko
Thanks a lot, Jim! Best Regards, Andriy Redko On Thu, May 12, 2022, 3:53 AM Jim Ma wrote: > It looks this hang issue is fixed : > https://ci-builds.apache.org/job/CXF/job/pipeline/job/CXF-8371/41/#showFailuresLink > > Regards, > Jim > > On Thu, May 12, 2022 at 9:19 AM Jim Ma wrote: > >>

[GitHub] [cxf] reta opened a new pull request, #948: CXF-8703: PAX Maven URL does not take local Maven repository into account

2022-05-12 Thread GitBox
reta opened a new pull request, #948: URL: https://github.com/apache/cxf/pull/948 PAX Maven URL does not take local Maven repository into account, see please [1]. It seems like the system property is not taken into account. [1] https://github.com/apache/cxf/pull/946/files --

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

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

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

2022-05-12 Thread GitBox
rotty3000 commented on PR #946: URL: https://github.com/apache/cxf/pull/946#issuecomment-1125107767 Oh, I see all checks have now passed :) -- 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

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

2022-05-12 Thread GitBox
rotty3000 commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r871485236 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -59,10 +60,10 @@ 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-12 Thread GitBox
rotty3000 commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r871318632 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -59,10 +60,10 @@ 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-12 Thread GitBox
reta commented on code in PR #946: URL: https://github.com/apache/cxf/pull/946#discussion_r871290492 ## services/sts/systests/sts-itests/src/test/java/org/apache/cxf/systest/sts/itests/BasicSTSIntegrationTest.java: ## @@ -59,10 +60,10 @@ public Option[] getConfig() {

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

2022-05-12 Thread GitBox
jimma commented on PR #912: URL: https://github.com/apache/cxf/pull/912#issuecomment-1124695153 > > Done, https://ci-builds.apache.org/job/CXF/job/pipeline/job/CXF-8371/38/consoleFull > Thanks for the quick change. The test result is out

Re: Jakarta build hangs on test

2022-05-12 Thread Jim Ma
It looks this hang issue is fixed : https://ci-builds.apache.org/job/CXF/job/pipeline/job/CXF-8371/41/#showFailuresLink Regards, Jim On Thu, May 12, 2022 at 9:19 AM Jim Ma wrote: > Hi Andriy, > I'll have a look today and get back to you later. > > Thanks, > Jim > > On Thu, May 12, 2022 at 8:26