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
1=== modified file 'innobackupex'
2--- innobackupex 2011-05-20 17:23:07 +0000
3+++ innobackupex 2011-05-24 23:21:03 +0000
4@@ -12,7 +12,7 @@
5 use Pod::Usage qw(pod2usage);
6 use POSIX "strftime";
7 use POSIX ":sys_wait_h";
8-use POSIX "tmpnam";
9+use File::Temp; # use POSIX "tmpnam";
10 use FileHandle;
11 use File::Basename;
12 use English qw(-no_match_vars);
13@@ -148,10 +148,10 @@
14 my $mysql_server_version = '';
15
16 # name of the file where stderr of mysql process is directed
17-my $mysql_stderr = 'stderr';
18+my $mysql_stderr = File::Temp->new();
19
20 # name of the file where stdout of mysql process is directed
21-my $mysql_stdout = 'stdout';
22+my $mysql_stdout = File::Temp->new();
23
24 # name of the file where binlog position info is written
25 my $binlog_info;
26@@ -214,6 +214,11 @@
27 print_version();
28
29 # initialize global variables and perform some checks
30+if (!$option_tmpdir){
31+ my $tmpdir = File::Temp->newdir();
32+ $option_tmpdir = $tmpdir->dirname;
33+}
34+
35 if ($option_copy_back) {
36 if ( $option_ibbackup_binary eq 'autodetect' ) {
37 $option_ibbackup_binary = 'xtrabackup';
38
39=== modified file 'patches/tar4ibd_libtar-1.2.11.patch'
40--- patches/tar4ibd_libtar-1.2.11.patch 2011-03-15 21:58:13 +0000
41+++ patches/tar4ibd_libtar-1.2.11.patch 2011-05-24 23:21:03 +0000
42@@ -531,11 +531,11 @@
43 void
44 usage()
45 {
46-- printf("Usage: %s [-C rootdir] [-g] [-z] -x|-t filename.tar\n",
47-+ printf("Usage: %s [-C rootdir] [-g] [-z] -x|-t < filename.tar\n",
48+- printf("Usage: %s [-C rootdir] -x|-t filename.tar\n",
49++ printf("Usage: %s [-C rootdir] -x|-t < filename.tar\n",
50 progname);
51-- printf(" %s [-C rootdir] [-g] [-z] -c filename.tar ...\n",
52-+ printf(" %s [-C rootdir] [-g] [-z] [-d] -c ... > filename.tar\n",
53+- printf(" %s [-C rootdir] -c filename.tar ...\n",
54++ printf(" %s [-C rootdir] [-d] -c ... > filename.tar\n",
55 progname);
56 exit(-1);
57 }

Subscribers

People subscribed via source and target branches