Pax Construct

Add provided dependencies as required libraries in the Eclipse classpath to better support testing

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 0.5.0, 0.6.0, 0.6.1
  • Fix Version/s: 0.6.2
  • Component/s: maven-pax-plugin
  • Labels:
    None

Description

Currently only explicit test dependencies are added as required libraries in the Eclipse .classpath file, because the idea is that Eclipse/PDE will add the relevant bundles in the "plugin dependencies" classpath container. However, in some cases (like Spring powered beans) the bundle code, and by definition its manifest, may not include any mention of spring classes - therefore Eclipse/PDE won't import them and the test code (which may refer to them, but doesn't contribute to the manifest) could then break.

So lets just add provided and test dependencies to the required libraries container, as this will really help in development (less breakage when the manifest gets zapped, etc.) and it shouldn't affect the runtime use, as we'll mark them as non-exported libraries...

Activity

Hide
mcculls added a comment -

done - will probably release 0.6.2 today if testing goes ok...

Show
mcculls added a comment - done - will probably release 0.6.2 today if testing goes ok...

People

Vote (0)
Watch (0)

Dates

  • Due:
    Created:
    Updated:
    Resolved: