[PATCH] wineserver: link to the inotify library on non-Linux systems

[Damjan Jovanovic]

Outside Linux, wineserver fails to link when sys/inotify.h
is present, as it tries to find inotify symbols in libc,
which doesn't have them. The inotify symbols are in libinotify.so.
Detect it properly, and link to it where available.

Does not regenerate "configure".

Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com>
---
 configure.ac       | 11 ++++++++++-
 server/Makefile.in |  2 +-
 2 files changed, 11 insertions(+), 2 deletions(-)

[Alexandre Julliard]

Damjan Jovanovic <damjan.jov(a)gmail.com> writes:
> @@ -1351,6 +1351,15 @@ fi
>  WINE_WARNING_WITH(xslt,[test "x$ac_cv_lib_soname_xslt" = "x"],
>                   [libxslt ${notice_platform}development files not found, xslt won't be supported.])
>
> +dnl **** Check for inotify ****
> +if test "x$with_inotify" != "xno"
> +then
> +    WINE_PACKAGE_FLAGS(INOTIFY,[libinotify],,,,
> +        [AC_CHECK_HEADER([sys/inotify.h])])
> +fi
> +WINE_WARNING_WITH(inotify,[test "$ac_cv_header_sys_inotify_h" != "yes"],
> +                 [libinotify ${notice_platform}development files not found (or too old), filesystem change notifications won't be supported.])

We don't require sys/inotify.h on i386 Linux at the moment. Maybe this
could be changed, but either way the warning should be in sync with what
the code does.

-- 
Alexandre Julliard
julliard(a)winehq.org

[Damjan Jovanovic]

On Fri, Apr 19, 2019 at 4:16 PM Alexandre Julliard <julliard(a)winehq.org>
wrote:
> Damjan Jovanovic <damjan.jov(a)gmail.com> writes:
>
> We don't require sys/inotify.h on i386 Linux at the moment. Maybe this
> could be changed, but either way the warning should be in sync with what
> the code does.

I see... thank you!