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

Fix OSGi Bundle deploy by setting symbolicName from manifest #140

Merged
merged 3 commits into from
Feb 24, 2020

Conversation

tom-foyle
Copy link
Contributor

@tom-foyle tom-foyle commented Feb 13, 2020

Apply OSGi changes from cics-bundle-common 0.0.3. It took me a while to figure out what was going on with the two Artifact classes, so I tried to make it clearer. Changed test-no-transitive to use war instead of jar for simplicity.

Signed-off-by: Tom Foyle [email protected]

@tom-foyle tom-foyle force-pushed the osgi branch 7 times, most recently from 5345070 to 37e1357 Compare February 13, 2020 10:29
@ind1go
Copy link
Member

ind1go commented Feb 13, 2020

Have you been able to verify this against the deployment endpoint (or otherwise got it installed into CICS)? If it works then perhaps let's release common 0.0.3 to reduce the number of PRs we need here.

@tom-foyle
Copy link
Contributor Author

Have you been able to verify this against the deployment endpoint (or otherwise got it installed into CICS)? If it works then perhaps let's release common 0.0.3 to reduce the number of PRs we need here.

Haven't quite got it working yet. Getting there though...

@tom-foyle tom-foyle force-pushed the osgi branch 3 times, most recently from 262ab37 to 30aec94 Compare February 13, 2020 15:13
Apply OSGi changes from cics-bundle-common 0.0.3. Name moved higher in binding hierarchy. It took me a while to figure out what was going on with the two Artifact classes, so I tried to make it clearer. test-no-transitive converted to war just so I didn't have to bother with a manifest file.

Signed-off-by: Tom Foyle <[email protected]>
davenice
davenice previously approved these changes Feb 24, 2020
Signed-off-by: Tom Foyle <[email protected]>
@tom-foyle tom-foyle merged commit 3b49c8c into IBM:master Feb 24, 2020
@tom-foyle tom-foyle deleted the osgi branch February 24, 2020 15:11
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.

3 participants