← Back to team overview

maria-developers team mailing list archive

Re: [Commits] ed5ec30: MDEV-7780 - Support for faking server version

 

Hi, Sergey!

On Dec 24, Sergey Vojtovich wrote:
> Hi Sergei,
> >   $mysql= $ENV{MYSQL_TEST} =~ /mysqltest_embedded / ? 'MYSQL_EMBEDDED' : 'MYSQL';
> >   $cnt= grep /my_favorite_version/, `$ENV{$mysql} -e status`;
> - mtr is looking for this binary in $basedir/libmysqld, while the correct path
>   is something like $bindir/libmysqld/examples$opt_vs_config => it's broken
> - mtr doesn't add arguments to $ENV{MYSQL_EMBEDDED}, like it does for
>   $ENV{MYSQL} => it's dead broken
> - we don't seem to install this binary => extra logic to handle installed tree
> - it won't connect to original server, so it needs another version arg
> - what's the point of testing protocol when there's no protocol whatsoever?

Ok, whatever. You decide where to stop adding tests for --version in
emdded. Ok to push.

Regards,
Sergei
Chief Architect MariaDB
and security@xxxxxxxxxxx
-- 
Vote for my Percona Live 2016 talks:
https://www.percona.com/live/data-performance-conference-2016/sessions/mariadb-connectors-fast-and-smart-new-protocol-optimizations#community-voting
https://www.percona.com/live/data-performance-conference-2016/sessions/mariadb-101-security-validation-authentication-encryption#community-voting


References