[Bug 994357] Re: Bad sed syntax in mcollective-plugins-facts-facter.postrm

[Launchpad Bug Tracker]

** Branch linked: lp:ubuntu/mcollective-plugins

-- 
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/994357

Title:
  Bad sed syntax in mcollective-plugins-facts-facter.postrm

Status in “mcollective-plugins” package in Ubuntu:
  Fix Released
Status in “mcollective-plugins” source package in Precise:
  Triaged

Bug description:
  The postrm script for mcollective-plugins-facts-facter is broken.
  Looks like it was never tested, since it's got plainly broken sed
  syntax on line 8:

  sed -i -e "s/^factsource.*/factsource = yaml\nplugin.yaml =
  /etc/mcollective/facts.yaml/" /etc/mcollective/server.cfg

  No escaping has been done on the forward slashes in the file path for
  '/etc/mcollective/facts.yaml'.  The package cannot be removed without
  manually fixing this sed command; removal attempts result in an error:

  Removing mcollective-plugins-facts-facter ...
  sed: -e expression #1, char 52: unknown option to `s'
  dpkg: error processing mcollective-plugins-facts-facter (--remove):
   subprocess installed post-removal script returned error exit status 1

  ############## SRU ###############
  [Impact]
  Package mcollective-plugins-facts-facter will never uninstall properly

  [Development Fix]
  Fixed on package mcollective-plugins_0.0.0~git20120507.df2fa81-0ubuntu1 for quantal

  [Stable Fix]
  Modified sed line, attached debdiff and checked that the sed line works properly now

  [Test Case]
  Install mcollective-plugins-facts-facter package, then proceed to uninstall it

  [Regression Potential]
  Very unlikely

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