-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
P2 is not supporting the effective directive properly #473
Comments
Do we have a fix for this? or any workaround to apply? We are also having the same constraint and the maven builds are failing. Please advise. |
I fear now one has worked on this until now, so the only option is to provide that dependency. |
Same issue s [ERROR] Missing requirement: org.bndtools.headless.build.manager 7.0.0.202310060912 requires 'osgi.service; (objectClass=org.bndtools.headless.build.manager.api.HeadlessBuildPlugin)' but it could not be found |
It's really hard to find the providers of these service registrations, but the do exist. These are all in the platform's target platform: We don't know how you've defined your target platform. Try adding this: You can also get them from here: |
You are right Build is OK in updating pom as below I am trying to update with your update site |
See this comment here:
p2/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/BundlesAction.java
Line 392 in 57ecc5a
Currently P2 generates for example when using this manifest headers:
the following requirement:
<requiredProperties namespace='osgi.service' match='(objectClass=org.bndtools.headless.build.manager.api.HeadlessBuildPlugin)' min='0'/>
this has some drawbacks so one can't choose for the effectiveness in a P2 resolve operation should apply or not.
I therefore would suggest that we transform this into something like
filter="(osgi.effective.active=true)"
so one can choose in the resolve context if such a requirement is to be considered or not.The text was updated successfully, but these errors were encountered: