Merge lp://staging/~trond-norbye/libmemcached/compile-fixes into lp://staging/~tangent-org/libmemcached/trunk

Proposed by Trond Norbye
Status: Merged
Merge reported by: Brian Aker
Merged at revision: not available
Proposed branch: lp://staging/~trond-norbye/libmemcached/compile-fixes
Merge into: lp://staging/~tangent-org/libmemcached/trunk
Diff against target: 39 lines (+6/-1)
3 files modified
clients/memparse.cc (+1/-1)
libmemcached/common.h (+1/-0)
libmemcached/server.h (+4/-0)
To merge this branch: bzr merge lp://staging/~trond-norbye/libmemcached/compile-fixes
Reviewer Review Type Date Requested Status
Libmemcached-developers Pending
Review via email: mp+69754@code.staging.launchpad.net

Description of the change

Minor compile fixes:

1) The code does not compile on Solaris 10 due to missing include files. The function index() is used in parser.cc, and according to the man page one should include <strings.h>. One of the client programs use strlen without including <cstring>

2) libmemcached/server.h that is installed as part of the client installation use NI_MAXHOST in the structure definition, but without including netdb.h that defines it. By doing so the clients of libmemcached would have to include netdb.h before including libmemcached/memcached.h.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: