# [PATCH (tig)] Supply explicit permission bits to 'install'.

2 messages from 2007-05-31 to 2007-05-31. Participants: Jeffrey C. Ollie, Jonas Fonseca.
Thread: https://gitlist.dev/t/8378

## Jeffrey C. Ollie, 2007-05-31 19:20

Subject: [PATCH (tig)] Supply explicit permission bits to 'install'.
Message-ID: <11806392321132-git-send-email-jeff@ocjtech.us>
URL: https://gitlist.dev/e/11806392321132-git-send-email-jeff%40ocjtech.us

```
'install' will install files with permissions set to '0755' if the
permissions are not specified on the command line.  Having the execute
bits set on non-executable content is not desireable.  Specify mode
0644 for non-executable content as well as specify mode 0755 for
executable content (in case the defaults change or are different on
different systems).  Also tell 'install' to preserve timestamps.

Signed-off-by: Jeffrey C. Ollie <jeff@ocjtech.us>
---
 Makefile |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index 57196b0..f9adecb 100644
--- a/Makefile
+++ b/Makefile
@@ -38,7 +38,7 @@ doc-html: $(HTMLDOC)
 install: all
 	mkdir -p $(DESTDIR)$(bindir) && \
 	for prog in $(PROGS); do \
-		install $$prog $(DESTDIR)$(bindir); \
+		install -p -m 0755 $$prog $(DESTDIR)$(bindir); \
 	done
 
 install-doc-man: doc-man
@@ -46,8 +46,8 @@ install-doc-man: doc-man
 		 $(DESTDIR)$(mandir)/man5
 	for doc in $(MANDOC); do \
 		case "$$doc" in \
-		*.1) install $$doc $(DESTDIR)$(mandir)/man1 ;; \
-		*.5) install $$doc $(DESTDIR)$(mandir)/man5 ;; \
+		*.1) install -p -m 0644 $$doc $(DESTDIR)$(mandir)/man1 ;; \
+		*.5) install -p -m 0644 $$doc $(DESTDIR)$(mandir)/man5 ;; \
 		esac \
 	done
 
-- 
1.5.2.GIT

```

## Jonas Fonseca, 2007-05-31 19:54

Subject: Re: [PATCH (tig)] Supply explicit permission bits to 'install'.
Message-ID: <20070531195422.GA9260@diku.dk>
URL: https://gitlist.dev/e/20070531195422.GA9260%40diku.dk
In-Reply-To: <11806392321132-git-send-email-jeff@ocjtech.us>

```
Jeffrey C. Ollie <jeff@ocjtech.us> wrote Thu, May 31, 2007:
> 'install' will install files with permissions set to '0755' if the
> permissions are not specified on the command line.  Having the execute
> bits set on non-executable content is not desireable.  Specify mode
> 0644 for non-executable content as well as specify mode 0755 for
> executable content (in case the defaults change or are different on
> different systems).  Also tell 'install' to preserve timestamps.

Thank you. I updated the patch to install HTML files with explicit
permissions.

-- 
Jonas Fonseca

```