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
Bugs, which is subscribed to 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

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to