[Bug 26751] New: configure improperly detects Wunused-but-set-parameter as supported on Clang

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

Summary: configure improperly detects Wunused-but-set-parameter
                    as supported on Clang
           Product: Wine
           Version: 1.3.17
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Keywords: download, source
          Severity: enhancement
          Priority: P2
         Component: build-env
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

configure:12269: clang -o conftest -std=gnu89 -g -Wunused-but-set-parameter  
conftest.c  >&5
warning: unknown warning option '-Wunused-but-set-parameter'
[-Wunknown-warning-option]
1 warning generated.
configure:12269: $? = 0
configure:12278: result: yes

It seems the WINE_TRY_CFLAGS check fails on clang, because clang warns when an
unused option is given, rather than considering it fatal as gcc does.

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

Frédéric Delanoy  changed:

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

--- Comment #1 from Frédéric Delanoy  2011-05-14 07:24:53 CDT ---
Clang workaround: see
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20110509/041838.h...

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

Frédéric Delanoy  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |http://llvm.org/bugs/show_b
                   |                            |ug.cgi?id=9701

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

Jeremy Huddleston  changed:

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

--- Comment #2 from Jeremy Huddleston  2013-05-13 00:44:29 CDT ---
Just use -Werror=unknown-warning-option or -Werror=unused-command-line-argument
like X.org does in XORG_TESTSET_CFLAG

http://cgit.freedesktop.org/xorg/util/macros/tree/xorg-macros.m4.in

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

André H.  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |7f810293b762912a4d956d6945b
                   |                            |fb6815f90e715
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
         Resolution|                            |FIXED

--- Comment #3 from André H.  2013-05-21 14:18:37 CDT ---
should be fixed by
http://source.winehq.org/git/wine.git/commitdiff/7f810293b762912a4d956d6945b...

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

--- Comment #4 from Austin English  2013-05-21 14:28:20 CDT ---
(In reply to comment #3)
> should be fixed by
> http://source.winehq.org/git/wine.git/commitdiff/7f810293b762912a4d956d6945b...

Yes, it is. Thanks AJ!

[Bug 26751] configure improperly detects Wunused-but-set-parameter as supported on Clang

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

Alexandre Julliard  changed:

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

--- Comment #5 from Alexandre Julliard  2013-05-24 13:33:05 CDT ---
Closing bugs fixed in 1.5.31.