[Bug 874981] Re: Please merge/sync mcollective from debian wheezy

[Dave Walker]

Hi Marc,

Thanks for the debdiff.  Great work.

I did have a couple of comments.

I'm not sure I agree that the debian/control diff of the Description fields is ideal, is this required?  For example:
-Description: Marionette Collective clustering framework - server
+Description: build server orchestration or parallel job execution systems

debian/changelog:  Could do with a more descriptive entry, it doesn't
really describe what has changed or been dropped.

debian/mcollective-middleware.postinst:  Calls the service command directly, it's going to be safer to do
-	service activemq stop
-	service activemq start
+      invoke-rc.d activemq stop || :
+      invoke-rc.d activemq start

Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Server Team, which is subscribed to mcollective in Ubuntu.
https://bugs.launchpad.net/bugs/874981

Title:
  Please merge/sync mcollective from debian wheezy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/mcollective/+bug/874981/+subscriptions