<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>172750</bug_id>
          
          <creation_ts>2017-05-31 07:56:06 -0700</creation_ts>
          <short_desc>[CMake] Do not pass -Wno-expansion-to-defined to Clang &lt;4.0.0</short_desc>
          <delta_ts>2017-05-31 12:54:27 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>Other</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adrian Perez">aperez</reporter>
          <assigned_to name="Adrian Perez">aperez</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>clopez</cc>
    
    <cc>lforschler</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1314154</commentid>
    <comment_count>0</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2017-05-31 07:56:06 -0700</bug_when>
    <thetext>As per Carlos López&apos;s comment on bug #167643:

&gt; I get thousands of warnings printed like this:
&gt; warning: unknown warning option &apos;-Wno-expansion-to-defined&apos;; did you mean &apos;-Wno-macro-redefined&apos;? [-Wunknown-warning-option]
&gt;
&gt; Using clang-3.8 with ccache.

With Clang 4.0 there is no such warning.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1314157</commentid>
    <comment_count>1</comment_count>
      <attachid>311589</attachid>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2017-05-31 08:10:19 -0700</bug_when>
    <thetext>Created attachment 311589
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1314274</commentid>
    <comment_count>2</comment_count>
      <attachid>311589</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2017-05-31 12:44:03 -0700</bug_when>
    <thetext>Comment on attachment 311589
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=311589&amp;action=review

&gt; Source/cmake/OptionsCommon.cmake:37
&gt; +    if (NOT(COMPILER_IS_CLANG AND &quot;${CLANG_VERSION}&quot; VERSION_LESS 4.0.0))

Style: add one space after the NOT</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1314279</commentid>
    <comment_count>3</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2017-05-31 12:54:27 -0700</bug_when>
    <thetext>Committed r217618: &lt;http://trac.webkit.org/changeset/217618&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>311589</attachid>
            <date>2017-05-31 08:10:19 -0700</date>
            <delta_ts>2017-05-31 12:44:03 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-172750-20170531181018.patch</filename>
            <type>text/plain</type>
            <size>1778</size>
            <attacher name="Adrian Perez">aperez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjE3NjAxCmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvT3B0
aW9uc0NvbW1vbi5jbWFrZSBiL1NvdXJjZS9jbWFrZS9PcHRpb25zQ29tbW9uLmNtYWtlCmluZGV4
IDU1ZTc0MDQ1ZWQ1NDA2ZGE5ODQ1OGQ4ODQxNmI0YWNmYjExN2Q1ZjQuLjNhNDcyNDM2NjBhZWQ1
MmM3ZDgwNzdlZDc5NWM1MDdhMTUxZjAzNWQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9jbWFrZS9PcHRp
b25zQ29tbW9uLmNtYWtlCisrKyBiL1NvdXJjZS9jbWFrZS9PcHRpb25zQ29tbW9uLmNtYWtlCkBA
IC0zMSwxMSArMzEsMTMgQEAgc2V0X3Byb3BlcnR5KEdMT0JBTCBQUk9QRVJUWSBVU0VfRk9MREVS
UyBPTikKIGRlZmluZV9wcm9wZXJ0eShUQVJHRVQgUFJPUEVSVFkgRk9MREVSIElOSEVSSVRFRCBC
UklFRl9ET0NTICJmb2xkZXIiIEZVTExfRE9DUyAiSURFIGZvbGRlciBuYW1lIikKIAogaWYgKENP
TVBJTEVSX0lTX0dDQ19PUl9DTEFORykKLSAgICBzZXQoQ01BS0VfQ19GTEFHUyAiJHtDTUFLRV9D
X0ZMQUdTfSAtV25vLWV4cGFuc2lvbi10by1kZWZpbmVkIikKICAgICBzZXQoQ01BS0VfQ19GTEFH
UyAiJHtDTUFLRV9DX0ZMQUdTfSAtZm5vLWV4Y2VwdGlvbnMgLWZuby1zdHJpY3QtYWxpYXNpbmci
KQotICAgIHNldChDTUFLRV9DWFhfRkxBR1MgIiR7Q01BS0VfQ1hYX0ZMQUdTfSAtV25vLWV4cGFu
c2lvbi10by1kZWZpbmVkIikKICAgICBzZXQoQ01BS0VfQ1hYX0ZMQUdTICIke0NNQUtFX0NYWF9G
TEFHU30gLWZuby1leGNlcHRpb25zIC1mbm8tc3RyaWN0LWFsaWFzaW5nIC1mbm8tcnR0aSIpCiAg
ICAgc2V0KENNQUtFX0NYWF9GTEFHUyAiJHtDTUFLRV9DWFhfRkxBR1N9IC1zdGQ9YysrMXkiKQor
ICAgIGlmIChOT1QoQ09NUElMRVJfSVNfQ0xBTkcgQU5EICIke0NMQU5HX1ZFUlNJT059IiBWRVJT
SU9OX0xFU1MgNC4wLjApKQorICAgICAgICBzZXQoQ01BS0VfQ19GTEFHUyAiJHtDTUFLRV9DX0ZM
QUdTfSAtV25vLWV4cGFuc2lvbi10by1kZWZpbmVkIikKKyAgICAgICAgc2V0KENNQUtFX0NYWF9G
TEFHUyAiJHtDTUFLRV9DWFhfRkxBR1N9IC1Xbm8tZXhwYW5zaW9uLXRvLWRlZmluZWQiKQorICAg
IGVuZGlmICgpCiBlbmRpZiAoKQogCiBpZiAoQ01BS0VfR0VORVJBVE9SIFNUUkVRVUFMICJOaW5q
YSIpCmRpZmYgLS1naXQgYS9DaGFuZ2VMb2cgYi9DaGFuZ2VMb2cKaW5kZXggMjI0ZDA3ZDZmOGQ4
YzhjNTFkYWNiMDk4OWM3ZWE1YjdjMmQ4YmM5MS4uNTQ2ZmExMDIyY2U1OTY0NDQ0ZGQyYjI2NDkx
NzcxM2E4ZmY3ODRjZiAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAt
MSw1ICsxLDE0IEBACiAyMDE3LTA1LTMxICBBZHJpYW4gUGVyZXogZGUgQ2FzdHJvICA8YXBlcmV6
QGlnYWxpYS5jb20+CiAKKyAgICAgICAgW0NNYWtlXSBEbyBub3QgcGFzcyAtV25vLWV4cGFuc2lv
bi10by1kZWZpbmVkIHRvIENsYW5nIDw0LjAuMAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTcyNzUwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgKiBTb3VyY2UvY21ha2UvT3B0aW9uc0NvbW1vbi5jbWFrZToK
KworMjAxNy0wNS0zMSAgQWRyaWFuIFBlcmV6IGRlIENhc3RybyAgPGFwZXJlekBpZ2FsaWEuY29t
PgorCiAgICAgICAgIENvbnNpZGVyIGVuYWJsaW5nIC1Xbm8tZXhwYW5zaW9uLXRvLWRlZmluZWQg
Zm9yIGdjYyA3CiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD0xNjc2NDMKIAo=
</data>
<flag name="review"
          id="332557"
          type_id="1"
          status="+"
          setter="mcatanzaro"
    />
          </attachment>
      

    </bug>

</bugzilla>