build: setuid apps.plugin if running under root

Signed-off-by: Alon Bar-Lev <alon.barlev@gmail.com>

diff --git a/netdata.spec.in b/netdata.spec.in
index af73a7b6d027d6e64eff26797a6b71e2b6330099..2aef64275f98e8f085f8358cae6f44ddf5abb90d 100644 (file)
--- a/netdata.spec.in
+++ b/netdata.spec.in
@@ -45,7 +45,6 @@ make %{?_smp_mflags} install DESTDIR="%{buildroot}"
 find "%{buildroot}" -name .keep -exec rm {} \;
 
 %files
-%attr(4755, root, root) %{_libexecdir}/%{name}/plugins.d/apps.plugin
 %attr(-, netdata, netdata) %dir %{_localstatedir}/cache/%{name}/
 %attr(-, netdata, netdata) %dir %{_localstatedir}/log/%{name}/
 %{_bindir}/%{name}

diff --git a/src/Makefile.am b/src/Makefile.am
index dde90145e4bb47f8351e50aa518c50306fd9491a..1d87d5be77778b74c8cdb7bb8a8debeabbad6dba 100644 (file)
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -73,10 +73,15 @@ apps_plugin_SOURCES = \
        procfile.c procfile.h \
        $(NULL)
 
-install-data-hook:
-       @echo
-       @echo "ATTENTION"
-       @echo
-       @echo "setuid bit of $(pluginsdir)/apps.plugin must be set, please execute as root:"
-       @echo "chown root '$(pluginsdir)/apps.plugin' && chmod 4755 '$(pluginsdir)/apps.plugin'"
-       @echo
+install-exec-hook:
+       if [ `id -u` == 0 ]; then \
+               chown root '$(DESTDIR)$(pluginsdir)/apps.plugin' && \
+                       chmod 4755 '$(DESTDIR)$(pluginsdir)/apps.plugin'; \
+       else \
+               echo; \
+               echo "ATTENTION"; \
+               echo; \
+               echo "setuid bit of $(pluginsdir)/apps.plugin must be set, please execute as root:"; \
+               echo "chown root '$(pluginsdir)/apps.plugin' && chmod 4755 '$(pluginsdir)/apps.plugin'"; \
+               echo; \
+       fi