| @@ -1,59 +1,59 @@ | | | @@ -1,59 +1,59 @@ |
1 | $NetBSD: patch-ab,v 1.7 2011/05/05 16:54:55 gdt Exp $ | | 1 | $NetBSD: patch-ab,v 1.8 2011/12/07 16:13:51 drochner Exp $ |
2 | | | 2 | |
3 | Multiple changes. | | 3 | Multiple changes. |
4 | | | 4 | |
5 | 1) Remove checks for valid user. | | 5 | 1) Remove checks for valid user. |
6 | XXX EXPLAIN WHY. XXX EXPLAIN UPSTREAM STATUS. | | 6 | XXX EXPLAIN WHY. XXX EXPLAIN UPSTREAM STATUS. |
7 | | | 7 | |
8 | 2) Remove apparent Linuxy startup work. | | 8 | 2) Remove apparent Linuxy startup work. |
9 | XXX EXPLAIN WHY. XXX EXPLAIN UPSTREAM STATUS. | | 9 | XXX EXPLAIN WHY. XXX EXPLAIN UPSTREAM STATUS. |
10 | | | 10 | |
11 | 3) When not isntalling a new config file, check in DESTDIR, not the | | 11 | 3) When not isntalling a new config file, check in DESTDIR, not the |
12 | real filesystem. Avoids failing to install the config in DESTDIR | | 12 | real filesystem. Avoids failing to install the config in DESTDIR |
13 | because it's installed on the host system. | | 13 | because it's installed on the host system. |
14 | | | 14 | |
15 | Not pushed upstream; it's not clear that upstream supports DESTDIR | | 15 | Not pushed upstream; it's not clear that upstream supports DESTDIR |
16 | at all. | | 16 | at all. |
17 | | | 17 | |
18 | --- GNUmakefile.in.orig 2010-02-20 12:53:30.000000000 +0000 | | 18 | --- GNUmakefile.in.orig 2011-09-06 18:45:28.000000000 +0000 |
19 | +++ GNUmakefile.in | | 19 | +++ GNUmakefile.in |
20 | @@ -85,7 +85,7 @@ INSTALL = @INSTALL@ | | 20 | @@ -86,7 +86,7 @@ INSTALL = @INSTALL@ |
21 | # Binaries | | 21 | # Binaries |
22 | BIN_MODE = 0755 | | 22 | BIN_MODE = 0755 |
23 | # Support files, docs, etc. | | 23 | # Support files, docs, etc. |
24 | -RA_MODE = 0664 | | 24 | -RA_MODE = 0664 |
25 | +RA_MODE = 0644 | | 25 | +RA_MODE = 0644 |
26 | # Directory | | 26 | # Directory |
27 | DIR_MODE = 0755 | | 27 | DIR_MODE = 0755 |
28 | # Files daemon writes to. | | 28 | # Files daemon writes to. |
29 | @@ -788,16 +788,6 @@ install-strip: | | 29 | @@ -808,16 +808,6 @@ install-strip: |
30 | # Perhaps the whole user/group validation should be done here, and simplified. | | 30 | # Perhaps the whole user/group validation should be done here, and simplified. |
31 | PROGRAM_V = Privoxy $(VERSION) $(CODE_STATUS) | | 31 | PROGRAM_V = Privoxy $(VERSION) $(CODE_STATUS) |
32 | install: CONF_DEST LOG_DEST PID_DEST check_doc GROUP_T | | 32 | install: CONF_DEST LOG_DEST PID_DEST check_doc GROUP_T |
33 | - @# Quick test for valid USER. | | 33 | - @# Quick test for valid USER. |
34 | - @if [ -n "$(USER)" ]; then \ | | 34 | - @if [ -n "$(USER)" ]; then \ |
35 | - $(ID) $(USER) >/dev/null || exit 1;\ | | 35 | - $(ID) $(USER) >/dev/null || exit 1;\ |
36 | - fi | | 36 | - fi |
37 | - @# Test for valid group. FIXME. USER does not have to belong to GROUP | | 37 | - @# Test for valid group. FIXME. USER does not have to belong to GROUP |
38 | - @# for file ownership purposes. | | 38 | - @# for file ownership purposes. |
39 | -# if [ -n "$(GROUP_T)" ] && [ -n "$(USER)" ] && ! $(GROUPS) $(USER) | $(GREP) "\<$(GROUP_T)\>" >/dev/null; then \ | | 39 | -# if [ -n "$(GROUP_T)" ] && [ -n "$(USER)" ] && ! $(GROUPS) $(USER) | $(GREP) "\<$(GROUP_T)\>" >/dev/null; then \ |
40 | -# $(ECHO) Group $(GROUP_T) for User $(USER) is invalid && exit 1 ;\ | | 40 | -# $(ECHO) Group $(GROUP_T) for User $(USER) is invalid && exit 1 ;\ |
41 | -# fi | | 41 | -# fi |
42 | - | | 42 | - |
43 | @$(ECHO) "Creating directories, and preparing $(PROGRAM_V) installation" | | 43 | @$(ECHO) "Creating directories, and preparing $(PROGRAM_V) installation" |
44 | $(CHMOD) $(DIR_MODE) $(MKDIR) | | 44 | $(CHMOD) $(DIR_MODE) $(MKDIR) |
45 | @$(MKDIR) $(DESTDIR)$(SBIN_DEST) $(DESTDIR)$(prefix) $(DESTDIR)$(CONF_DEST) \ | | 45 | @$(MKDIR) $(DESTDIR)$(SBIN_DEST) $(DESTDIR)$(prefix) $(DESTDIR)$(CONF_DEST) \ |
46 | @@ -865,48 +855,13 @@ install: CONF_DEST LOG_DEST PID_DEST che | | 46 | @@ -885,48 +875,13 @@ install: CONF_DEST LOG_DEST PID_DEST che |
47 | $(INSTALL) $(INSTALL_T) $$i $(DESTDIR)$(CONF_DEST)/templates ;\ | | 47 | $(INSTALL) $(INSTALL_T) $$i $(DESTDIR)$(CONF_DEST)/templates ;\ |
48 | done | | 48 | done |
49 | | | 49 | |
50 | - @# FIXME: group/user validation is overly convoluted. | | 50 | - @# FIXME: group/user validation is overly convoluted. |
51 | - @# If superuser install ... we require a minimum of group ownership | | 51 | - @# If superuser install ... we require a minimum of group ownership |
52 | - @# of those files the daemon writes to, to be non-root owned. | | 52 | - @# of those files the daemon writes to, to be non-root owned. |
53 | - @if [ "`$(ID) |sed 's/(.*//' |sed 's/.*=//'`" = "0" ] ;then\ | | 53 | - @if [ "`$(ID) |sed 's/(.*//' |sed 's/.*=//'`" = "0" ] ;then\ |
54 | - if [ x$(USER) = x ] || [ $(USER) = root ]; then \ | | 54 | - if [ x$(USER) = x ] || [ $(USER) = root ]; then \ |
55 | - if [ x$(GROUP) = x ] || [ $(GROUP) = root ]; then \ | | 55 | - if [ x$(GROUP) = x ] || [ $(GROUP) = root ]; then \ |
56 | - if [ "`$(ID) privoxy`" ] && \ | | 56 | - if [ "`$(ID) privoxy`" ] && \ |
57 | - $(GROUPS) privoxy | $(SED) 's/^.*://' |$(GREP) "\<privoxy\>" >/dev/null; then \ | | 57 | - $(GROUPS) privoxy | $(SED) 's/^.*://' |$(GREP) "\<privoxy\>" >/dev/null; then \ |
58 | - $(ECHO) "Warning: Setting group owner to privoxy";\ | | 58 | - $(ECHO) "Warning: Setting group owner to privoxy";\ |
59 | - GROUP_T=privoxy ;\ | | 59 | - GROUP_T=privoxy ;\ |
| @@ -83,27 +83,27 @@ Multiple changes. | | | @@ -83,27 +83,27 @@ Multiple changes. |
83 | - INSTALL_CONF="$(INSTALL_R)" ;\ | | 83 | - INSTALL_CONF="$(INSTALL_R)" ;\ |
84 | - fi ;\ | | 84 | - fi ;\ |
85 | $(ECHO) Installing configuration files to $(DESTDIR)$(CONF_DEST);\ | | 85 | $(ECHO) Installing configuration files to $(DESTDIR)$(CONF_DEST);\ |
86 | for i in $(CONFIGS); do \ | | 86 | for i in $(CONFIGS); do \ |
87 | if [ "$$i" = "default.action" ] || [ "$$i" = "default.filter" ] ; then \ | | 87 | if [ "$$i" = "default.action" ] || [ "$$i" = "default.filter" ] ; then \ |
88 | $(RM) $(DESTDIR)$(CONF_DEST)/$$i ;\ | | 88 | $(RM) $(DESTDIR)$(CONF_DEST)/$$i ;\ |
89 | $(ECHO) Installing fresh $$i;\ | | 89 | $(ECHO) Installing fresh $$i;\ |
90 | $(INSTALL) $$INSTALL_CONF $$i $(DESTDIR)$(CONF_DEST) || exit 1;\ | | 90 | $(INSTALL) $$INSTALL_CONF $$i $(DESTDIR)$(CONF_DEST) || exit 1;\ |
91 | - elif [ -s "$(CONF_DEST)/$$i" ]; then \ | | 91 | - elif [ -s "$(CONF_DEST)/$$i" ]; then \ |
92 | + elif [ -s "$(DESTDIR)/$(CONF_DEST)/$$i" ]; then \ | | 92 | + elif [ -s "$(DESTDIR)/$(CONF_DEST)/$$i" ]; then \ |
93 | $(ECHO) Installing $$i as $$i.new ;\ | | 93 | $(ECHO) Installing $$i as $$i.new ;\ |
94 | $(INSTALL) $$INSTALL_CONF $$i $(DESTDIR)$(CONF_DEST)/$$i.new || exit 1;\ | | 94 | $(INSTALL) $$INSTALL_CONF $$i $(DESTDIR)$(CONF_DEST)/$$i.new || exit 1;\ |
95 | NEW=1;\ | | 95 | NEW=1;\ |
96 | @@ -918,36 +873,6 @@ install: CONF_DEST LOG_DEST PID_DEST che | | 96 | @@ -938,36 +893,6 @@ install: CONF_DEST LOG_DEST PID_DEST che |
97 | $(CHMOD) $(RWD_MODE) $(DESTDIR)$(CONF_DEST)/*.new || exit 1 ;\ | | 97 | $(CHMOD) $(RWD_MODE) $(DESTDIR)$(CONF_DEST)/*.new || exit 1 ;\ |
98 | $(ECHO) "Warning: Older config files are preserved. Check new versions for changes!" ;\ | | 98 | $(ECHO) "Warning: Older config files are preserved. Check new versions for changes!" ;\ |
99 | fi ;\ | | 99 | fi ;\ |
100 | - [ ! -f $(DESTDIR)$(LOG_DEST)/logfile ] && $(ECHO) Creating logfiles in $(DESTDIR)$(LOG_DEST) || \ | | 100 | - [ ! -f $(DESTDIR)$(LOG_DEST)/logfile ] && $(ECHO) Creating logfiles in $(DESTDIR)$(LOG_DEST) || \ |
101 | - $(ECHO) Checking logfiles in $(DESTDIR)$(LOG_DEST) ;\ | | 101 | - $(ECHO) Checking logfiles in $(DESTDIR)$(LOG_DEST) ;\ |
102 | - $(TOUCH) $(DESTDIR)$(LOG_DEST)/logfile || exit 1 ;\ | | 102 | - $(TOUCH) $(DESTDIR)$(LOG_DEST)/logfile || exit 1 ;\ |
103 | - if [ x$$USER != x ]; then \ | | 103 | - if [ x$$USER != x ]; then \ |
104 | - $(CHOWN) $$USER $(DESTDIR)$(LOG_DEST)/logfile || \ | | 104 | - $(CHOWN) $$USER $(DESTDIR)$(LOG_DEST)/logfile || \ |
105 | - $(ECHO) "** WARNING ** current install user different from configured user. Logging may fail!!" ;\ | | 105 | - $(ECHO) "** WARNING ** current install user different from configured user. Logging may fail!!" ;\ |
106 | - fi ;\ | | 106 | - fi ;\ |
107 | - if [ x$$GROUP_T != x ]; then \ | | 107 | - if [ x$$GROUP_T != x ]; then \ |
108 | - $(CHGRP) $$GROUP_T $(DESTDIR)$(LOG_DEST)/logfile || \ | | 108 | - $(CHGRP) $$GROUP_T $(DESTDIR)$(LOG_DEST)/logfile || \ |
109 | - $(ECHO) "** WARNING ** current install user different from configured user. Logging may fail!!" ;\ | | 109 | - $(ECHO) "** WARNING ** current install user different from configured user. Logging may fail!!" ;\ |