Re: [DISCUSS] - Retire CXF DOSGI?

2022-10-18 Thread Jean-Baptiste Onofré
Hi Colm

Yes, I think it makes sense. I don't think CXF DOSGi is heavily used.

Regards
JB

On Tue, Oct 18, 2022 at 10:49 PM Colm O hEigeartaigh
 wrote:
>
> Hi all,
>
> What do you think about moving CXF DOSGI to the attic? There is very
> little activity for many years now (last commit 2.5 years ago -
> https://github.com/apache/cxf-dosgi/commits/main). It's just a
> maintenance burden unless someone steps up to maintain it properly.
>
> Colm.


Re: [DISCUSS] - Retire CXF DOSGI?

2022-10-18 Thread Andriy Redko
Hi Colm,

I have never seen it used in the wild (at least, the projects I was/am 
involved with), I think it would make sense to retire it, especially taking
into account the OSGi vague plans to support Jakarta. +1 from my side to
move it to the attic. Thank you.

Best Regards,
Andriy Redko


COh> Hi all,

COh> What do you think about moving CXF DOSGI to the attic? There is very
COh> little activity for many years now (last commit 2.5 years ago -
COh> https://github.com/apache/cxf-dosgi/commits/main). It's just a
COh> maintenance burden unless someone steps up to maintain it properly.

COh> Colm.



[DISCUSS] - Retire CXF DOSGI?

2022-10-18 Thread Colm O hEigeartaigh
Hi all,

What do you think about moving CXF DOSGI to the attic? There is very
little activity for many years now (last commit 2.5 years ago -
https://github.com/apache/cxf-dosgi/commits/main). It's just a
maintenance burden unless someone steps up to maintain it properly.

Colm.


[GitHub] [cxf-dosgi] dependabot[bot] closed pull request #8: Bump jackson-databind from 2.10.1 to 2.12.6.1 in /distribution/multi-bundle

2022-10-18 Thread GitBox


dependabot[bot] closed pull request #8: Bump jackson-databind from 2.10.1 to 
2.12.6.1 in /distribution/multi-bundle
URL: https://github.com/apache/cxf-dosgi/pull/8


-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf-dosgi] dependabot[bot] commented on pull request #8: Bump jackson-databind from 2.10.1 to 2.12.6.1 in /distribution/multi-bundle

2022-10-18 Thread GitBox


dependabot[bot] commented on PR #8:
URL: https://github.com/apache/cxf-dosgi/pull/8#issuecomment-1282813249

   Superseded by #10.


-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf-dosgi] dependabot[bot] opened a new pull request, #10: Bump jackson-databind from 2.10.1 to 2.13.4.1 in /distribution/multi-bundle

2022-10-18 Thread GitBox


dependabot[bot] opened a new pull request, #10:
URL: https://github.com/apache/cxf-dosgi/pull/10

   Bumps [jackson-databind](https://github.com/FasterXML/jackson) from 2.10.1 
to 2.13.4.1.
   
   Commits
   
   See full diff in https://github.com/FasterXML/jackson/commits";>compare view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.fasterxml.jackson.core:jackson-databind&package-manager=maven&previous-version=2.10.1&new-version=2.13.4.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   - `@dependabot use these labels` will set the current labels as the default 
for future PRs for this repo and language
   - `@dependabot use these reviewers` will set the current reviewers as the 
default for future PRs for this repo and language
   - `@dependabot use these assignees` will set the current assignees as the 
default for future PRs for this repo and language
   - `@dependabot use this milestone` will set the current milestone as the 
default for future PRs for this repo and language
   
   You can disable automated security fix PRs for this repo from the [Security 
Alerts page](https://github.com/apache/cxf-dosgi/network/alerts).
   
   


-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf] jimma commented on a diff in pull request #1013: [CXF-8724]:Add codegen module back

2022-10-18 Thread GitBox


jimma commented on code in PR #1013:
URL: https://github.com/apache/cxf/pull/1013#discussion_r998142368


##
systests/wsdl_maven/codegen/src/it/cxf-4004/pom.xml:
##
@@ -37,12 +37,12 @@
 cxf-xjc-ts
 ${cxf.xjc-utils.version}
 
-
+

Review Comment:
   Done. Thanks for review @reta 



-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf] reta commented on a diff in pull request #1013: [CXF-8724]:Add codegen module back

2022-10-18 Thread GitBox


reta commented on code in PR #1013:
URL: https://github.com/apache/cxf/pull/1013#discussion_r998138208


##
systests/wsdl_maven/codegen/src/it/cxf-4004/pom.xml:
##
@@ -37,12 +37,12 @@
 cxf-xjc-ts
 ${cxf.xjc-utils.version}
 
-
+

Review Comment:
   @jimma just a comment, thanks, so we know why it is commented out



-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf] jimma commented on a diff in pull request #1013: [CXF-8724]:Add codegen module back

2022-10-18 Thread GitBox


jimma commented on code in PR #1013:
URL: https://github.com/apache/cxf/pull/1013#discussion_r998127232


##
systests/wsdl_maven/codegen/src/it/cxf-4004/pom.xml:
##
@@ -37,12 +37,12 @@
 cxf-xjc-ts
 ${cxf.xjc-utils.version}
 
-
+

Review Comment:
   @reta  Do you mean change the jira id in the commit message ? Or add some 
comment to https://issues.apache.org/jira/browse/CXF-8774  ?



-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [cxf] reta commented on a diff in pull request #1013: [CXF-8724]:Add codegen module back

2022-10-18 Thread GitBox


reta commented on code in PR #1013:
URL: https://github.com/apache/cxf/pull/1013#discussion_r998123614


##
systests/wsdl_maven/codegen/src/it/cxf-4004/pom.xml:
##
@@ -37,12 +37,12 @@
 cxf-xjc-ts
 ${cxf.xjc-utils.version}
 
-
+

Review Comment:
   thanks @jimma, LGTM, could you please add reference to 
https://issues.apache.org/jira/browse/CXF-8774 here, once the replacement is 
available, will be easy to trace the changes back.



-- 
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 queries about this service, please contact Infrastructure at:
us...@infra.apache.org