[xen-tools-discuss] Some Problems building a checkout on an ubuntu system

[Simon Hürlimann]

Hi all

I'm trying to build the current sources from the mercurial repository.

Unfortunately there's some problems:

Missing 'man' directory
=======================
The mercurial repository contains no 'man' directory, probably as it's
empty anyway.

Workaround: create by hand

Possible patch:
diff -r 80fe283d60de Makefile
--- a/Makefile  Wed Dec 10 21:39:59 2008 +0000
+++ b/Makefile  Thu Jan 01 22:39:52 2009 +0100
@@ -56,7 +56,7 @@ clean:
        @find . -name '*.tmp' -exec rm \{\} \;
        @find . -name 'tags' -exec rm \{\} \;
        @find . -name '*.8.gz' -exec rm \{\} \;
-       @find man -name '*.html' -exec rm \{\} \;
+       @if [ -d man ]; then rm -rf man ; fi
        @if [ -e build-stamp ]; then rm -f build-stamp ; fi
        @if [ -e configure-stamp ]; then rm -f configure-stamp ; fi
        @if [ -d debian/xen-tools ]; then rm -rf ./debian/xen-tools; fi
@@ -195,6 +195,7 @@ install: fixup-perms install-bin install
 #  Build our manpages via the `pod2man` command.
 #
 manpages:
+       mkdir man
        cd bin; for i in *-*; do pod2man --release=${VERSION} --official
--section=8 $$i ../man/$$i.8; done
        for i in man/*.8; do gzip --force -9 $$i; done


No $TMP guaranteed
==================
The 'release' target wants to use a directory under $TMP. When this
variable isn't set, then it tries to install somewhere below /:

 cp -R . /xen-tools-4.1
 cp: cannot create directory `/xen-tools-4.1': Permission denied

Workaround: pass TMP in one way or the other...

Possible path:
diff -r 74cdba8dffc6 Makefile
--- a/Makefile  Thu Jan 01 22:53:55 2009 +0100
+++ b/Makefile  Thu Jan 01 23:08:46 2009 +0100
@@ -12,6 +12,7 @@
 #
 #  Only used to build distribution tarballs.
 #
+TMP        ?= /tmp
 DIST_PREFIX = ${TMP}
 VERSION     = 4.1
 BASE        = xen-tools

This just uses /tmp as a fallback.

Not knowing anymore if Debian systems have a TMP variable by default, I
checked mktemp(1) and tempfile(1): they both support TMPDIR, so maybe
using that one would make sense, too...


Big Thank you
=============
Well, that's it for now;-) Thanx for this nice peace of software!

Simon