Extend demo bnd-workspace with subbundles

[chrisrueger]

see the bnd.bnd of the new tycho.demo.markdown bundle.
it uses the -sub: *.bnd instruction to create 2 jar files based on the other .bnd files (api.bnd and impl.bnd)

• https://bnd.discourse.group/t/generating-multiple-bundles-per-project/84
• https://bnd.discourse.group/t/sub-bundles-buildpath-vs-classpath/482
• https://bnd.bndtools.org/instructions/sub.html

this build currently fails (which is intentional) and we want to use that to fix tycho to make it compile.
It seems that tycho currently does not recognize the 2 (sub) jars so it cannot find them as dependencies for the HelloWorldService

[ERROR] Failed to execute goal org.eclipse.tycho:tycho-bnd-plugin:4.0.10:compile (compile) on project tycho.demo.impl: javac failed /bnd-workspace/tycho.demo.impl/src/main/java/org/eclipse/tycho/demo/impl/HelloWorldService.java:18: error: package tycho.demo.utils.markdown.api does not exist
[ERROR] import tycho.demo.utils.markdown.api.MarkdownRenderer;
[ERROR]                                     ^
[ERROR] /bnd-workspace/tycho.demo.impl/src/main/java/org/eclipse/tycho/demo/impl/HelloWorldService.java:24: error: cannot find symbol
[ERROR]     private MarkdownRenderer markdown;
[ERROR]             ^
[ERROR]   symbol:   class MarkdownRenderer
[ERROR]   location: class HelloWorldService
[ERROR] 2 errors

Expected result

The build should have produced two jars in tycho.demo.markdown

• tycho.demo.markdown.api.jar
• tycho.demo.markdown.impl.jar

And those should be recognized as dependency as required by the -buildpath of the tycho.demo.impl/bnd.bnd, so that tycho.demo.impl compiles successfully.

[laeubi]

Thanks for enhancing the example, we need some license headers on the java code and it would be good to squash all into one commit (it is the easier to later cherry pick/backport the test).

[chrisrueger]

I can confirm that #4666 fixes the build of demo/bnd-workspace in this PR

Note to self:

1. Build tycho from PR Add support for building sub-bundle with tycho-bnd-extension #4666 with: mvn clean install -T1C -DskipTests
2. switch to this PR and build it with

cd demo/bnd-workspace
mvn clean install -Dtycho-version=5.0.0-SNAPSHOT

[INFO] Reactor Summary for bnd-parent 1.0.0-SNAPSHOT:
[INFO] 
[INFO] bnd-parent ......................................... SUCCESS [  0.059 s]
[INFO] tycho.demo.api ..................................... SUCCESS [  0.987 s]
[INFO] tycho.demo.markdown ................................ SUCCESS [  0.908 s]
[INFO] tycho.demo.impl .................................... SUCCESS [  0.769 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  3.616 s
[INFO] Finished at: 2025-01-31T20:25:59+01:00
[INFO] ------------------------------------------------------------------------

[laeubi]

This now has failed the DemoTest#testTychoBndWorkspaceDemo in the CI as expected 👍

@chrisrueger can you please:

1. rebase the PR on current main
2. shorten / cleanup the commit message a bit, it currently contains some intermediate comments and we don't need the detailed failure description I think

If that is done the test should be green again and we can already merge this!

[chrisrueger]

@laeubi will do.
Question: Which version should I put in .mvn/maven.config
-Dtycho-version=4.0.11
or
-Dtycho-version=5.0.0-SNAPSHOT

?

[laeubi]

Use the current released version: -Dtycho-version=4.0.10 in the build this will be overridden by 5.0.0-SNAPSHOT or 4.0.11-SNAPSHOT (depending on branch).

[eclipse-tycho-bot]

💚 All backports created successfully

Status	Branch	Result
✅	tycho-4.0.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

[laeubi]

@chrisrueger thanks for enhancing the example!