Re: CXF 4.0.0 jakarta release

2022-11-30 Thread Jim Ma
Thanks Andriy and Dan ! On Mon, Nov 28, 2022 at 10:28 PM Andrey Redko wrote: > Works for me, thanks a lot Dan! > > Best Regards, > Andriy Redko > > On Mon, Nov 28, 2022, 8:45 AM Daniel Kulp wrote: > > > > > > > > > I don't think anything blocking is left, +1 to move forward! @Colm @Dan > >

Re: CXF 4.0.0 jakarta release

2022-11-30 Thread Colm O hEigeartaigh
It works for me as well. I'm coordinating with @Andriy Redko on two fixes that remain for 3.5.x/3.4.x and then we should be good to go. Colm. On Wed, Nov 30, 2022 at 10:10 AM Jim Ma wrote: > > Thanks Andriy and Dan ! > > On Mon, Nov 28, 2022 at 10:28 PM Andrey Redko wrote: > > > Works for me,

[GitHub] [cxf] aziubin commented on pull request #1033: CXF-8796: Replace REMOVED_MARKER with null

2022-11-30 Thread GitBox
aziubin commented on PR #1033: URL: https://github.com/apache/cxf/pull/1033#issuecomment-1331969825 @reta thanks. I have verified that last change and it work OK for me too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [cxf] reta commented on pull request #1033: CXF-8796: Replace REMOVED_MARKER with null

2022-11-30 Thread GitBox
reta commented on PR #1033: URL: https://github.com/apache/cxf/pull/1033#issuecomment-1332149388 @dkulp @coheigea guys I am wondering if you have any comments / concerns here -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cxf] reta commented on a diff in pull request #960: CXF-8706: CXF MTOM handler allow content injection

2022-11-30 Thread GitBox
reta commented on code in PR #960: URL: https://github.com/apache/cxf/pull/960#discussion_r1036059188 ## core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java: ## @@ -546,24 +550,46 @@ public static Attachment createMtomAttachmentFromDH( } public static Da

[GitHub] [cxf] coheigea commented on a diff in pull request #960: CXF-8706: CXF MTOM handler allow content injection

2022-11-30 Thread GitBox
coheigea commented on code in PR #960: URL: https://github.com/apache/cxf/pull/960#discussion_r1036106931 ## core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java: ## @@ -546,24 +550,46 @@ public static Attachment createMtomAttachmentFromDH( } public stati

[GitHub] [cxf] reta commented on a diff in pull request #960: CXF-8706: CXF MTOM handler allow content injection

2022-11-30 Thread GitBox
reta commented on code in PR #960: URL: https://github.com/apache/cxf/pull/960#discussion_r1036114943 ## core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java: ## @@ -546,24 +550,46 @@ public static Attachment createMtomAttachmentFromDH( } public static Da

[GitHub] [cxf] reta merged pull request #960: CXF-8706: CXF MTOM handler allow content injection

2022-11-30 Thread GitBox
reta merged PR #960: URL: https://github.com/apache/cxf/pull/960 -- 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 que

[GitHub] [cxf] reta opened a new pull request, #1037: CXF-8774: Migration path for jaxb2-basics (to hisrc-basicjaxb)

2022-11-30 Thread GitBox
reta opened a new pull request, #1037: URL: https://github.com/apache/cxf/pull/1037 Migration path for jaxb2-basics to hisrc-basicjaxb -- 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

[GitHub] [cxf] jimma commented on a diff in pull request #1037: CXF-8774: Migration path for jaxb2-basics (to hisrc-basicjaxb)

2022-11-30 Thread GitBox
jimma commented on code in PR #1037: URL: https://github.com/apache/cxf/pull/1037#discussion_r1036651630 ## tools/wadlto/jaxrs/pom.xml: ## @@ -59,6 +59,12 @@ junit test + +org.patrodyne.jvnet +hisrc-basicjaxb-pl