aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2009-06-27 21:06:46 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2009-06-27 21:06:46 +0000
commit4d53a2f9699547bdc12831d2860c9d44c465e805 (patch)
treef92c73ecced3600574ce7345a3452ad5a3a68656
parent41f467f3437af7d66a81f90cfdc2fa45399ad2a5 (diff)
downloadpostgresql-REL8_4_0.tar.gz
postgresql-REL8_4_0.zip
Revert addition of "o" to tar options. This was intended to fix bug #4883,REL8_4_0
but the cure appears to be worse than the disease. It turns out that GNU tar versions 1.14.x misinterpret -o as --same-owner, not --no-same-owner, leading to exactly the wrong behavior for both root and nonroot users. While that bug has been fixed for nearly five years, these tar versions are still found in the wild, notably in OS X 10.4. Given that #4883 was the first complaint we'd heard, it's definitely not worth fixing at the risk of breaking things for other users. Perhaps revisit at a later date when we're not up against a release deadline.
-rw-r--r--doc/Makefile6
1 files changed, 3 insertions, 3 deletions
diff --git a/doc/Makefile b/doc/Makefile
index fe3d787b9e6..247b04752cd 100644
--- a/doc/Makefile
+++ b/doc/Makefile
@@ -4,7 +4,7 @@
#
# Copyright (c) 1994, Regents of the University of California
#
-# $PostgreSQL: pgsql/doc/Makefile,v 1.36 2009/06/26 06:40:57 petere Exp $
+# $PostgreSQL: pgsql/doc/Makefile,v 1.37 2009/06/27 21:06:46 tgl Exp $
#
#----------------------------------------------------------------------------
@@ -53,7 +53,7 @@ man1/.timestamp: man7/.timestamp
@echo timestamp >$@
man7/.timestamp: man.tar.gz
- gzip -d -c $< | $(TAR) xof -
+ gzip -d -c $< | $(TAR) xf -
ifneq ($(sqlmansectnum),7)
for file in man1/*.1; do \
mv $$file $$file.bak && \
@@ -76,7 +76,7 @@ endif # found_man
install: all installdirs
ifdef found_html
- gzip -d -c $(srcdir)/postgres.tar.gz | ( cd $(DESTDIR)$(htmldir)/html && $(TAR) xof - )
+ gzip -d -c $(srcdir)/postgres.tar.gz | ( cd $(DESTDIR)$(htmldir)/html && $(TAR) xf - )
endif
ifdef found_man
for file in man1/*.1 man$(sqlmansectnum)/*.$(sqlmansect) ; do \