Skip to content

Commit 4d53a2f

Browse files
committed
Revert addition of "o" to tar options. This was intended to fix bug #4883,
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.
1 parent 41f467f commit 4d53a2f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

doc/Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
#
55
# Copyright (c) 1994, Regents of the University of California
66
#
7-
# $PostgreSQL: pgsql/doc/Makefile,v 1.36 2009/06/26 06:40:57 petere Exp $
7+
# $PostgreSQL: pgsql/doc/Makefile,v 1.37 2009/06/27 21:06:46 tgl Exp $
88
#
99
#----------------------------------------------------------------------------
1010

@@ -53,7 +53,7 @@ man1/.timestamp: man7/.timestamp
5353
@echo timestamp >$@
5454

5555
man7/.timestamp: man.tar.gz
56-
gzip -d -c $< | $(TAR) xof -
56+
gzip -d -c $< | $(TAR) xf -
5757
ifneq ($(sqlmansectnum),7)
5858
for file in man1/*.1; do \
5959
mv $$file $$file.bak && \
@@ -76,7 +76,7 @@ endif # found_man
7676

7777
install: all installdirs
7878
ifdef found_html
79-
gzip -d -c $(srcdir)/postgres.tar.gz | ( cd $(DESTDIR)$(htmldir)/html && $(TAR) xof - )
79+
gzip -d -c $(srcdir)/postgres.tar.gz | ( cd $(DESTDIR)$(htmldir)/html && $(TAR) xf - )
8080
endif
8181
ifdef found_man
8282
for file in man1/*.1 man$(sqlmansectnum)/*.$(sqlmansect) ; do \

0 commit comments

Comments
 (0)