Hide Forgot
Description of problem: When multiple files contain declaration/import/global they are not properly merged and only one package's header is used. Version-Release number of selected component (if applicable): BRMS-5.2.0.CR1 How reproducible: Every time Steps to Reproduce: 1. Open attached project 2. Run main method 3. Check test-resources/repository.xml Actual results: <sv:property sv:name="drools:content" sv:type="String"><sv:value><![CDATA[package org.jboss.qa.guvnor declare Dog name : String end declare Dog name : String end ]]></sv:value></sv:property> Expected results: <sv:property sv:name="drools:content" sv:type="String"><sv:value><![CDATA[package org.jboss.qa.guvnor import java.util.List global String logPrefix declare Dog name : String end ]]></sv:value></sv:property> Additional info: Also please note that Dog declared fact is mentioned twice!
Created attachment 527223 [details] Simple reproducer
Bulk importer has had a major refactor for 5.4 (Mat's pull request), making it hard to backport patches. This looks like a new feature, so it should be implemented in the community version first.
Lukas, Are you ok with postponing these issues for 5.4 (see message above)? If ok, please remove jboss‑brms‑5.3.0+.
I can't "-", but I can "?". Anyway, Maven Bulk Importer is out of scope for the release, that much is clear.
Reassign to the guvnor team, now that I am working full time on planner. So these issues aren't forgotten.