mirrors/weston
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

build: fix setuid check

Browse Source 
At least in some shells test needs 3 arguments for string comparison

if test $foo=yes ...

will always be true.

if test $foo = yes ...

will perform a string comparison.

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-By: Giulio Camuffo <giuliocamuffo@gmail.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
This commit is contained in:
Derek Foreman 2015-04-10 11:55:27 -05:00 committed by Pekka Paalanen
parent 9a0b2b54e2
commit 6d11679843
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Makefile.am
View File

@ -165,7 +165,7 @@ install-exec-hook:
chown root $(DESTDIR)$(bindir)/weston-launch \ chown root $(DESTDIR)$(bindir)/weston-launch \
&& chmod u+s $(DESTDIR)$(bindir)/weston-launch \ && chmod u+s $(DESTDIR)$(bindir)/weston-launch \
&& can_suid_files=yes;\ && can_suid_files=yes;\
if test $$can_suid_files=no; then \ if test $$can_suid_files = no; then \
echo 'Error: unable to unable to change ownership/setuid on weston-launch.'; \ echo 'Error: unable to unable to change ownership/setuid on weston-launch.'; \
echo 'To skip this step, re-run ./configure using --disable-setuid-install'; \ echo 'To skip this step, re-run ./configure using --disable-setuid-install'; \
false; \ false; \