Community
Participate
Working Groups
Currently the .tests suffix is supported for test plugins. It would be nice if this in configurable or does span to plugins ending with .test as well
regarding configuration of the pomless extension, there is a limitation in maven core, details see bug 492819 As long as it can't be configured, just rename your test plugins to end with ".tests" if you want to use pomless. test plugins should be dependency leaves i.e. no other plugin should depend on a test plugin so the rename should be trivial.
I agree that renaming is a viable solution but I have looked at the source and it is rather a minor extension to add .test as well for the time being as it is such a common case. For us changing the id would unfortunately amount to hundreds of plugins that would have to be renamed potentially breaking tests that were somehow relying on the id.
New Gerrit change created: https://git.eclipse.org/r/116842
I uploaded a patch which could solve the issue. https://git.eclipse.org/r/#/c/116842/ This currently checks the ending of a plugin id. It could, however, be extended to also check regex.
(In reply to Ralph Gerbig from comment #4) > I uploaded a patch which could solve the issue. > > https://git.eclipse.org/r/#/c/116842/ > > This currently checks the ending of a plugin id. It could, however, be > extended to also check regex. simply allowing ".test" in addition to ".tests" is much easier and pragmatic I think. Making it generally configurable boils down to maven core bug https://issues.apache.org/jira/browse/MNG-5897 and I would not like to work around this in Tycho code.
Allowing ".test" and ".tests" will declare plug-ins as test plug-ins which have not been test plug-ins before the change. Will this break existing builds?
(In reply to Ralph Gerbig from comment #6) > Allowing ".test" and ".tests" will declare plug-ins as test plug-ins which > have not been test plug-ins before the change. Will this break existing > builds? yes it's an incompatible change and should be announced in the release notes. the worst thing that could happen due to this change is that you have a plugin ending in .test and you don't want it to be of packaging type eclipse-test-plugin then you can either - use a different plugin suffix (neither .test nor .tests ) - use explicit pom.xml for this plugin (don't use pomless)
We are going to rename our plug-ins to ".tests". This is better than introducing a incompatible change.
.test and .tests will soon be supported, as mentioned this does not makes more problem as the current guess, but it should be extended in the future, maybe together with https://bugs.eclipse.org/bugs/show_bug.cgi?id=532575 we could even made this configurable by e.g. adding a property to builds.properties: tycho.testplugin = true/false that overrides default behaviour
New Gerrit change created: https://git.eclipse.org/r/148339
Additionally to support test/tests suffix one can define the property: tycho.pomless.testbundle=true/false in build.properties to override the default behavior.
Gerrit change https://git.eclipse.org/r/148339 was merged to [master]. Commit: http://git.eclipse.org/c/tycho/org.eclipse.tycho.extras.git/commit/?id=b44ff10de7ef5de6171740a11e036d189ec540f4
I think this deserve a note in the release notes.
done: https://wiki.eclipse.org/Tycho/Release_Notes/1.5#Pomless_Build
