Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump sshd-common version to 2.9.2 #29541

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

sberyozkin
Copy link
Member

No description provided.

@sberyozkin sberyozkin requested a review from gsmet November 28, 2022 17:21
@quarkus-bot quarkus-bot bot added the area/dependencies Pull requests that update a dependency file label Nov 28, 2022
Comment on lines +4742 to +4746
<dependency>
<groupId>org.apache.sshd</groupId>
<artifactId>sshd-common</artifactId>
<version>${sshd-common.version}</version>
</dependency>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some Camel Quarkus extensions depend on both sshd-core and sshd-common. Hence it would be nice if Quarkus could manage both.

Suggested change
<dependency>
<groupId>org.apache.sshd</groupId>
<artifactId>sshd-common</artifactId>
<version>${sshd-common.version}</version>
</dependency>
<dependency>
<groupId>org.apache.sshd</groupId>
<artifactId>sshd-core</artifactId>
<version>${sshd-common.version}</version>
</dependency>
<dependency>
<groupId>org.apache.sshd</groupId>
<artifactId>sshd-common</artifactId>
<version>${sshd-common.version}</version>
</dependency>

@aloubyansky aloubyansky merged commit 4800e61 into quarkusio:main Nov 30, 2022
@quarkus-bot quarkus-bot bot added this to the 2.15 - main milestone Nov 30, 2022
@gsmet
Copy link
Member

gsmet commented Nov 30, 2022

@aloubyansky a particular reason why you ignored @ppalaga 's comment? I think it makes sense. I can take care of it if it should be done.

@aloubyansky
Copy link
Member

Oh, sorry, I missed it.

@aloubyansky
Copy link
Member

Let me re-do that.

@aloubyansky
Copy link
Member

Would a separate PR be OK? #29569

@gsmet
Copy link
Member

gsmet commented Nov 30, 2022

Perfect thanks (and with all the backport labels!).

@gsmet gsmet modified the milestones: 2.15.0.CR1, 2.14.3.Final Dec 2, 2022
@sberyozkin sberyozkin deleted the sshd_common_2.9.2 branch December 7, 2022 13:10
@gsmet gsmet modified the milestones: 2.14.3.Final, 2.13.6.Final Dec 14, 2022
@gsmet gsmet modified the milestones: 2.13.6.Final, 2.7.7.Final Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants