[Sheepdog] [PATCH] Autotoolize sheepdog

[Steven Dake]

On Thu, 2010-05-20 at 17:42 +0900, MORITA Kazutaka wrote:
> At Wed, 19 May 2010 01:38:33 +0900,
> MORITA Kazutaka wrote:
> > 
> > At Mon, 17 May 2010 08:47:24 -0700,
> > Steven Dake wrote:
> > > 
> > > This patch contains the initial work to autotoolize sheepdog.  To generate the
> > > configure script, run the autogen.sh script.  Then run ./configure followed by
> > > make.  make install also works properly.
> > > 
> > > Then run configure as normal.  The configure.ac supports several options
> > > including --enable-profiling and --enable-debug as well as the standard GNU
> > > automake setup.
> > > 
> > > The warnings list is a pretty standard list of warning catches, but does
> > > generate alot of warning output.
> > > 
> > > One thing that is missing is using the git version field rather then
> > > PACKAGE_VERSION generated from configure.ac.
> > > 
> > > Finally, if you want to generate an output tarball, run make dist.
> > > 
> > > make distcheck verifies the make system included all the proper files to
> > > support a self-build.
> > > 
> > > Signed-off-by: Steven Dake <sdake at redhat.com>
> > > ---
> > >  Makefile            |   40 ----
> > >  Makefile.am         |   23 ++
> > >  autogen.sh          |    5 +
> > >  collie/Makefile     |   30 ---
> > >  collie/Makefile.am  |   42 ++++
> > >  configure.ac        |  326 ++++++++++++++++++++++++++
> > >  include/Makefile.am |    3 +
> > >  lib/Makefile        |    3 -
> > >  lib/Makefile.am     |    3 +
> > >  script/Makefile.am  |    3 +
> > >  sheep/Makefile      |   30 ---
> > >  sheep/Makefile.am   |   42 ++++
> > >  sheep/net.c         |  643 ---------------------------------------------------
> > >  sheep/sdnet.c       |  643 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  sheep/sheep.c       |    7 +-
> > >  sheep/store.c       |    3 +
> > >  16 files changed, 1098 insertions(+), 748 deletions(-)
> > >  delete mode 100644 Makefile
> > >  create mode 100644 Makefile.am
> > >  create mode 100755 autogen.sh
> > >  delete mode 100644 collie/Makefile
> > >  create mode 100644 collie/Makefile.am
> > >  create mode 100644 configure.ac
> > >  create mode 100644 include/Makefile.am
> > >  delete mode 100644 lib/Makefile
> > >  create mode 100644 lib/Makefile.am
> > >  create mode 100644 script/Makefile.am
> > >  delete mode 100644 sheep/Makefile
> > >  create mode 100644 sheep/Makefile.am
> > >  delete mode 100644 sheep/net.c
> > >  create mode 100644 sheep/sdnet.c
> > > 
> > 
> > I get the following configure error:
> > 
> > checking for pkg-config... /usr/bin/pkg-config
> > checking pkg-config is at least version 0.9.0... yes
> > checking for ncurses... configure: error: Package requirements (ncurses) were not met:
> > 
> > No package 'ncurses' found
> > 
> > Consider adjusting the PKG_CONFIG_PATH environment variable if you
> > installed software in a non-standard prefix.
> > 
> > Alternatively, you may set the environment variables ncurses_CFLAGS
> > and ncurses_LIBS to avoid the need to call pkg-config.
> > See the pkg-config man page for more details.
> > 
> > 
> > It seems that debian ncurses package doesn't provide ncurses.pc for pkg-config.
> > Is it possible to avoid pkg-config check?
> > 
> 
> I've removed ncurses dependency and applied this patch to the sheepdog
> tree.  If we use other libraries which is not support pkg-config in
> future, let's reconsider this problem again.
> 

Yup makes sense.  I've sent a patch which removes the ncurses devel
package dependency as well.  Otherwise sheepdog wont build without
ncurses-devel.

Regards
-steve

> Thanks,
> 
> Kazutaka