[PATCH weston] build: fix setuid check

[Giulio Camuffo]

2015-04-10 19:55 GMT+03:00 Derek Foreman <derekf at osg.samsung.com>:
> At least in some shells test needs 3 arguments for string comparison

Hi,

this fixes the issue for me.

Reviewed-By: Giulio Camuffo <giuliocamuffo at gmail.com>

>
> if test $foo=yes ...
>
> will always be true.
>
> if test $foo = yes ...
>
> will perform a string comparison.
>
> Signed-off-by: Derek Foreman <derekf at osg.samsung.com>
> ---
>  Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile.am b/Makefile.am
> index c1347b8..2f16fac 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -165,7 +165,7 @@ install-exec-hook:
>         chown root $(DESTDIR)$(bindir)/weston-launch \
>                 && chmod u+s $(DESTDIR)$(bindir)/weston-launch \
>                 && 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 'To skip this step, re-run ./configure using --disable-setuid-install'; \
>                 false; \
> --
> 2.1.4
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel