Hi, On Thu, 26 Jul 2018 at 16:11, Emil Velikov <emil.l.veli...@gmail.com> wrote: > On 25 July 2018 at 18:36, Derek Foreman <derek.foreman.sams...@gmail.com> > wrote: > > On 2018-07-21 06:13 AM, Daniel Stone wrote: > >> The check for the execinfo.h header is only advisory; the build will not > >> fail if it is not present, and set HAVE_EXECINFO_H if it is. The check > >> was added in commit 5cfdbef3d299 ("build: Allow disabling building of > >> wayland libraries") with no obvious use or reasoning. > > > > 5cfdbe3d299 just moved it, it was added in commit bc3e020475 > > > > It appears to have been accidentally included there, as it doesn't look > > relevant to the bug ticket referenced in the commit. > > > >> Remove the no-op check. > >> > >> Signed-off-by: Daniel Stone <dani...@collabora.com> > > > > Reviewed-by: Derek Foreman <derek.foreman.sams...@gmail.com> > > > > Though, I wouldn't mind seeing a more accurate commit log. ;) > > > It's always better to not get called out for other's copy/paste > mistakes. If you guys like Fixes tags, here is one > > Fixes: bc3e020475e ("build: Add declaration checks to check for > required syscall flags") > > Fwiw, series is > Reviewed-by: Emil Velikov <emil.veli...@collabora.com>
Thanks both for the review and accuracy check: I've pushed both patches now. Cheers, Daniel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel