Fix build warnings in xtrabackup.cc

Bug #721690 reported by Valentine Gostev
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Percona XtraBackup moved to https://jira.percona.com/projects/PXB
Fix Released
Medium
Alexey Kopytov
2.0
Won't Fix
Medium
Unassigned
2.1
Won't Fix
Medium
Unassigned
2.2
Fix Released
Medium
Alexey Kopytov

Bug Description

While building xtrabackup from source 21 warnings were met:

~# bzr branch lp:percona-xtrabackup
~# cd percona-xtrabackup
~# ./utils/build.sh xtradb > out 2>&1
~# cat out | grep warning
vi.c:1015: warning: ignoring return value of 'write', declared with attribute warn_unused_result
vi.c:1016: warning: ignoring return value of 'write', declared with attribute warn_unused_result
log/log0recv.c:65:1: warning: "RECV_READ_AHEAD_AREA" redefined
./include/log0recv.h:447:1: warning: this is the location of the previous definition
log/log0recv.c:2816: warning: unknown conversion type character ')' in format
mysqld.cc:4404: warning: deprecated conversion from string constant to 'char*'
mysqld.cc:4410: warning: deprecated conversion from string constant to 'char*'
mysqld.cc:4417: warning: deprecated conversion from string constant to 'char*'
mysqld.cc:4432: warning: deprecated conversion from string constant to 'char*'
mysqld.cc:4439: warning: deprecated conversion from string constant to 'char*'
mysqld.cc:4448: warning: deprecated conversion from string constant to 'char*'
xtrabackup.c:2701: warning: format '%02u' expects type 'unsigned int', but argument 3 has type 'char *'
xtrabackup.c:2701: warning: too few arguments for format
xtrabackup.c:5447: warning: format '%lu' expects type 'long unsigned int', but argument 4 has type 'unsigned int'
extract.c:114: warning: incompatible implicit declaration of built-in function 'strlen'
extract.c:448: warning: incompatible implicit declaration of built-in function 'strlen'
util.c:154: warning: field width should have type 'int', but argument 4 has type 'size_t'
util.c:154: warning: field width should have type 'int', but argument 4 has type 'size_t'
util.c:177: warning: field width should have type 'int', but argument 4 has type 'size_t'
util.c:177: warning: field width should have type 'int', but argument 4 has type 'size_t'
libtar.c:271: warning: incompatible implicit declaration of built-in function 'exit'

description: updated
Changed in percona-xtrabackup:
importance: Undecided → Medium
status: New → Confirmed
assignee: nobody → Vadim Tkachenko (vadim-tk)
Stewart Smith (stewart)
Changed in percona-xtrabackup:
status: Confirmed → Triaged
assignee: Vadim Tkachenko (vadim-tk) → nobody
Revision history for this message
Stewart Smith (stewart) wrote :

Marking as Fix Released as we're not seeing any of this in current Jenkins builds.

Changed in percona-xtrabackup:
status: Triaged → Fix Released
Revision history for this message
Alexey Kopytov (akopytov) wrote :

I actually still see warnings. This one in particular:

log/log0recv.c:65:1: warning: "RECV_READ_AHEAD_AREA" redefined

And tar4ibd ones too. Plus a bunch of other ones that are not mentioned in the original report.

Changed in percona-xtrabackup:
status: Fix Released → Triaged
Revision history for this message
Alexey Kopytov (akopytov) wrote :

There are still warnings in both 2.0 and 2.1.

summary: - xtrabackup ./utils/build.sh xtradb warnings
+ Fix build warnings in xtrabackup.cc
Revision history for this message
Alexey Kopytov (akopytov) wrote :

The single-binary 2.2 branch build with -Werror in debug mode.

Revision history for this message
Shahriyar Rzayev (rzayev-sehriyar) wrote :

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PXB-63

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.