Sounds ok for now. I agree security script module should be considered core.
I’ll try to think more about how to improve our dependency chain.
Thanks
-Vincent
On 15 Sep 2015 at 15:34:32, Marius Dumitru Florea (mariusdumitru.florea(a)xwiki.com) wrote:
Hi devs,
Checkout this commit
https://github.com/xwiki/xwiki-platform/commit/1805c5b1c2cf58d69760c1e0755c…
and it's comment. I added platform-security-script as mandatory top
level artifact in PackageMojo in order to fix some functional tests in
platform. I consider it to be a core module and ideally it should be
brought indirectly as a transitive dependency of the modules that are
using it. The problem is that I'm using it in the Velocity templates
from platform-web which doesn't declare any dependency ATM (they are
declared in enterprise-web). This is a quick fix for 7.2RC1. If you
have a better solution please speak.
Thanks,
Marius