Re: [PATCH wayland 1/2] build: Remove execinfo.h check

2018-07-26 Thread Daniel Stone 
Hi,

On Thu, 26 Jul 2018 at 16:11, Emil Velikov  wrote:
> On 25 July 2018 at 18:36, Derek Foreman  
> 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 
> >
> > Reviewed-by: Derek Foreman 
> >
> > 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 

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

Re: [PATCH wayland 1/2] build: Remove execinfo.h check

2018-07-26 Thread Emil Velikov 
On 25 July 2018 at 18:36, Derek Foreman  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 
>
> Reviewed-by: Derek Foreman 
>
> 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
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Re: [PATCH wayland 1/2] build: Remove execinfo.h check

2018-07-25 Thread Derek Foreman 
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 

Reviewed-by: Derek Foreman 

Though, I wouldn't mind seeing a more accurate commit log. ;)

> ---
>  configure.ac | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 81cf4077..8c2fb822 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -109,7 +109,6 @@ if test "x$enable_libraries" = "xyes"; then
>   AC_CHECK_DECL(CLOCK_MONOTONIC,[],
> [AC_MSG_ERROR("CLOCK_MONOTONIC is needed to compile 
> wayland libraries")],
> [[#include ]])
> - AC_CHECK_HEADERS([execinfo.h])
>  fi
>  
>  PKG_CHECK_MODULES(EXPAT, [expat], [],
> 

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

[PATCH wayland 1/2] build: Remove execinfo.h check

2018-07-21 Thread Daniel Stone 
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.

Remove the no-op check.

Signed-off-by: Daniel Stone 
---
 configure.ac | 1 -
 1 file changed, 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index 81cf4077..8c2fb822 100644
--- a/configure.ac
+++ b/configure.ac
@@ -109,7 +109,6 @@ if test "x$enable_libraries" = "xyes"; then
AC_CHECK_DECL(CLOCK_MONOTONIC,[],
  [AC_MSG_ERROR("CLOCK_MONOTONIC is needed to compile 
wayland libraries")],
  [[#include ]])
-   AC_CHECK_HEADERS([execinfo.h])
 fi
 
 PKG_CHECK_MODULES(EXPAT, [expat], [],
-- 
2.17.1

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel