Merge lp:~percona-dev/percona-xtrabackup/docs-updates-bug-783453 into lp:percona-xtrabackup/2.0

Proposed by Rodrigo Gadea
Status: Rejected
Rejected by: Stewart Smith
Proposed branch: lp:~percona-dev/percona-xtrabackup/docs-updates-bug-783453
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 57 lines (+12/-7)
2 files modified
innobackupex (+8/-3)
patches/tar4ibd_libtar-1.2.11.patch (+4/-4)
To merge this branch: bzr merge lp:~percona-dev/percona-xtrabackup/docs-updates-bug-783453
Reviewer Review Type Date Requested Status
Stewart Smith (community) Needs Fixing
Percona developers Pending
Review via email: mp+62221@code.launchpad.net

Description of the change

Removed -g and -z from Usuage | Bug # 783453

To post a comment you must log in.
Revision history for this message
Stewart Smith (stewart) wrote :

This will need to be either resubmitted as a branch with just this fix or along with the fixed libtar patch

review: Needs Fixing
Revision history for this message
Stewart Smith (stewart) wrote :

these fixes have been merged in other branches.

Unmerged revisions

256. By Rodrigo Gadea

Remove options -g and -z from Usage

255. By Rodrigo Gadea

Add File::Temp to handle tmpfiles instead of POSIX "tmpnam"

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'innobackupex'
--- innobackupex 2011-05-20 17:23:07 +0000
+++ innobackupex 2011-05-24 23:21:03 +0000
@@ -12,7 +12,7 @@
12use Pod::Usage qw(pod2usage);12use Pod::Usage qw(pod2usage);
13use POSIX "strftime";13use POSIX "strftime";
14use POSIX ":sys_wait_h";14use POSIX ":sys_wait_h";
15use POSIX "tmpnam";15use File::Temp; # use POSIX "tmpnam";
16use FileHandle;16use FileHandle;
17use File::Basename;17use File::Basename;
18use English qw(-no_match_vars);18use English qw(-no_match_vars);
@@ -148,10 +148,10 @@
148my $mysql_server_version = '';148my $mysql_server_version = '';
149149
150# name of the file where stderr of mysql process is directed150# name of the file where stderr of mysql process is directed
151my $mysql_stderr = 'stderr';151my $mysql_stderr = File::Temp->new();
152152
153# name of the file where stdout of mysql process is directed153# name of the file where stdout of mysql process is directed
154my $mysql_stdout = 'stdout';154my $mysql_stdout = File::Temp->new();
155155
156# name of the file where binlog position info is written156# name of the file where binlog position info is written
157my $binlog_info;157my $binlog_info;
@@ -214,6 +214,11 @@
214print_version();214print_version();
215215
216# initialize global variables and perform some checks216# initialize global variables and perform some checks
217if (!$option_tmpdir){
218 my $tmpdir = File::Temp->newdir();
219 $option_tmpdir = $tmpdir->dirname;
220}
221
217if ($option_copy_back) {222if ($option_copy_back) {
218 if ( $option_ibbackup_binary eq 'autodetect' ) {223 if ( $option_ibbackup_binary eq 'autodetect' ) {
219 $option_ibbackup_binary = 'xtrabackup';224 $option_ibbackup_binary = 'xtrabackup';
220225
=== modified file 'patches/tar4ibd_libtar-1.2.11.patch'
--- patches/tar4ibd_libtar-1.2.11.patch 2011-03-15 21:58:13 +0000
+++ patches/tar4ibd_libtar-1.2.11.patch 2011-05-24 23:21:03 +0000
@@ -531,11 +531,11 @@
531 void531 void
532 usage()532 usage()
533 {533 {
534- printf("Usage: %s [-C rootdir] [-g] [-z] -x|-t filename.tar\n",534- printf("Usage: %s [-C rootdir] -x|-t filename.tar\n",
535+ printf("Usage: %s [-C rootdir] [-g] [-z] -x|-t < filename.tar\n",535+ printf("Usage: %s [-C rootdir] -x|-t < filename.tar\n",
536 progname);536 progname);
537- printf(" %s [-C rootdir] [-g] [-z] -c filename.tar ...\n",537- printf(" %s [-C rootdir] -c filename.tar ...\n",
538+ printf(" %s [-C rootdir] [-g] [-z] [-d] -c ... > filename.tar\n",538+ printf(" %s [-C rootdir] [-d] -c ... > filename.tar\n",
539 progname);539 progname);
540 exit(-1);540 exit(-1);
541 }541 }

Subscribers

People subscribed via source and target branches