Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Andriy Redko
Hi Dan, Yes, as Jim mentioned, most of our tests need JDK-17 to run (because of Spring 6), we also need JDK-17 to compile (same reason), but when Spring is not involved (it is optional by and large), JDK-11 is sufficient. We do have a number of samples (bundled with distribution) that run on

Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Jim Ma
Thanks for the update, Dan. Are these basic tests failed because Spring6 is built with java17 and it isn't compatible with java11 anymore ? On Mon, Dec 12, 2022 at 11:01 AM Daniel Kulp wrote: > > On Dec 11, 2022, at 9:36 PM, Jim Ma wrote: > > > > Hi Andriy, > > Thanks for the quick update.

Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Daniel Kulp
On Dec 11, 2022, at 9:36 PM, Jim Ma wrote: > > Hi Andriy, > Thanks for the quick update. Good to see this is running with the jenkins > pipeline and all tests are green. > Did it include all things for the CXF 4.0.0 release ? I didn’t do the 4.0.0 release as I kind of ran out of time. My

Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Jim Ma
Hi Andriy, Thanks for the quick update. Good to see this is running with the jenkins pipeline and all tests are green. Did it include all things for the CXF 4.0.0 release ? Thanks, Jim On Mon, Dec 12, 2022 at 10:06 AM Andriy Redko wrote: > Hi Jim, > > I disabled this build (and >

Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Andriy Redko
Hi Jim, I disabled this build (and https://ci-builds.apache.org/job/CXF/job/CXF-JDK19/ as well) because we run pipeline matrix [1] against both JDKs. These builds were quite useful when the main branch was unstable, but the pipeline should be sufficient now. Please let me know if you have

Re: CXF 4.0.0 jakarta release

2022-12-11 Thread Jim Ma
Did we already start the 4.0 release work ? I saw the CI build for CXF 4.0.0 is disabled now : https://ci-builds.apache.org/job/CXF/job/CXF-JDK17/ and last build result was 7 days ago. On Wed, Dec 7, 2022 at 8:46 PM Andriy Redko wrote: > Hi Colm, > > I still wait for review on [1], if you

[GitHub] [cxf] reta commented on a diff in pull request #1040: Fix org.apache.cxf.transport.jms.RequestResponseTest test cases which fail with 'Can't receive the Conduit Message in 10 seconds'

2022-12-11 Thread GitBox
reta commented on code in PR #1040: URL: https://github.com/apache/cxf/pull/1040#discussion_r1045304456 ## rt/transports/jms/src/test/java/org/apache/cxf/transport/jms/AbstractJMSTester.java: ## @@ -94,7 +94,7 @@ public static void startServices() throws Exception {

[GitHub] [cxf] reta opened a new pull request, #1040: Fix org.apache.cxf.transport.jms.RequestResponseTest test cases which fail with 'Can't receive the Conduit Message in 10 seconds'

2022-12-11 Thread GitBox
reta opened a new pull request, #1040: URL: https://github.com/apache/cxf/pull/1040 Fix `org.apache.cxf.transport.jms.RequestResponseTest` test cases which fail with `Can't receive the Conduit Message in 10 seconds` -- This is an automated message from the Apache Git Service. To respond