Skip to content

Conversation

@orzeh
Copy link
Contributor

@orzeh orzeh commented Feb 14, 2017

Manifest is generated by the Bundlor plugin according to the template defined in template.mf file.

Import package versions are mainly derived from properties defined in pom.xml.
Imports of Netty packages are marked as optional.

Bundle version is generated by the Build Helper plugin to be OSGi compliant when developing SNAPSHOTs.

This will fix #789


This change is Reviewable

@codecov-io
Copy link

codecov-io commented Feb 14, 2017

Codecov Report

Merging #797 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #797   +/-   ##
=======================================
  Coverage   71.64%   71.64%           
=======================================
  Files         306      306           
  Lines        6582     6582           
  Branches      483      483           
=======================================
  Hits         4716     4716           
  Misses       1580     1580           
  Partials      286      286
Impacted Files Coverage Δ
...ava/netty/handler/FramedResponseStreamHandler.java 89.65% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0a3ac8d...6dee405. Read the comment docs.

@KostyaSha
Copy link
Member

@marcuslinke WDYT?

Łukasz Warchał added 2 commits March 13, 2017 08:54
Manifest is generated by the Bundlor plugin according to the template defined in template.mf file. Import package versions are mainly derived from properties defined in pom.xml.
Imports of Netty packages are marked as optional.
@marcuslinke
Copy link
Contributor

As I've no experience with OSGI I can't say anything to this PR. But as no code was changed I think its OK to merge.

@KostyaSha KostyaSha added this to the 3.0.9 milestone Mar 28, 2017
@KostyaSha
Copy link
Member

Let's try..

@KostyaSha KostyaSha merged commit f9fdc39 into docker-java:master Mar 28, 2017
vuminhkh added a commit to alien4cloud/docker-java that referenced this pull request Apr 30, 2017
* master-docker-java: (25 commits)
  Add some coding styles.
  Allow an explicit Dockerfile location string to be specified to the build command (docker-java#825)
  Proposed fix for docker-java#830 (docker-java#832)
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release 3.0.9
  Fix test.
  Update CHANGELOG.md
  Responce may contains token.
  adding support for unless-stopped restart policy
  Generate OSGi compliant manifest (docker-java#797)
  exclude commons-logging since docker-java uses slf4j/logback
  Unable to deserialize status response from amazon ECR login
  Ensure FakeDockerServer is started on a random free port
  Fixes docker-java#807. NettyDockerCmdExecFactory ignores API version configuration
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release 3.0.8
  Update CHANGELOG.md
  Added healthcheck support in container inspect JSON result
  Use TLSv2 by default
  rerun failing tests count to 3
  ...

# Conflicts:
#	pom.xml
panuse pushed a commit to TuKangTech/docker-java that referenced this pull request Aug 20, 2017
* Generate OSGi compliant manifest

Manifest is generated by the Bundlor plugin according to the template defined in template.mf file. Import package versions are mainly derived from properties defined in pom.xml.
Imports of Netty packages are marked as optional.

* Export com.github.dockerjava.core package and subpackages
@KostyaSha
Copy link
Member

When i run test from idea it fails

Error:osgi: [docker-java] Bundlor failed: error generating manifest
Warning:osgi: [docker-java] The JAR is empty: The instructions for the JAR named compile-server did not cause any content to be included, this is likely wrong

Open idea, click run on some test.

@KostyaSha
Copy link
Member

Removed IDEA OSGI facet, now fine.

@KostyaSha
Copy link
Member

@orzeh now build fails eclipse repo returning weird errors

@orzeh
Copy link
Contributor Author

orzeh commented Feb 21, 2018

@KostyaSha maybe we should switch to BND I'll take a look on it.

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.

Does not work in OSGi enviroionment

4 participants