[GitHub] activemq-artemis pull request #1231: ARTEMIS-904 Remove cyclic dependencies ...

2017-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1231 --- 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] activemq-artemis pull request #1231: ARTEMIS-904 Remove cyclic dependencies ...

2017-04-27 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1231#discussion_r113715125 --- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/util/ServerUtil.java --- @@ -121,18 +121,6 @@ public static int

[GitHub] activemq-artemis pull request #1231: ARTEMIS-904 Remove cyclic dependencies ...

2017-04-27 Thread bennetelli
GitHub user bennetelli opened a pull request: https://github.com/apache/activemq-artemis/pull/1231 ARTEMIS-904 Remove cyclic dependencies from artemis-cli move classes and methods to their correct location to avoid cyclic dependencies between packages and classes. I also removed