Christiano, Charles,

Your pull requests conflicted massively with each other :(

I 've done my best to apply the best of both worlds.
Due to the conflict, changes might be lost. Sorry if that has happened.
Contradicting conflicts have been written below.

Some notes on this approach and the current state:

• All common felix properties have been extracted to the droolsjbpm-parent pom. Individual modules should not define any of these specifically.
• If you want to add/remove/change any of those, change them in the parent pom only:
• https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/blob/master/pom.xml
  
• These are currently in the parent pom:
• <extensions>true</extensions>
• <excludeDependencies>true</excludeDependencies>
  
• <_removeheaders>Ignore-Package</_removeheaders>
• What does this mean? Christiano wants to remove this.
• @charles are you ok with removing this?
  
• <_nouses>true</_nouses>
• What does this mean? Christiano wants this.
  
• <_snapshot>${osgi-version-qualifier}</_snapshot>
• Christiano: "To make eclipse happy"
  
• <Bundle-Version>${parsedVersion.majorVersion}.${parsedVersion.minorVersion}.${parsedVersion.incrementalVersion}.${osgi.version.qualifier}</Bundle-Version>
• Christiano: "To make eclipse happy"
• There are not added (because less code is better maintainable):
  
• <DynamicImport-Package>*</> has been removed everywhere, as per Christiano's change
• @charles @christiano If you need it anyway, edit the droolsjbpm-parent pom and supply a pull request
  
• <Bundle-ActivationPolicy>lazy</> has not been added anywhere, as Charles didn't seem to need it
  
• @christiano @charles If you need it anyway, edit the droolsjbpm-parent pom and supply a pull request
• Generally, christiano's imports/export statements survived. (I found they to contain little or no dead imports/exports.)
• Some of Charles imports/export statement changes were added too.
• The original state of the imports/exports was mostly ignored as they were totally out-of-date.
• The singleton discussion is lost to me. As Charles is supplying the unit test in droolsjbpm, I believe he should make the call which modules should be singleton and which should not, taking Christiano's advice into consideration of course.
  
• Some modules currently have singleton=true, others don't. This seems to be the way you guys wanted: it's differs per module
• Pull Request to add/remove singleton as needed welcome
• Empty<Private-Package> have been removed everywhere
• <Require-Bundle> has been removed everywhere.
• This makes our build and release procedure far less complex (no more separate osgi.version property).
• Don't add it back pls: I strongly prefer it stays dead.
  

I've now spend a lot of time on drools OSGi, and I really need to focus on optaplanner issues.
Edson has agreed to look into future osgi related pull requests for drools.

_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev