gmni: Makefile: install libgmni.a with 644 perms v1 APPLIED

Ariadna Vigo: 1
 Makefile: install libgmni.a with 644 perms

 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH gmni] Makefile: install libgmni.a with 644 perms Export this patch

Hi!
Static libraries don't need execution perms. I know this is very minor,
but best practice is to give everything the least amount of necessary
privileges. Thus, I propose changing the install command to use 644 
perms.

Yours,
Ariadna

---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 5e468e2..1064a1e 100644
--- a/Makefile
+++ b/Makefile
@@ -61,7 +61,7 @@ install: all install_docs
 	mkdir -p $(LIBDIR)/pkgconfig
 	install -m755 gmni $(BINDIR)/gmni
 	install -m755 gmnlm $(BINDIR)/gmnlm
-	install -m755 libgmni.a $(LIBDIR)/libgmni.a
+	install -m644 libgmni.a $(LIBDIR)/libgmni.a
 	install -m644 include/gmni/gmni.h $(INCLUDEDIR)/gmni/gmni.h
 	install -m644 include/gmni/tofu.h $(INCLUDEDIR)/gmni/tofu.h
 	install -m644 include/gmni/url.h $(INCLUDEDIR)/gmni/url.h
-- 
2.30.1

Drew DeVault

3 months ago

Thanks!

To git@git.sr.ht:~sircmpwn/gmni
   42d8022..31dfd38  master -> master