Merge lp:~akopytov/percona-xtrabackup/bug1089375-2.1 into lp:percona-xtrabackup/2.1

Proposed by Alexey Kopytov
Status: Merged
Approved by: Alexey Kopytov
Approved revision: no longer in the source branch.
Merged at revision: 462
Proposed branch: lp:~akopytov/percona-xtrabackup/bug1089375-2.1
Merge into: lp:percona-xtrabackup/2.1
Diff against target: 18 lines (+3/-3)
1 file modified
innobackupex (+3/-3)
To merge this branch: bzr merge lp:~akopytov/percona-xtrabackup/bug1089375-2.1
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Approve
Review via email: mp+140822@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :

Approved (it's not my patch, but a contribution from Hartmut Holzgraefe <email address hidden> merged from a 2.0-based branch).

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'innobackupex'
--- innobackupex 2012-12-17 03:36:27 +0000
+++ innobackupex 2012-12-20 06:31:22 +0000
@@ -551,11 +551,11 @@
551 my $empty_dir = shift;551 my $empty_dir = shift;
552 my $is_directory_empty_comment = shift;552 my $is_directory_empty_comment = shift;
553 if (! -d $empty_dir) {553 if (! -d $empty_dir) {
554 die "$is_directory_empty_comment directory does not exist!";554 die "$is_directory_empty_comment directory '$empty_dir' does not exist!";
555 }555 }
556 opendir (my $dh, $empty_dir) or die "Not a directory";556 opendir (my $dh, $empty_dir) or die "$is_directory_empty_comment directory '$empty_dir': Not a directory";
557 if ( ! scalar( grep { $_ ne "." && $_ ne ".." && $_ ne "my.cnf" && $_ ne "master.info"} readdir($dh)) == 0) {557 if ( ! scalar( grep { $_ ne "." && $_ ne ".." && $_ ne "my.cnf" && $_ ne "master.info"} readdir($dh)) == 0) {
558 die "$is_directory_empty_comment directory is not empty!";558 die "$is_directory_empty_comment directory '$empty_dir' is not empty!";
559 }559 }
560 closedir($dh);560 closedir($dh);
561}561}

Subscribers

People subscribed via source and target branches

to all changes: