Skip to content

Please add more OSGI package exports to the Java manifest file#36

Closed
mattmarshall wants to merge 1 commit intoapache:masterfrom
mattmarshall:master
Closed

Please add more OSGI package exports to the Java manifest file#36
mattmarshall wants to merge 1 commit intoapache:masterfrom
mattmarshall:master

Conversation

@mattmarshall
Copy link
Copy Markdown

When trying to use the Thrift jar in an OSGI application, I get the following error:

org.osgi.framework.BundleException: Unresolved constraint in bundle ... : Unable to resolve 9.0: missing requirement [9.0] osgi.wiring.package; (osgi.wiring.package=org.apache.thrift.server)

Upon further inspection, I noticed that the jar's manifest file only exports the package org.apache.thrift, which is useful but arguably not sufficient in scope for most applications (I need access to TSimpleServer, etc.)

Please consider adding additional package exports so I don't have to re-wrap the compiled class files in a custom manifest. Thanks :)

@jfarrell
Copy link
Copy Markdown
Contributor

Thank you for issuing a pull request for your patch. Currently we do not accept pull requests from Github, please see http://thrift.apache.org/docs/HowToContribute/ for instructions on submitting your patch back to Apache Thrift. Thanks

@mattmarshall
Copy link
Copy Markdown
Author

Ok thank you, I've signed up for a JIRA account and see that the issue is being tracked in THRIFT-184.

jsgf pushed a commit to jsgf/thrift that referenced this pull request Jul 16, 2016
jsgf pushed a commit to jsgf/thrift that referenced this pull request Jul 16, 2016
jsgf pushed a commit to jsgf/thrift that referenced this pull request Jul 31, 2016
jsgf pushed a commit to jsgf/thrift that referenced this pull request Jul 31, 2016
keynslug pushed a commit to keynslug/thrift that referenced this pull request Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants