[Bug 37534] 1.7.30 ./configure errors to report

https://bugs.winehq.org/show_bug.cgi?id=37534

Todd Chester  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CLOSED                      |UNCONFIRMED
         Resolution|INVALID                     |---

--- Comment #11 from Todd Chester  ---
Hi Guys.

Adam Jackson of Red Hat has just chimed in.  This is a Wine bug.

"libXrender doesn't link against libXext at all, nor do any
      of libXrender's header files refer to any of libXext's."

He goes on to prove his point.  Would one of our intrepid heroes at Wine please
correct this?

Many thanks,
-T

https://bugzilla.redhat.com/show_bug.cgi?id=1162848

--- Comment #2 from Adam Jackson  ---
(In reply to Todd from comment #0)
> The error from Wine is a bogus error.  Wine's "./configure" was missing
> "libXext", which libXrender has a dependency.  Installing libXext corrected
> the bogus error.
>
> The bogus error was reported to Wine over on:
> https://bugs.winehq.org/show_bug.cgi?id=37534#c8
>
> Their response was:
>  Every X11 extension (which libXrender is) depends on libXext, that's
>  an X11 dependency, not a Wine one.  

But that's not true:

hyoscyamine:~% ldd /usr/lib64/libXrender.so.1   
    linux-vdso.so.1 =>  (0x00007fff813fc000)
    libX11.so.6 => /lib64/libX11.so.6 (0x0000003f22e00000)
    libc.so.6 => /lib64/libc.so.6 (0x0000003f20200000)
    libxcb.so.1 => /lib64/libxcb.so.1 (0x0000003f22a00000)
    libdl.so.2 => /lib64/libdl.so.2 (0x0000003f20600000)
    /lib64/ld-linux-x86-64.so.2 (0x0000003f1fe00000)
    libXau.so.6 => /lib64/libXau.so.6 (0x0000003f22600000)

libXrender doesn't link against libXext at all, nor do any of libXrender's
header files refer to any of libXext's.

The check for libXext appears to come from wine itself, either in the expansion
of AC_PATH_XTRA or from WINE_CHECK_SONAME(Xext, ...) in configure.ac.  The
Fedora (and presumably EPEL if it's there) wine spec file explicitly says
BuildRequires: libXext-devel to pull it into the buildroot.  It's a shame
wine's configure script isn't better about telling you this error, I agree.
> Would you please add libXext as a dependency in libxslt/libXrender's RPM?

Neither libxslt nor libXrender actually depend on libXext, so it would be wrong
to add it.