rsnapshot LVM backup fails at line 3720

Bug #1270686 reported by Jon Stumpf
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
rsnapshot (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

This error is received for each LVM listed in the rsnapshot.conf file:
        Use of uninitialized value $linux_lvm_oldpwd in concatenation (.) or string at /usr/bin/rsnapshot line 3720.
        chdir()
        Use of uninitialized value $linux_lvm_oldpwd in chdir at /usr/bin/rsnapshot line 3722.
        Use of chdir('') or chdir(undef) as chdir() is deprecated at /usr/bin/rsnapshot line 3722.

The error actually occurs at the assignment of $linux_lvm_oldpwd on line 3543.
        $linux_lvm_oldpwd = $ENV{PWD};

$ENV{PWD} returns nothing. This was not the case in December and this started failing for me sometime around Jan 10th, 2014. I am not a Perl programmer but the documentation I found states that the proper way to get the current working directory is to call cwd() and NOT use $ENV{PWD}.

I replaced the offending line with the following:
        $linux_lvm_oldpwd = cwd();

I no longer experience the issue.

- jss

Tags: patch
Revision history for this message
Jon Stumpf (jon-stumpf) wrote :
Revision history for this message
Jon Stumpf (jon-stumpf) wrote :

$ dpkg --status perl
Package: perl
Status: install ok installed
Priority: standard
Section: perl
Installed-Size: 17115
Maintainer: Ubuntu Developers <email address hidden>
Architecture: amd64
Version: 5.14.2-21build1
... lines deleted ...
$

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "rsnapshot.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in rsnapshot (Ubuntu):
status: New → Confirmed
Revision history for this message
Matthew Spraggs (matthew-spraggs) wrote :

Can confirm this bug, and that the proposed fix works.

Revision history for this message
Guillaume Delacour (gui-iroqwa) wrote :

This is fixed in Debian rsnapshot 1.4.0-1 package.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers