+2003-08-27 Junichi Uekawa <dancer@debian.org>

+
+	* Documentation/pbuilder-doc.xml (PBUILDER_UML_IMAGE): Update
+	  documentation to note updates that I am aware of.
+
+	* THANKS: update the list.
+
+	* pbuilder.8: fix unintended italic in manual page
+	Subject: Bug#207390: italicized period in manual page
+	From: Matt Kraai <kraai@alumni.cmu.edu>
+
+	* pbuilder-buildpackage (BUILD_PID): add default BUILDRESULTGID for chown.
+

7 files changed, 41 insertions, 10 deletions

diff --git a/ChangeLog b/ChangeLog
index 08dcf00..2da32b4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2003-08-27  Junichi Uekawa  <dancer@debian.org>
+
+	* Documentation/pbuilder-doc.xml (PBUILDER_UML_IMAGE): Update 
+	  documentation to note updates that I am aware of.
+
+	* THANKS: update the list.
+
+	* pbuilder.8: fix unintended italic in manual page
+	Subject: Bug#207390: italicized period in manual page
+	From: Matt Kraai <kraai@alumni.cmu.edu>
+
+	* pbuilder-buildpackage (BUILD_PID): add default BUILDRESULTGID for chown.
+
 2003-08-25  Junichi Uekawa  <dancer@debian.org>
 
 	* debian/README.Debian (Testsuite): update README.Debian
diff --git a/Documentation/pbuilder-doc.xml b/Documentation/pbuilder-doc.xml
index 3ebae98..1b19522 100644
--- a/Documentation/pbuilder-doc.xml
+++ b/Documentation/pbuilder-doc.xml
@@ -668,6 +668,13 @@ PBUILDER_UML_IMAGE="/home/dancer/uml-image"
 	      </para>
 	    </listitem>
 	    <listitem>
+	      <para>
+		Make sure BUILDPLACE is writable by the user.
+		Change BUILDPLACE in the configuration file to a place
+		where the user can access.
+	      </para>
+	    </listitem>
+	    <listitem>
 	      <para>Run <command>pbuilder-user-mode-linux create --distribution sid</command> to create the image</para>
 	    </listitem>
 	    <listitem>
@@ -694,10 +701,11 @@ PBUILDER_UML_IMAGE="/home/dancer/uml-image"
 	    <para>
 	      /tmp is handled differently inside pbuilder-uml.
 	      In pbuilder-uml, /tmp is mounted as tmpfs inside UML,
-	      so accessing files under /tmp outside the user-mode-linux
+	      so accessing files under /tmp from outside the user-mode-linux
 	      does not work.
 	      It affects options like
-	      <command><option>--configfile</option></command>.
+	      <command><option>--configfile</option></command>,
+	      and when trying to build packages placed under <filename>/tmp</filename>.
 	    </para>
 	  </listitem>
 	</itemizedlist>
diff --git a/THANKS b/THANKS
index 2babb8b..634b8b0 100644
--- a/THANKS
+++ b/THANKS
@@ -28,6 +28,8 @@ Barak Pearlmutter <bap@cs.unm.edu>
 Tommaso Moroni <tommaso.m@tiscalinet.it>
 Barak Pearlmutter <bap@cs.unm.edu>
 Marek Habersack <grendel@debian.org>
+Kenneth Pronovici <pronovic@debian.org>
+Matt Kraai <kraai@alumni.cmu.edu>
 
 I thank them all!
 
diff --git a/debian/changelog b/debian/changelog
index c818b11..01792a7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,8 +1,16 @@
-pbuilder (0.84) UNRELEASED; urgency=low
-
-  * README.Debian: update outdated parts.
-
- -- Junichi Uekawa <dancer@debian.org>  Mon, 25 Aug 2003 07:19:06 +0900
+pbuilder (0.84) unstable; urgency=low
+
+  * README.Debian: update very outdated parts.
+  * change use of chown, supply group info when available. 
+  Subject: BUILDRESULTUID and BUILDRESULTGID no longer work as numeric values
+  From: Kenneth Pronovici <pronovic@debian.org>
+    (closes: #207392)
+  * fix manual page slightly.
+  From: Matt Kraai <kraai@alumni.cmu.edu> (closes: #207390)
+  * pbuilder-doc.xml: update.
+  * Standards-version: 3.6.1
+
+ -- Junichi Uekawa <dancer@debian.org>  Wed, 27 Aug 2003 05:58:22 +0900
 
 pbuilder (0.83) unstable; urgency=low
 
diff --git a/debian/control b/debian/control
index fe5a961..737c46b 100644
--- a/debian/control
+++ b/debian/control
@@ -3,7 +3,7 @@ Section: devel
 Priority: extra
 Maintainer: Junichi Uekawa <dancer@debian.org>
 Build-Depends: debhelper (>= 4.1.0), docbook-xsl, ldp-docbook-xsl, xsltproc
-Standards-Version: 3.6.0
+Standards-Version: 3.6.1
 
 Package: pbuilder
 Architecture: all
diff --git a/pbuilder-buildpackage b/pbuilder-buildpackage
index 74e744c..4ce3a06 100755
--- a/pbuilder-buildpackage
+++ b/pbuilder-buildpackage
@@ -129,7 +129,7 @@ if [ -n "${BUILDRESULT}" ]; then
     mkdir -p "${BUILDRESULT}"
     if [ -d "${BUILDRESULT}" ]; then
 	if [ -n "${BUILDRESULTUID}" ]; then
-	    chown "${BUILDRESULTUID}:" "${BUILDPLACE}"/tmp/buildd/*
+	    chown "${BUILDRESULTUID}:${BUILDRESULTGID}" "${BUILDPLACE}"/tmp/buildd/*
 	fi
 	if [ -n "${BUILDRESULTGID}" ]; then
 	    chgrp "${BUILDRESULTGID}" "${BUILDPLACE}"/tmp/buildd/*
diff --git a/pbuilder.8 b/pbuilder.8
index 5a94c93..9703e50 100644
--- a/pbuilder.8
+++ b/pbuilder.8
@@ -185,7 +185,7 @@ option.
 .TP
 .BI "--removepackages [" "packages to remove" "]"
 Removes the packages on creating the 
-.BI "base.tgz" "."
+.BR "base.tgz" "."
 Use this option to remove potentially dangerous or undesirable
 packages, like
 .B "lilo"