Skip to content

WIP Upgrade bndtools to Eclipse 4.34 2024-12 #6536

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

Draft
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

chrisrueger
Copy link
Contributor

@chrisrueger chrisrueger commented Mar 24, 2025

Trying with 2024-12 and/or 2025-03

instead of handcrafted at https://bndtools.jfrog.io/bndtools/bnd-build/eclipse/4.25/index.xml.gz

This could make it easier to test and use newer Eclipse Versions, since we start having more and more bugs only happening in newer Eclipse versions.

Signed-off-by: Christoph Rueger <[email protected]>
Signed-off-by: Christoph Rueger <[email protected]>
note: version difference is on purpose (the asm:9.2 and the rest of asm.* 9.3). 9.2 is required by org.parboiled:parboiled-java:1.4.1 )

Signed-off-by: Christoph Rueger <[email protected]>
which is smaller, but requires a few other repositories (emf,jgit,m2e,wst) to be able to resolve the .bndrun files

Signed-off-by: Christoph Rueger <[email protected]>
Still some startup issues m2e, but bndtools starts
Signed-off-by: Christoph Rueger <[email protected]>
@chrisrueger chrisrueger force-pushed the upgrade-bndtools-to-higher-eclipse branch from 416d184 to b52bf27 Compare April 1, 2025 22:09
Note: for windows and linux I had to remove some requirements, since I guess the bundles do not exist anymore. I noticed via Repository Explorer they are disappearing around 2024-03

Signed-off-by: Christoph Rueger <[email protected]>

upgrade to Eclipse 4.35 (2025-03)

- resolve all .bndruns
- fixed some compilation problems because of stuff which was removed from Eclipse 4.35

Signed-off-by: Christoph Rueger <[email protected]>
@chrisrueger chrisrueger force-pushed the upgrade-bndtools-to-higher-eclipse branch from b52bf27 to b0f4f74 Compare April 1, 2025 22:28
@chrisrueger chrisrueger changed the title WIP Upgrade bndtools to higher eclipse WIP Upgrade bndtools to Eclipse 4.34 2024-12 Apr 1, 2025
it took me hours to find the reason for the error which was that the BSN changed from org.slf4j.api (in slf4j 1.7.3 which was part of Eclipse 4.25) to slf4j.api ( slf4j 1.7.25 which is in central.maven but not part of eclipse 4.34 anymore)

java.lang.IllegalArgumentException
        at org.eclipse.equinox.internal.provisional.p2.core.VersionParser.parseInto(VersionParser.java:71)

Signed-off-by: Christoph Rueger <[email protected]>
TODO still getting build errors  (Task :bndtools.core.test:testOSGi FAILED)

bndtools.core.test.editors.quickfix.BuildpathQuickFixProcessor_WithEmptyBuildpath_Test <<< ERROR: Cannot invoke "org.osgi.framework.BundleContext.getBundles()" because "bc" is null

May related to the jupiter and junit versions which I downgraded to make it resolve. May have to adjust again.

Signed-off-by: Christoph Rueger <[email protected]>
because Eclipse 4.34 requires Java 21 minimum

Signed-off-by: Christoph Rueger <[email protected]>
this finally fixed bndtools.core.test for me.
resolved all test .bndruns again

Signed-off-by: Christoph Rueger <[email protected]>
hopefully fixes build / test errors like java.lang.NoClassDefFoundError: org/bouncycastle/asn1/misc/MiscObjectIdentifiers

Signed-off-by: Christoph Rueger <[email protected]>
try to fix  Exception java.lang.NoClassDefFoundError: org/bouncycastle/asn1/misc/MiscObjectIdentifiers in biz.aQute.bndlib.comm.tests:test

Signed-off-by: Christoph Rueger <[email protected]>
to fix HttpTestServerTest

Signed-off-by: Christoph Rueger <[email protected]>
Signed-off-by: Christoph Rueger <[email protected]>
Signed-off-by: Christoph Rueger <[email protected]>
@chrisrueger
Copy link
Contributor Author

chrisrueger commented Apr 2, 2025

Current status:

  • I can launch the bnd eclipse instance with Eclipse 4.34 2024-12
  • but still tests failing... mainly because changed dependencies e.g. which were part of of Eclipse 2022-09, but are not in 2024-12 anymore.
image

Signed-off-by: Christoph Rueger <[email protected]>
versions, versions, versions, versions.... especially since Eclipse repo now has higher versions which causes trouble e.g. slf4j.api 2.0 , hamcrest.... so lots of places where we said version=latest did not work anymore, because in the past that has been 'our' version, but now latest is Eclipse version is is often incompatible

Signed-off-by: Christoph Rueger <[email protected]>
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

TODO still fail: aQute.tester.junit.platform.test.ActivatorJUnitPlatformTest

Signed-off-by: Christoph Rueger <[email protected]>
Signed-off-by: Christoph Rueger <[email protected]>
@chrisrueger
Copy link
Contributor Author

@kriegfrj You seem to be very familiar with all the JUnit Testcases stuff in bnd / bndtools.
Bndtools can be started with the Eclipse 2024-12 just fine, but I cannot sort out how to get the testcases running.
I think I got lost in all the JUnit versions since a lot of libs got updated or were removed in the newer Eclipse. So there is something messed up with different bundle (incompatible) versions loaded and classloading issues and what not.

Some tests are working, but I got there via trial and error and I always feel I'm not doing it right.

So I could need some support there.
Maybe you see something in the errors https://github.com/bndtools/bnd/actions/runs/14252255522/job/39947564487?pr=6536 and have some pointers or thoughts.

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.

1 participant