Fix #401 - Support nested targets

[laeubi]

This adds support for nested targets.

[laeubi]

FYI @vogella

[github-actions]

Unit Test Results

142 files  142 suites   38m 30s ⏱️
223 tests 220 ✔️ 3 💤 0 ❌

Results for commit 5311241.

♻️ This comment has been updated with latest results.

[Bananeweizen]

I don't really understand most of the changes (due to my lacking knowledge, not because of the changes). Generally it looks okay to me.

[Bananeweizen]

I still haven't understood why this is defined again and again in many ITs (and with older versions than 2.6 in some cases). Shouldn't all tests point to the current development version for testing (by inheriting from the root parent of tycho)? What am I missing there?

[laeubi]

The tycho-version is overridden in the actual i-test run with the current build version so in that value is then irrelevant in this case as it is never used.
But if you are open the pom.xml in eclipse m2e of course complains that tycho-version is undefined and all kind of error markers are shown and code-completion does not work. Thus it is defined in (some but not all) of the itest mostly with the version that was recent at time of writing the test, it can always be adjusted to the latest version of course.

Just for convenience, it allows to right-click and Choose Run As > Maven install to test without a full itest run or specifying the version on the commandline.

[laeubi]

I don't really understand most of the changes (due to my lacking knowledge, not because of the changes). Generally it looks okay to me.

Thanks for your suggestions, I had to refactor out an xml-lib tycho previously used to pare target files to move the parsing code to common location so don't mind it looks a bit crowded.