<p dir="ltr">Hello Bryce,</p>
<p dir="ltr">Not really. I'll try to have a go still this weekend.</p>
<p dir="ltr">Regards,<br>
Frederico</p>
<br><div class="gmail_quote"><div dir="ltr">On Sat, Sep 26, 2015, 03:06 Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Jul 13, 2015 at 10:03:46PM +0200, Frederico Cadete wrote:<br>
> On Mon, Jul 13, 2015 at 8:21 PM, Quentin Glidic<br>
> <<a href="mailto:sardemff7%2Bwayland@sardemff7.net" target="_blank">sardemff7+wayland@sardemff7.net</a>> wrote:<br>
> > On 2015-07-13 18:07, Frederico Cadete wrote:<br>
> >> I thought of doing that, but can't think of a way to do it without<br>
> >> breaking someone's build.<br>
> >><br>
> >> Currently, weston enables sd-login if it finds the corresponding .pc,<br>
> >> and disables it if not. If I add a --enable-sd-login/--disable-sd-login,<br>
> >> I see two choices:<br>
> >><br>
> >> - Make --enable-sd-login the default. The build will fail if it does not<br>
> >>   find libsystemd. People who don't have it will have to add --disable-sd-login<br>
> >>   when calling configure.<br>
> >> - Make --disable-sd-login the default. This will silently change the<br>
> >>   builds of people who had libsystemd and expect it to come enabled.<br>
> >><br>
> >> I am leaning for the first option, because the breakage is louder.<br>
> >><br>
> >> Am I missing some way to do this better?<br>
> ><br>
> > See how the vaapi-recorder switch is doing it. You can force<br>
> > enable/disable and the default is to auto-detect.<br>
><br>
> Doh.<br>
><br>
> I'll look it up and probably submit another patch in a few days.<br>
> This one can go in independently as far as I am concerned.<br>
<br>
Hi Frederico,<br>
<br>
By chance have you had time to look at this option?<br>
Right now would be a great time to land both the original patch and the<br>
config option patch.  Looks like this is still quite relevant.<br>
<br>
Bryce<br>
</blockquote></div>