BackupPC_sendEmail is missing a "use" directive to support encode.

Bug #430879 reported by Simon Déziel
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
backuppc (Ubuntu)
Fix Released
Undecided
Unassigned
Intrepid
Won't Fix
Undecided
Unassigned
Jaunty
Won't Fix
Undecided
Unassigned
Karmic
Fix Released
Undecided
Unassigned

Bug Description

The perl script that sends email notification is missing a "use" directive to support mail encoding.
Here are the steps to reproduce the bug :

su backuppc
$ /usr/share/backuppc/bin/BackupPC_sendEmail -u [removed for privacy]
Sending test email using /usr/sbin/sendmail -t -f backuppc01.norris
Undefined subroutine &main::encode called at /usr/share/backuppc/bin/BackupPC_sendEmail line 405.
sendmail: No recipients specified although -t option used

The bug is fixed upstream in BackupPC version 3.2.0 but this version is tagged as beta quality. Currently, Intrepid, Jaunty and Karmic all use BackupPC version 3.1.0 and are affected by this bug. I would suggest to just fix this issue and keep the stable release. The fix is a one liner and should cause no problem :

diff -Naur BackupPC_sendEmail-orig BackupPC_sendEmail
--- BackupPC_sendEmail-orig 2009-09-16 13:20:30.000000000 -0400
+++ BackupPC_sendEmail 2009-09-16 13:20:43.000000000 -0400
@@ -42,6 +42,7 @@
 use lib "__INSTALLDIR__/lib";
 use BackupPC::Lib;
 use BackupPC::FileZIO;
+use Encode;

 use Data::Dumper;
 use Getopt::Std;

Related branches

Revision history for this message
Simon Déziel (sdeziel) wrote :
Changed in backuppc (Ubuntu Karmic):
status: New → In Progress
Simon Déziel (sdeziel)
Changed in backuppc (Ubuntu Intrepid):
status: New → Confirmed
Changed in backuppc (Ubuntu Jaunty):
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package backuppc - 3.1.0-6ubuntu3

---------------
backuppc (3.1.0-6ubuntu3) karmic; urgency=low

  * Add dpatch system to package.
    - debian/control: Add build dependency on dpatch.
    - debian/rules: Add dpatch targets.
  * Add missing "use Encode" in sendEmail. (LP: #430879)
  * Fix encoding of special characters in french translation. (LP: #373327)

 -- Michael Jeanson <email address hidden> Wed, 16 Sep 2009 18:04:03 -0400

Changed in backuppc (Ubuntu Karmic):
status: In Progress → Fix Released
Changed in backuppc (Ubuntu Jaunty):
assignee: nobody → Michael Jeanson (mjeanson)
status: Confirmed → In Progress
Changed in backuppc (Ubuntu Intrepid):
assignee: nobody → Michael Jeanson (mjeanson)
status: Confirmed → In Progress
Changed in backuppc (Ubuntu Karmic):
assignee: nobody → Michael Jeanson (mjeanson)
Revision history for this message
Michael Jeanson (mjeanson) wrote :

Debdiff for Jaunty.

Revision history for this message
Michael Jeanson (mjeanson) wrote :

Debdiff for Intrepid.

Revision history for this message
Michael Jeanson (mjeanson) wrote :

Corrected version in Jaunty debdiff

Revision history for this message
Michael Jeanson (mjeanson) wrote :

Corrected version in Intrepid debdiff.

Revision history for this message
Chuck Short (zulcss) wrote :

Closing this SRU request based on the fact that intrepid has reached EOL.

Changed in backuppc (Ubuntu Intrepid):
status: In Progress → Won't Fix
Chuck Short (zulcss)
Changed in backuppc (Ubuntu):
assignee: Michael Jeanson (mjeanson) → nobody
Changed in backuppc (Ubuntu Intrepid):
assignee: Michael Jeanson (mjeanson) → nobody
Changed in backuppc (Ubuntu Jaunty):
assignee: Michael Jeanson (mjeanson) → nobody
Changed in backuppc (Ubuntu Karmic):
assignee: Michael Jeanson (mjeanson) → nobody
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Jaunty reached end-of-life on 23 October 2010, so this bug will not be fixed in that version of Ubuntu. It has been fixed in newer versions.

Changed in backuppc (Ubuntu Jaunty):
status: In Progress → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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