[Bug 29906] New: wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

Bug #: 29906
           Summary: wine build broken on clang from trunk
           Product: Wine
           Version: 1.4-rc3
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Keywords: download, source
          Severity: critical
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

make[1]: Entering directory `/home/austin/src/wine-clang/dlls/oleaut32/tests'
clang -c -I. -I. -I../../../include -I../../../include 
-DWINE_STRICT_PROTOTYPES -DWINE_NO_NAMELESS_EXTENSION  -D_REENTRANT -fPIC -Wall
-pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body
-Wignored-qualifiers -Wstrict-prototypes -Wtype-limits
-Wunused-but-set-parameter -Wwrite-strings -fno-omit-frame-pointer
-Wpointer-arith -I/usr/include/freetype2  -g -O2  -o tmarshal.o tmarshal.c
In file included from tmarshal.c:29:
./tmarshal.h:611:41: error: expected member name or ';' after declaration
specifiers
    HRESULT (STDMETHODCALLTYPE *restrict)(
    ~~~~~~~                             ^
./tmarshal.h:611:32: error: pointer to function type 'HRESULT (IWidget *, INT
*) __attribute__((stdcall))' may not be 'restrict' qualified
    HRESULT (STDMETHODCALLTYPE *restrict)(
                               ^
2 errors generated.

[Bug 29906] wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

--- Comment #1 from Juan Lang  2012-02-15 18:07:30 CST ---
That looks like a clang false positive: restrict is a variable name, not a type
qualifier.

[Bug 29906] wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

--- Comment #2 from Austin English  2012-02-15 18:13:27 CST ---
Looks like CFLAGS="-std=gnu89" works around it.

[Bug 29906] wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|critical                    |minor

[Bug 29906] wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #3 from Dan Kegel  2012-02-15 23:45:45 CST ---
Still probably want to remove that new keyword in our sources sometime.

[Bug 29906] wine build broken on clang from trunk

http://bugs.winehq.org/show_bug.cgi?id=29906

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Austin English  ---
wine-1.7.18-155-gdb42703 builds with clang-3.4 and no special settings, just:
./configure CC=clang

I also tried with 1.4-rc3 (which is what the bug report was from). I had to
disable freetype to get configure to pass, but the build later fails:
clang -c -I. -I. -I../../../include -I../../../include 
-DWINE_STRICT_PROTOTYPES -DWINE_NO_NAMELESS_EXTENSION  -D_REENTRANT -fPIC -Wall
-pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body
-Wignored-qualifiers -Wstrict-prototypes -Wtype-limits
-Wunused-but-set-parameter -Wwrite-strings -fno-omit-frame-pointer
-Wpointer-arith  -g -O2  -o tmarshal.o tmarshal.c
warning: unknown warning option '-Wunused-but-set-parameter'; did you mean
'-Wunused-parameter'? [-Wunknown-warning-option]
In file included from tmarshal.c:29:
./tmarshal.h:611:41: error: expected member name or ';' after declaration
specifiers
    HRESULT (STDMETHODCALLTYPE *restrict)(
    ~~~~~~~                             ^
./tmarshal.h:611:32: error: pointer to function type 'HRESULT (IWidget *, INT
*) __attribute__((stdcall))' may not be 'restrict' qualified
    HRESULT (STDMETHODCALLTYPE *restrict)(
                               ^
1 warning and 2 errors generated.
Makefile:194: recipe for target 'tmarshal.o' failed
make: *** [tmarshal.o] Error 1

Fixed in wine.

[Bug 29906] wine build broken on clang from trunk

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

Jerome Leclanche  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |8e14c9413b33dc47673d48d7ffc
                   |                            |f7e7ae5c51dbf
          Component|-unknown                    |oleaut32

[Bug 29906] wine build broken on clang from trunk

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

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.19.