Skip to content
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

adjustments in import / export packages in pom.xml for osgi support #9

Merged
merged 1 commit into from
Apr 7, 2015

Conversation

ingoschaal
Copy link
Contributor

Signed-off-by: ingo schaal ingo.schaal@bosch-si.com
I fixed a few import / export packages instructions in the pom.xml of
leshan-core
leshan-server-cf
leshan-server-core
in order to use this artefacts as OSGi-bundles in a OSGi container.

Signed-off-by: ingo schaal <ingo.schaal@bosch-si.com>
@ingoschaal
Copy link
Contributor Author

Hello Kai,
yes, no problem.
In general in this commit, it's about that each bundle has to export the packages which are used in other bundles. And bundles which are using packages from other bundles has to import these.
This is done using the import / export instructions of the maven bundle plugin in pom.xml

sophokles73 pushed a commit that referenced this pull request Apr 7, 2015
adjustments in import / export packages in pom.xml for osgi support
@sophokles73 sophokles73 merged commit 90831fa into eclipse-leshan:master Apr 7, 2015
@jvermillard
Copy link
Contributor

@sophokles73 when merging, please try to avoid merge commits by merging with --ff-only option in the command line and maybe "git rebase master" on the PR branch

@sophokles73
Copy link
Contributor

@jvermillard Thanks for pointing this out, Julien. I'll keep that in mind on the next commit ...

@ghost ghost deleted the osgi-fixes branch September 30, 2015 10:31
mike-scott pushed a commit to mike-scott/leshan that referenced this pull request Jan 31, 2018
mike-scott pushed a commit to mike-scott/leshan that referenced this pull request Jan 31, 2018
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.

None yet

3 participants