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

Simplify maven-bundle-plugin configuration #343

Merged
merged 1 commit into from
Mar 25, 2023

Conversation

HannesWell
Copy link
Contributor

@HannesWell HannesWell commented Mar 21, 2023

Instead of the 'maven-bundle-plugin:cleanVersions' goal, use BND-lib's range macro:
https://bnd.bndtools.org/macros/range.html
@tomaswolf maybe you are interested in this as well?

pom.xml Outdated Show resolved Hide resolved
@HannesWell HannesWell force-pushed the simplify_bundle-plugin_config branch 2 times, most recently from 462549a to e2b6caa Compare March 22, 2023 21:24
pom.xml Outdated Show resolved Hide resolved
Instead of the 'maven-bundle-plugin:cleanVersions' goal, use
BND-lib's range macro:
https://bnd.bndtools.org/macros/range.html

Additionally use angular brackets for BND-macros
(https://bnd.bndtools.org/chapters/850-macros.html) and make the
version-ranges for imported sshd packages more strict.
@HannesWell HannesWell force-pushed the simplify_bundle-plugin_config branch from e2b6caa to 9a384bb Compare March 22, 2023 22:32
@tomaswolf tomaswolf merged commit 9222146 into apache:master Mar 25, 2023
@HannesWell HannesWell deleted the simplify_bundle-plugin_config branch March 25, 2023 14:53
@HannesWell
Copy link
Contributor Author

Thank you @tomaswolf.

Can you tell me if there are already plans for the next release of sshd or can you give a rough estimate?

@tomaswolf
Copy link
Member

I had seen that question in the other PR; was getting around to it :-)

We have no release plan; we release whenever we feel it's needed and our release manager has the time to do so.

Luckily we don't have a lot in the pipeline currently, and no blocker bugs.

If we want to have that new release used in Eclipse 2023-06, we should target a new Apache MINA sshd release for Eclipse 2023-06 M3 (2023-05-26). I'll need some time to prepare a JGit version using the new release. (First, I run the JGit SSH tests as part of my approval of an Apache MINA sshd release, and second, we'd want the new JGit in that Eclipse to use the new Apache MINA sshd release anyway.) I don't think I'll get to that before April 15. So a reasonable timeframe for a new Apache MINA sshd release would be "first half of May". That would give me a little time to get the JGit changes in before M3.

I suggest we go to version 2.10; there are some API changes.

@gnodet, would you be available beginning of May to do a new release?

@HannesWell
Copy link
Contributor Author

HannesWell commented Mar 28, 2023

Luckily we don't have a lot in the pipeline currently, and no blocker bugs.

Glad to hear that.

If we want to have that new release used in Eclipse 2023-06, we should target a new Apache MINA sshd release for Eclipse 2023-06 M3 (2023-05-26). I'll need some time to prepare a JGit version using the new release. (First, I run the JGit SSH tests as part of my approval of an Apache MINA sshd release, and second, we'd want the new JGit in that Eclipse to use the new Apache MINA sshd release anyway.) I don't think I'll get to that before April 15. So a reasonable timeframe for a new Apache MINA sshd release would be "first half of May". That would give me a little time to get the JGit changes in before M3.

Thank you, it would be really great and highly appreciated to have the mentioned changes released in time for Eclipse 2023-06 and the previous version of sshd replaced by the new one, so that the Eclipse SimRel can complete the shift to sfl4j-2.
In case I can assist you with anything in this regard, please let me know.

@tomaswolf
Copy link
Member

I've created JGit issue 581770 to track the JGit update.

@HannesWell
Copy link
Contributor Author

I've created JGit issue 581770 to track the JGit update.

Great.

Would it be possible to deploy a snapshot version of the current state of sshd to any snapshot repo, e.g. the one of Apache? That would allow to test the current state in verification builds.

@HannesWell
Copy link
Contributor Author

@gnodet, would you be available beginning of May to do a new release?

@gnodet can you already say if a release is possible then?

@gnodet
Copy link
Contributor

gnodet commented Apr 18, 2023

I had seen that question in the other PR; was getting around to it :-)

We have no release plan; we release whenever we feel it's needed and our release manager has the time to do so.

Luckily we don't have a lot in the pipeline currently, and no blocker bugs.

If we want to have that new release used in Eclipse 2023-06, we should target a new Apache MINA sshd release for Eclipse 2023-06 M3 (2023-05-26). I'll need some time to prepare a JGit version using the new release. (First, I run the JGit SSH tests as part of my approval of an Apache MINA sshd release, and second, we'd want the new JGit in that Eclipse to use the new Apache MINA sshd release anyway.) I don't think I'll get to that before April 15. So a reasonable timeframe for a new Apache MINA sshd release would be "first half of May". That would give me a little time to get the JGit changes in before M3.

I suggest we go to version 2.10; there are some API changes.

@gnodet, would you be available beginning of May to do a new release?

I'll try to do that next week. Is there any remaining issue/PR to look at ?

@tomaswolf
Copy link
Member

Is there any remaining issue/PR to look at ?

@gnodet: none except your own #362, which I think should also go in that release. (But it has to pass the CI build first. Looks like there's still some work needed in the tests. Plus it should be mentioned in CHANGES.md when it goes in.)

Otherwise I think we're good to go.

There's also the closed issue #294, which perhaps wasn't quite fixed by PR #295 according to comments, but for which I have no reproducer. It's about a memory leak involving ThreadLocals in SftpFileSystemProvider, which might be your area of expertise :-).

@HannesWell
Copy link
Contributor Author

@gnodet, would you be available beginning of May to do a new release?

I'll try to do that next week. Is there any remaining issue/PR to look at ?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants