[PATCH xserver 1/5] xwayland: Allow "-eglstream" option

Olivier Fourdan fourdan at gmail.com
Tue Jun 5 11:24:56 UTC 2018


Hi Emil,

On 5 June 2018 at 12:41, Emil Velikov <emil.l.velikov at gmail.com> wrote:

> [...]
>
>> > +#else
> >> > +            ErrorF("xwayland glamor: eglstream backend support not
> >> > enabled\n");
> >> Something is really weird here:
> >>
> >> On one hand '-eglstream' is recognised and used (by potential user) on
> >> the other "... support is not _enabled_" is printed.
> >> Surely you meant "not built", right? After all explicitly passing the
> >> enable (runtime) flag should be enough to enable it ;-)
> >
> >
> > Yes, I literally mean "enabled at build time".
> >
> This wording is even better.
>

Eventually, in the updated series I am about to post in a minute, I opted
for the following wording:

   "xwayland glamor: this build does not have eglstream support"

which, I reckon, seems like the most accurate description :)

Cheers,
Olivier
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.x.org/archives/xorg-devel/attachments/20180605/a2e6cb21/attachment.html>


More information about the xorg-devel mailing list