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

Dave Walker davewalker at ubuntu.com
Thu Nov 10 09:59:52 UTC 2011


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
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/874981

Title:
  Please merge/sync mcollective from debian wheezy

Status in “mcollective” package in Ubuntu:
  Fix Committed

Bug description:
  mcollective mcollective_1.2.1+dfsg is now in Debian wheezy.  It would
  be good to rebase the Ubuntu package based on the Debian package
  (which is already based on the Ubuntu package).

  However, a straight sync might not be appropriate as the delta is
  rather large.  We need to determine if we can sync, or merge.

  Thanks.

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



More information about the Ubuntu-sponsors mailing list