<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 30, 2013 at 12:42 AM,  <span dir="ltr"><<a href="mailto:sardemff7+wayland@sardemff7.net" target="_blank">sardemff7+wayland@sardemff7.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On 30/05/2013 08:24, Michael Hasselmann wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Tue, 2013-05-28 at 22:16 -0700, Bill Spitzak wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Running autogen.sh in weston with --disable-colord works to avoid this,<br>
I suspect nothing I care about is lost this way.<br>
</blockquote>
<br>
I ran into the very same problems. I would have preferred if such new<br>
dependencies were optional or if someone would explain what we will miss<br>
if we disable them. For colord, trying to build the latest required<br>
version from git just pulls in more dependencies that you also have to<br>
build from git (as Bill mentioned).<br>
<br>
It's kind of frustrating, even if weston is meant to be a playground and<br>
thus new dependencies have to be expected.<br>
<br>
<a href="http://wayland.freedesktop.org/building.html" target="_blank">http://wayland.freedesktop.<u></u>org/building.html</a> should be updated I guess?<br>
<br>
ciao Michael<br>
</blockquote>
<br>
colord is an optional dependency: you just need to pass --disable-colord to ./configure (or ./autogen.sh), and it is gone.<br></blockquote><div><br></div><div style>Making the configure script have a hard failure by default if colord is unavailable is not consistent with the dependency being optional.  Even if --disable-colord is added to the configure/autogen.sh command line "make distcheck" will still fail on platforms without colord installed since the configure script flags are not passed down.  One could tweak the flags passed to the distcheck target using AM_DISTCHECK_CONFIGURE_FLAGS and DISTCHECK_CONFIGURE_FLAGS but that would be hack. I think it would be better to issue a warning if colord wasn't found rather than error out, or flip the logic so that the user must enable colord support, and only then error out.  The same goes for libunwind.</div>
<div style><br></div><div style>I just submitted a patch that causes a missing colord or libunwind to issue a warning rather than error out.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

It is not an automagic dependency (which automatically detect the package’s presence), because it is more work for the developer to add the autotools logic to do so. Also, it is much more work to make packagers happy (packagers hate automagic dependencies, it makes either more work by patching or hard-dependency on optional features).<br>

<br>
A little configure switch, and you are done, nothing to complain about. :-)</blockquote><div><br></div><div style>The configure switch in question doesn't appear to be enough to keep some of us from complaining.  :)</div>
<div style><br></div><div style>Thanks,</div><div style>-Ossama</div><div><br></div></div></div></div>