[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-04-05 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r58542079 --- Diff: karaf/init/src/main/resources/OSGI-INF/blueprint/blueprint.xml --- @@ -20,63 +20,63 @@ limitations under the License.

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-04-05 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r58541474 --- Diff: core/src/main/resources/catalog.bom --- @@ -0,0 +1,21 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/80 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-25 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/80#issuecomment-201273208 Finished review, comments in commit https://github.com/apache/brooklyn-server/pull/80/commits/f76f9eefe5c85c100e1d9d78873dbdf2e968150b. --- If your project is

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-25 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r57442486 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -0,0 +1,185 @@ +/* + * Licensed to the

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-25 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r57437435 --- Diff: core/src/main/resources/catalog.bom --- @@ -0,0 +1,21 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-25 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r57437307 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -0,0 +1,185 @@ +/* + * Licensed to the

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-25 Thread neykov
Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/80#discussion_r57437262 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -0,0 +1,185 @@ +/* + * Licensed to the

[GitHub] brooklyn-server pull request: Populate catalog from bundle bom

2016-03-24 Thread geomacy
GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-server/pull/80 Populate catalog from bundle bom This PR adds the ability for bundles to contain a 'catalog.bom' file specifying items that the bundle contains which should be added to the catalog.