[PATCH ocserv 4/4] README: Add info on build dependencies

[Kevin Cernekee]

On Sun, Dec 1, 2013 at 2:12 AM, Nikos Mavrogiannopoulos <nmav at gnutls.org> wrote:
>> FYI: the "sometimes libopts/Makefile exists, sometimes not" logic
>> breaks the distclean and dist targets.  I spent some time messing
>> around with it but did not come up with a satisfactory fix.
>
> Thank you. I've applied the patches. Is that due to libopts.m4? Maybe
> that should go as a bug report to autogen.

It seems like making AC_CONFIG_FILES conditional may be a problem:

AC_DEFUN([LIBOPTS_CHECK], [
  m4_pushdef([AO_Libopts_Dir],
              [ifelse($1, , [libopts], [$1])])
  LIBOPTS_BUILD_BLOCKED=''
  LIBOPTS_CHECK_COMMON(AO_Libopts_Dir)
  AM_COND_IF([NEED_LIBOPTS], [
    INVOKE_LIBOPTS_MACROS
    AC_CONFIG_FILES(AO_Libopts_Dir/Makefile)
  ])dnl

If I try adding libopts/Makefile to AC_CONFIG_FILES in configure.ac so
that we always generate libopts/Makefile to support distclean and
dist, autoconf aborts because it was added twice.

I also noticed that the autogen binary on my system (FC19) generates
*-args files which reference symbols that do not exist in the bundled
libopts, causing a compile error when --enable-local-libopts is used.
So it looks like there is a dependency between the library version and
the autogen version.  This could be a problem for users building the
released tarball sources (with bundled *-args files) against the
libopts system library.

> btw. Does the license: accept fix the mobile client connection?

Yes, I can now successfully connect with both Android and iOS clients.