Github user ahgittin commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/868#discussion_r147362460
--- Diff:
core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java
---
@@ -1492,7 +1503,7 @@ protected OsgiBundleInstallationResult
addItemsOsgi(String yaml, boolean forceUp
OsgiBundleInstallationResult result = null;
try {
- result = osgiManager.get().install(null, new
FileInputStream(bf), true, true, forceUpdate).get();
+ result = osgiManager.get().install(new
BasicManagedBundle(vn.getSymbolicName(), vn.getVersionString(), null), new
FileInputStream(bf), true, true, forceUpdate).get();
--- End diff --
agree re passing `OsgiManager` rather than it wrapped in `Maybe`, i've done
that now
---
