[Sheepdog] [PATCH] Autotoolize sheepdog

MORITA Kazutaka morita.kazutaka at lab.ntt.co.jp
Mon May 24 09:31:15 CEST 2010


At Fri, 21 May 2010 10:53:15 -0700,
Steven Dake wrote:
> 
> 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.
> 

Applied the patches, thanks.

Kazutaka



More information about the sheepdog mailing list