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:
- 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.