~sircmpwn/gmni-devel

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(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~sircmpwn/gmni-devel/patches/20823/mbox | git am -3
Learn more about email & git
View this thread in the archives

[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
Thanks!

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