<div dir="ltr">Bill's patch looks a lot more comprehensive, I'd use that one.<div><br></div><div>Thanks,</div><div>--Robert</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 21 August 2014 00:37, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Fri, 20 Jun 2014 15:23:59 +1200<br>
Robert Ancell <<a href="mailto:robert.ancell@gmail.com">robert.ancell@gmail.com</a>> wrote:<br>
<br>
> I'm not sure if the expected behaviour is for:<br>
> $ weston --shell foo.so<br>
> to work, if so the patch can be modified.<br>
<br>
</div>Ah, sorry for not noticing this patch in time.<br>
What do you think of Bill's series here:<br>
<a href="http://lists.freedesktop.org/archives/wayland-devel/2014-August/016657.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2014-August/016657.html</a><br>
<br>
<br>
Thanks,<br>
pq<br>
<div class="HOEnZb"><div class="h5"><br>
> On 20 June 2014 15:22, Robert Ancell <<a href="mailto:robert.ancell@gmail.com">robert.ancell@gmail.com</a>> wrote:<br>
><br>
> > On my system:<br>
> > $ weston --shell<br>
> > would use the first environment variable as the argument to --shell<br>
> > and<br>
> > $ weston --shell foo.so<br>
> > Would use foo.so as the argument and then parse foo.so as a new option.<br>
> > ---<br>
> >  shared/option-parser.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/shared/option-parser.c b/shared/option-parser.c<br>
> > index c00349a..bddaccb 100644<br>
> > --- a/shared/option-parser.c<br>
> > +++ b/shared/option-parser.c<br>
> > @@ -66,7 +66,7 @@ parse_options(const struct weston_option *options,<br>
> >                             argv[i][1] == '-' &&<br>
> >                             strncmp(options[k].name, &argv[i][2], len) ==<br>
> > 0 &&<br>
> >                             (argv[i][len + 2] == '=' || argv[i][len + 2]<br>
> > == '\0')) {<br>
> > -                               handle_option(&options[k], &argv[i][len +<br>
> > 3]);<br>
> > +                               handle_option(&options[k], argv[i][len +<br>
> > 2] == '=' ? &argv[i][len + 3] : &argv[i][len + 2]);<br>
> >                                 break;<br>
> >                         } else if (options[k].short_name &&<br>
> >                                    argv[i][0] == '-' &&<br>
> > --<br>
> > 2.0.0<br>
> ><br>
> ><br>
<br>
</div></div></blockquote></div><br></div>