[GitHub] brooklyn-server pull request: Remove feature brookly-software-cm

2016-05-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/135 --- 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: Remove feature brookly-software-cm

2016-05-09 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/135#issuecomment-217902997 +1 for the quick fix. A proper solution would be to move `karaf/apacke-brooklyn` to `brooklyn-dist`. --- If your project is set up for it, you can reply to

[GitHub] brooklyn-server pull request: Remove feature brookly-software-cm

2016-05-09 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-server/pull/135#issuecomment-217895958 👍 LGTM. This was causing problems with a build, when tests were failing upon building with a clean Maven repository. Taking the cm feature out allows the

[GitHub] brooklyn-server pull request: Remove feature brookly-software-cm

2016-05-09 Thread duncangrant
GitHub user duncangrant opened a pull request: https://github.com/apache/brooklyn-server/pull/135 Remove feature brookly-software-cm You can merge this pull request into a Git repository by running: $ git pull https://github.com/duncangrant/brooklyn-server