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
1=== modified file 'innobackupex'
2--- innobackupex 2012-12-17 03:36:27 +0000
3+++ innobackupex 2012-12-20 06:31:22 +0000
4@@ -551,11 +551,11 @@
5 my $empty_dir = shift;
6 my $is_directory_empty_comment = shift;
7 if (! -d $empty_dir) {
8- die "$is_directory_empty_comment directory does not exist!";
9+ die "$is_directory_empty_comment directory '$empty_dir' does not exist!";
10 }
11- opendir (my $dh, $empty_dir) or die "Not a directory";
12+ opendir (my $dh, $empty_dir) or die "$is_directory_empty_comment directory '$empty_dir': Not a directory";
13 if ( ! scalar( grep { $_ ne "." && $_ ne ".." && $_ ne "my.cnf" && $_ ne "master.info"} readdir($dh)) == 0) {
14- die "$is_directory_empty_comment directory is not empty!";
15+ die "$is_directory_empty_comment directory '$empty_dir' is not empty!";
16 }
17 closedir($dh);
18 }

Subscribers

People subscribed via source and target branches

to all changes: