Ubuntu
munin package

munin-node missing semi colon

Bug #247805 reported by Scott Zawalski
4
Affects Status Importance Assigned to Milestone
munin (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: munin
Version: 1.2.6-2ubuntu1
Ibex

There is a semi colon missing on line 206 of munin-node in the munin package.

diff -uNr munin-1.2.6.old/node/munin-node.in munin-1.2.6.new/node/munin-node.in
--- munin-1.2.6.old/node/munin-node.in 2008-07-12 00:37:46.000000000 -0700
+++ munin-1.2.6.new/node/munin-node.in 2008-07-12 00:40:36.000000000 -0700
@@ -203,7 +203,7 @@
 $ENV{'LC_ALL'}='C';

 MyPackage->run(conf_file => $conffile,
- pid_file => "@@STATEDIR@@/munin-node.pid")
+ pid_file => "@@STATEDIR@@/munin-node.pid");
 exit;

See original description

Related branches

lp:ubuntu/karmic/munin
Scott Zawalski (cowbud)
description: updated
description: updated
Nathan Handler (nhandler)
Changed in munin:
assignee: nobody → nhandler
status: New → In Progress
Revision history for this message
Nathan Handler (nhandler) wrote :

I have prepared a patch that fixes this bug and another bug in the munin package. The patch is attached to LP: #247806. Once that patch is uploaded, this bug will be closed.

Changed in munin:
assignee: nhandler → nobody
status: In Progress → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package munin - 1.2.6-4ubuntu1

---------------
munin (1.2.6-4ubuntu1) intrepid; urgency=low

  * Merge from debian unstable, remaining changes:
    + debian/munin-node.postinst:
      - Restart munin-node to load new plugins. (LP: #224428)
    + debian/control:
      - Suggest cron (LP: #220561)
    + Updated debian/patches/221-apt-ubuntu-changes.patch:
      - Make apt_all.in Ubuntu specific. Thanks to Soren Hansen.

   [Nathan Handler]
   - Fix typo in debian/munin-node.postinst (LP: #247806)
   - Fixed missing semi-colon in node/munin-node.in (LP: #247805)

munin (1.2.6-4) unstable; urgency=medium

  * Install munin-node gracefully (ie. inside a chroot, where port 4949 is
    bound to another munin-node already) : munin-node.postinst: don't exit
    postinst with error if just munin-node-configure fails. debian/rules: use
    dh_installinit with --error-handler=true. (Closes: #491130)
  * munin-node: if_ and if_err_ plugins: add msh|venet|veth to the regex of
    detected interfaces. (Closes: #489505)
  * munin-node: postfix_mailqueue plugin: use postconf to detect postfix's
    spooldir. (Closes: #383397)
  * munin-plugins-extra: include vserver plugins (Closes: #489737), thanks to
    Micah Anderson for maintaining them. Make them use /bin/bash instead of
    /bin/sh as they don't work with dash yet.
  * munin-node: depend on gawk. (Closes: #491559)
  * munin-node: add versioned dependency on lsb-base. (Closes: #469311)
  * munin-node-configure: don't treat empty suggests as errors.
    (Closes: #491475, #489502)
  * munin-node: sensors plugin: don't output errors on autoconf and suggest if
    no sensors binary is installed. (Closes: #491473)
  * munin-plugins-extra: fix typo in courier plugin. (Closes: #491463)

munin (1.2.6-3) unstable; urgency=low

  [ Holger Levsen ]
  * Add liblwp-useragent-determined-perl to suggests for the munin-node
    package as its needed for the apache_accesses plugin.

  [ Matthias Schmitz ]
  * Fix typo in munin-node postinst. (Closes: #489591)
  * Fixed POD typo in Plugin.pm (Closes: #488700)

 -- Chuck Short <email address hidden> Mon, 21 Jul 2008 12:50:17 +0100

Changed in munin:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.