+1 on revert I wrote a comment on the PR itself. Long story short, python usage inside a java context has always been a sort of stretched use case, and IMO it should be kept well isolated and not impact everyone else who is not interested, nor need, it.
Il giorno mar 1 apr 2025 alle ore 10:03 Jan Šťastný <jstastn...@apache.org> ha scritto: > Hello all, > > I think kogito-runtimes build is not broken after: > https://github.com/apache/incubator-kie-kogito-runtimes/pull/3863 > > The change in the linked PR moved the module quarkus/addons/python into the > default reactor, instead of the original 'full' profile. > Problem is that this module has dependencies on other python-related > modules inside kogito-runtimes which are NOT in the default reactor, but > still in 'full' profile. > > As a result the build is broken for users not running the 'full' profile > explicitly - in which case a stale snapshot of the dependencies which are > by default not part of the reactor is used. > > I've commented on the original PR too. > > Similar changes need to be well tested in affected build "modes", if we're > moving things from profile into default build, we need to test both builds > - and keep in mind, CI does NOT necessarily run builds in all the possible > setups (I think it runs full profile, so the problem was not apparent). > > @Francisco if you could either revert your change or fix the setup in a > reasonably short time. > > Regards > Jan >
