Re: Serf 1.2.0 has been released

Philip Martin <philip.martin_at_wandisco.com> writes:

> The build/install don't report any errors even if the tests don't run,
> however the installed library doesn't work:
>
> /usr/local/lib/libserf-1.so: undefined reference to `apr_sockaddr_ip_getbuf'
>
> APR 1.2.7 doesn't have such a symbol. This means the minimum APR that
> can be used is APR 1.3.0. Until recently we have been supporting APR
> as far back as 0.9.

The function is only used for some error messages:

            if (apr_socket_addr_get(&sa, APR_LOCAL, skt) == APR_SUCCESS) {
                char buf[32];
                apr_sockaddr_ip_getbuf(buf, 32, sa);
                fprintf(stderr, "%s:%d", buf, sa->port);
            }
            fprintf(stderr, " r:");
            if (apr_socket_addr_get(&sa, APR_REMOTE, skt) == APR_SUCCESS) {
                char buf[32];
                apr_sockaddr_ip_getbuf(buf, 32, sa);
                fprintf(stderr, "%s:%d", buf, sa->port);
            }

There may be other reasons to upgrade but I don't think we should force
an upgrade just to support that error message.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download