Never install weston setuid
authorKristian Høgsberg <krh@bitplanet.net>
Thu, 19 Jul 2012 18:13:06 +0000 (14:13 -0400)
committerKristian Høgsberg <krh@bitplanet.net>
Thu, 19 Jul 2012 18:17:58 +0000 (14:17 -0400)
We don't support this anymore.  weston requires a setuid helper (such as
weston-launch) to run under kms, and should never run as root itself.

Disabe the setuid warning in configure.ac since we now only install the
minimal weston-launch as setuid.

configure.ac
src/Makefile.am

index 6c27cd4..30c12b4 100644 (file)
@@ -247,13 +247,3 @@ AC_CONFIG_FILES([Makefile
                 protocol/Makefile
                 tests/Makefile])
 AC_OUTPUT
-
-if test "x$enable_setuid_install" = xyes; then
-AC_MSG_WARN([
-
- *** You've enabled the setuid install hook.  Weston is still a 
- *** pre-alpha project and may have bugs and issues that make a
- *** setuid install unsafe. Proceed at your own risk.
-])
-fi
-
index 282df73..f7cb90c 100644 (file)
@@ -62,14 +62,6 @@ install-exec-hook:
        chmod u+s $(DESTDIR)$(bindir)/weston-launch
 endif
 
-else # BUILD_WESTON_LAUNCH
-
-if ENABLE_SETUID_INSTALL
-install-exec-hook:
-       chown root $(DESTDIR)$(bindir)/weston
-       chmod u+s $(DESTDIR)$(bindir)/weston
-endif
-
 endif # BUILD_WESTON_LAUNCH
 
 moduledir = @libdir@/weston