MySQL Bugs: #47957: A build "--without-server" fails if using "--with-ssl" (YaSSL)

Bug #47957	A build "--without-server" fails if using "--with-ssl" (YaSSL)
Submitted:	9 Oct 2009 19:25	Modified:	18 Dec 2009 11:43
Reporter:	Joerg Bruehe	Email Updates:
Status:	Closed	Impact on me:	None
Category:	MySQL Server: Compiling	Severity:	S3 (Non-critical)
Version:	5.1.40	OS:	Any
Assigned to:	Joerg Bruehe	CPU Architecture:	Any

Description:
In 5.1, the list of directories to build for the client does not include "extra".
As a consequence, a build configured "--without-server" does not compile the YaSSL code, because those directories are below "extra", so a build "--without-server --with-ssl" (client-only, including YaSSL) fails.

How to repeat:
Just try it ...

Suggested fix:
1) Include "extra" in the list of directories to build for the client.

2) Change the unification of "server" and "client" directories to start with the client:
We support client-only builds, but not server-only, so it is better to always start with the client directories and the just to leave out the server-only ones (if "--without-server").

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/86452

3154 Joerg Bruehe	2009-10-09
      Fix  Bug #47957
        A build "--without-server" fails if using "--with-ssl" (YaSSL)
      
      The problem was the lack of directory "extra" in a build
      which did not include the server.
     @ configure.in
        1) Ensure "extra" is in the list of client directories.
        2) Always handle the client directories first, as they will
           be contained in every build (we have client-only, but not
           server-only).

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/86460

3639 Joerg Bruehe	2009-10-09 [merge]
      Upmerge the fixes for bug#47923 and bug#47957 from 5.1.

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/87019

3639 Joerg Bruehe	2009-10-15 [merge]
      Upmerge the fixes for bug#47923 and bug#47957 from 5.1.
      
      This includes further changes which are needed in 6.0,
      see the comments for "storage/archive/Makefile.am"
      and "unittest/backup/Makefile.am".
     @ configure.in
        Upmerge from 5.1
        
        Specific in 6.0, we need a conditional "SERVER"
        which is used in "unittest/backup/" to not build some
        server-related tests if the build is "--without-server".
     @ mysys/mf_keycache.c
        Upmerge from 5.1
     @ storage/archive/Makefile.am
        See comment in "concurrency_test.c":
        That test cannot be used unless there is thread support
        in the client library.
        
        The code treated this as a runtime issue, but that is wrong:
        It is even a linkage issue, because the test program needs
        both "libmysqlclient" (from "concurrency_test.c") and
        "libmysyslt" (from "azio.c"), and these two will conflict
        unless it is the threaded client library.
        
        The solution is to make the build of this test program
        depend on "THREAD_SAFE_CLIENT".
     @ unittest/backup/Makefile.am
        The necessary "libbackupstream" will not be built
        unless the build includes the server, so building
        the test program must be restricted to that case.

Patch is pushed to the bugteam tree,
currently set at versions 5.1.41.

Looks reasonable and if well tested, OK

Pushed into 5.1.41 (revid:joerg@mysql.com-20091016133539-zjiu7dyiha31lbpo) (version source revid:joerg@mysql.com-20091015182804-pqh97ci3t17x6jo3) (merge vers: 5.1.41) (pib:13)

Noted in 5.1.41 changelogs.

A build configured using the --without-server option did not compile
the yaSSL code, so if --with-ssl was also used, the build failed.

Setting report to NDI pending push into 5.5.x+.

Pushed into 6.0.14-alpha (revid:alik@sun.com-20091022063126-l0qzirh9xyhp0bpc) (version source revid:alik@sun.com-20091019135554-s1pvptt6i750lfhv) (merge vers: 6.0.14-alpha) (pib:13)

Pushed into 5.5.0-beta (revid:alik@sun.com-20091022060553-znkmxm0g0gm6ckvw) (version source revid:alik@sun.com-20091019132831-5j6xs19ld16m7r8j) (merge vers: 5.5.0-beta) (pib:13)

Noted in 5.5.0, 6.0.14 changelogs.

Pushed into 5.1.41-ndb-7.1.0 (revid:jonas@mysql.com-20091218102229-64tk47xonu3dv6r6) (version source revid:jonas@mysql.com-20091218095730-26gwjidfsdw45dto) (merge vers: 5.1.41-ndb-7.1.0) (pib:15)

Added entry to 5.1.40-ndb-7.1.0

Pushed into 5.1.41-ndb-6.2.19 (revid:jonas@mysql.com-20091218100224-vtzr0fahhsuhjsmt) (version source revid:jonas@mysql.com-20091217101452-qwzyaig50w74xmye) (merge vers: 5.1.41-ndb-6.2.19) (pib:15)

Added entry to 5.1.41-ndb-6.2.19

Pushed into 5.1.41-ndb-6.3.31 (revid:jonas@mysql.com-20091218100616-75d9tek96o6ob6k0) (version source revid:jonas@mysql.com-20091217154335-290no45qdins5bwo) (merge vers: 5.1.41-ndb-6.3.31) (pib:15)

Pushed into 5.1.41-ndb-7.0.11 (revid:jonas@mysql.com-20091218101303-ga32mrnr15jsa606) (version source revid:jonas@mysql.com-20091218064304-ezreonykd9f4kelk) (merge vers: 5.1.41-ndb-7.0.11) (pib:15)

Already in 5.1.41