Hi Mikael
> -----Original Message-----
> From: Mikael Ronstrom [mailto:mikael@stripped]
> Sent: Friday, November 20, 2009 6:46 PM
> To: commits@stripped
> Subject: bzr commit into mysql-5.6-next-mr branch (mikael:2924)
>
> #At file:///home/mikael/mysql_clones/mysql-next-mr-WL5105/ based on
> revid:davi.arnaut@stripped
>
> 2924 Mikael Ronstrom 2009-11-20 [merge]
> Merge
>
> modified:
> mysys/ptr_cmp.c
> === modified file 'mysys/ptr_cmp.c'
> --- a/mysys/ptr_cmp.c 2007-05-10 09:59:39 +0000
> +++ b/mysys/ptr_cmp.c 2009-09-23 13:15:35 +0000
> @@ -22,16 +22,39 @@
> #include "mysys_priv.h"
> #include <myisampack.h>
>
> +#ifdef UNIV_SOLARIS
Is not UNIV_SOLARIS is something that is defined by Innodb and meant to be
used by innodb internally? As MySql should also be able to run without
Innodb, MySQLish TARGET_OS_SOLARIS seems more logical to me than UNIV_XXX
stuff .