Code review comment for lp:~vlad-lesin/percona-xtrabackup/2.1-bug-1227240

On 11/11/2013 02:05 PM, Alexey Kopytov wrote:
> Vlad,
>
> "require_server_version_higher_than '5.6.10'" ensures that the test is only run with xtrabackup_56 as the XtraBackup binary. If it wasn't the case, a lot of other tests would fail too. Thus checks for xtrabackup_56 (and thus, the function) are redundant, please remove them.
>

This is true for our default use case but this is not true for common
case. "require_server_version_higher_than" checks only server version.
If "XB_BUILD" variable is set to "autodetect"(default value) then
xtrabackup binary name is chosen based on server version(see
get_version_info()). But the default value and consequently xtrabackup
binary name can be changed with "-c" option of "run.sh".