Document (better) referencedRepositoryMode in TargetPlatformConfigura…

[LorenzoBettini]

…tionMojo.java

This should hopefully improve the documentation of referencedRepositoryMode (it was not clear to me which values to specify)

[laeubi]

@LorenzoBettini as you seem to using that option already it is currently off by default, but maybe you thing we should make it on by default for Tycho 5?

[LorenzoBettini]

@laeubi since PDE automatically uses referenced repositories I think it would be a good idea to enable it by default, if you ask me :)

In fact, my target platform was resolving correctly in PDE, while it was failing in the Tycho build (I'm using Epsilon, whose update site refers to older Orbit sites for its dependencies, I noted this problem when updating my target platform to newer Eclipse/Orbit). Luckily, I've discovered this new cool feature of Tycho, enabled it, and everything works like a charm.

[github-actions]

💚 All backports created successfully

Status	Branch	Result
✅	tycho-4.0.x

Questions ?

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

[laeubi]

@laeubi since PDE automatically uses referenced repositories I think it would be a good idea to enable it by default, if you ask me :)

Do you want to open a PR + a note to the release notes that this is now the default and reference how to restore old behavior for Tycho 5 here?

[LorenzoBettini]

@laeubi OK. So, if I understand correctly in that PR I should make include the default (I seem to understand it is enough to make that element the first one in the enum), change the Javadoc accordingly and update the release notes for 5 accordingly (of course, I guess also a few IT will have to be updated). Is that correct?

[laeubi]

I should make include the default (I seem to understand it is enough to make that element the first one in the enum)

Correct, you might to verify with a local build that the default is taken into account

change the Javadoc accordingly and update the release notes for 5 accordingly

Yep

I guess also a few IT will have to be updated

I hope not, but some might fail and then should simply use the old "default" value then, I think we have no real IT to actually test the feature itself so if you like to add one it would of course also be great :-)

Is that correct?

Would really be appreciated 👍