<?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>113737</bug_id>
          
          <creation_ts>2013-04-01 18:15:04 -0700</creation_ts>
          <short_desc>Make feature_defines.sh work for VS2010</short_desc>
          <delta_ts>2013-04-01 22:41:16 -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>Platform</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows 7</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="Roger Fong">roger_fong</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>roger_fong</cc>
    
    <cc>thorton</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>867462</commentid>
    <comment_count>0</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2013-04-01 18:15:04 -0700</bug_when>
    <thetext>VS2010 build uses a new FeatureDefines.props file with a different format from the original FeatureDefines.vsprops.

The feature defines string outputted by feature-defines.sh needs to be corrected for this.

Also creating a feature_defines2010.sh because we still need both feature_defines scripts for now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867463</commentid>
    <comment_count>1</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2013-04-01 18:15:22 -0700</bug_when>
    <thetext>&lt;rdar://problem/13343147&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867466</commentid>
    <comment_count>2</comment_count>
      <attachid>196048</attachid>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2013-04-01 18:23:22 -0700</bug_when>
    <thetext>Created attachment 196048
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867478</commentid>
    <comment_count>3</comment_count>
      <attachid>196052</attachid>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2013-04-01 19:09:33 -0700</bug_when>
    <thetext>Created attachment 196052
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867480</commentid>
    <comment_count>4</comment_count>
      <attachid>196052</attachid>
    <who name="Tim Horton">thorton</who>
    <bug_when>2013-04-01 19:11:04 -0700</bug_when>
    <thetext>Comment on attachment 196052
patch

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

&gt; WebKitLibraries/win/tools/scripts/feature-defines2010.sh:3
&gt; +# Copyright (C) 2010 Apple Inc. All rights reserved.

2013</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>867556</commentid>
    <comment_count>5</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2013-04-01 22:41:16 -0700</bug_when>
    <thetext>committed: http://trac.webkit.org/changeset/147375</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>196048</attachid>
            <date>2013-04-01 18:23:22 -0700</date>
            <delta_ts>2013-04-01 19:09:33 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch.patch</filename>
            <type>text/plain</type>
            <size>2536</size>
            <attacher name="Roger Fong">roger_fong</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0NzM3MykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDEzLTA0LTAxICBSb2dlciBG
b25nICA8cm9nZXJfZm9uZ0BhcHBsZS5jb20+CisKKyAgICAgICAgVlMyMDEwIFdlYkNvcmVHZW5l
cmF0ZWQgYnVpbGQgc2NyaXB0cyBzaG91bGQgdXNlIG5ldyBmZWF0dXJlLWRlZmluZXMgc2NyaXB0
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTEzNzM3
LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogV2Vi
Q29yZS52Y3hwcm9qL2J1aWxkLWdlbmVyYXRlZC1maWxlcy5zaDoKKwogMjAxMy0wNC0wMSAgVGlt
IEhvcnRvbiAgPHRpbW90aHlfaG9ydG9uQGFwcGxlLmNvbT4KIAogICAgICAgICBBdXRvc2l6ZSBz
aG91bGQgdXNlIGRvY3VtZW50UmVjdCBoZWlnaHQgaW5zdGVhZCBvZiBzY3JvbGxIZWlnaHQKSW5k
ZXg6IFNvdXJjZS9XZWJDb3JlL1dlYkNvcmUudmN4cHJvai9idWlsZC1nZW5lcmF0ZWQtZmlsZXMu
c2gKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvV2ViQ29yZS52Y3hwcm9qL2J1aWxkLWdl
bmVyYXRlZC1maWxlcy5zaAkocmV2aXNpb24gMTQ3MzU4KQorKysgU291cmNlL1dlYkNvcmUvV2Vi
Q29yZS52Y3hwcm9qL2J1aWxkLWdlbmVyYXRlZC1maWxlcy5zaAkod29ya2luZyBjb3B5KQpAQCAt
NTksNSArNTksNSBAQAogY2QgIiR7QlVJTFRfUFJPRFVDVFNfRElSfS9EZXJpdmVkU291cmNlcyIK
IAogZXhwb3J0IFdlYkNvcmU9IiR7WFNSQ1JPT1R9IgotZXhwb3J0IEZFQVRVUkVfREVGSU5FUz1g
JFNES1JPT1QvdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMuc2ggJFNES1JPT1QgJDNgCitl
eHBvcnQgRkVBVFVSRV9ERUZJTkVTPWAkU0RLUk9PVC90b29scy9zY3JpcHRzL2ZlYXR1cmUtZGVm
aW5lczIwMTAuc2ggJFNES1JPT1QgJDNgCiBtYWtlIC1mICIkV2ViQ29yZS9EZXJpdmVkU291cmNl
cy5tYWtlIiAtaiAke05VTUNQVVN9IHx8IGV4aXQgMQpJbmRleDogV2ViS2l0TGlicmFyaWVzL0No
YW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRMaWJyYXJpZXMvQ2hhbmdlTG9nCShyZXZpc2lv
biAxNDczNzMpCisrKyBXZWJLaXRMaWJyYXJpZXMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTIgQEAKKzIwMTMtMDQtMDEgIFJvZ2VyIEZvbmcgIDxyb2dlcl9mb25nQGFwcGxl
LmNvbT4KKworICAgICAgICBVcGRhdGUgRkVBVFVSRV9ERUZJTkVTIHN0cmluZyBnZW5lcmF0aW9u
IGZvciBWUzIwMTAgc29sdXRpb24uCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD0xMTM3MzcuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgKiB3aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMuc2g6CisK
IDIwMTMtMDMtMjkgIFJvZ2VyIEZvbmcgIDxyb2dlcl9mb25nQGFwcGxlLmNvbT4KIAogICAgICAg
ICBVbnJldmlld2VkLiBSb2xsb3V0IHIxNDY4MTguCkluZGV4OiBXZWJLaXRMaWJyYXJpZXMvd2lu
L3Rvb2xzL3NjcmlwdHMvZmVhdHVyZS1kZWZpbmVzLnNoCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdExp
YnJhcmllcy93aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMuc2gJKHJldmlzaW9uIDE0
NzM1OCkKKysrIFdlYktpdExpYnJhcmllcy93aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmlu
ZXMuc2gJKHdvcmtpbmcgY29weSkKQEAgLTIzLDkgKzIzLDkgQEAKICMgU09GVFdBUkUsIEVWRU4g
SUYgQURWSVNFRCBPRiBUSEUgUE9TU0lCSUxJVFkgT0YgU1VDSCBEQU1BR0UuCiAKIGlmIFsgIiQy
IiA9ICJjYWlybyIgXTsgdGhlbgotICAgIEZlYXR1cmVEZWZpbmVzPSQxL3Rvb2xzL3ZzcHJvcHMv
RmVhdHVyZURlZmluZXNDYWlyby52c3Byb3BzCisgICAgRmVhdHVyZURlZmluZXM9JDEvdG9vbHMv
dnNwcm9wcy9GZWF0dXJlRGVmaW5lc0NhaXJvLnByb3BzCiBlbHNlCi0gICAgRmVhdHVyZURlZmlu
ZXM9JDEvdG9vbHMvdnNwcm9wcy9GZWF0dXJlRGVmaW5lcy52c3Byb3BzCisgICAgRmVhdHVyZURl
ZmluZXM9JDEvdG9vbHMvdnNwcm9wcy9GZWF0dXJlRGVmaW5lcy5wcm9wcwogZmkKIAogZ3JlcCBW
YWx1ZT1cIkVOQUJMRV8gJEZlYXR1cmVEZWZpbmVzIHwgc2VkICdzL14uKlZhbHVlXD1cIi8gLycg
fCBzZWQgJ3MvXCIvLycgfCB0ciAtZCAnXG4nCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>196052</attachid>
            <date>2013-04-01 19:09:33 -0700</date>
            <delta_ts>2013-04-01 19:11:04 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch.patch</filename>
            <type>text/plain</type>
            <size>3741</size>
            <attacher name="Roger Fong">roger_fong</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0NzM3MykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDEzLTA0LTAxICBSb2dlciBG
b25nICA8cm9nZXJfZm9uZ0BhcHBsZS5jb20+CisKKyAgICAgICAgVlMyMDEwIFdlYkNvcmVHZW5l
cmF0ZWQgYnVpbGQgc2NyaXB0cyBzaG91bGQgdXNlIG5ldyBmZWF0dXJlLWRlZmluZXMgc2NyaXB0
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTEzNzM3
LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogV2Vi
Q29yZS52Y3hwcm9qL2J1aWxkLWdlbmVyYXRlZC1maWxlcy5zaDoKKwogMjAxMy0wNC0wMSAgVGlt
IEhvcnRvbiAgPHRpbW90aHlfaG9ydG9uQGFwcGxlLmNvbT4KIAogICAgICAgICBBdXRvc2l6ZSBz
aG91bGQgdXNlIGRvY3VtZW50UmVjdCBoZWlnaHQgaW5zdGVhZCBvZiBzY3JvbGxIZWlnaHQKSW5k
ZXg6IFNvdXJjZS9XZWJDb3JlL1dlYkNvcmUudmN4cHJvai9idWlsZC1nZW5lcmF0ZWQtZmlsZXMu
c2gKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvV2ViQ29yZS52Y3hwcm9qL2J1aWxkLWdl
bmVyYXRlZC1maWxlcy5zaAkocmV2aXNpb24gMTQ3MzU4KQorKysgU291cmNlL1dlYkNvcmUvV2Vi
Q29yZS52Y3hwcm9qL2J1aWxkLWdlbmVyYXRlZC1maWxlcy5zaAkod29ya2luZyBjb3B5KQpAQCAt
NTksNSArNTksNSBAQAogY2QgIiR7QlVJTFRfUFJPRFVDVFNfRElSfS9EZXJpdmVkU291cmNlcyIK
IAogZXhwb3J0IFdlYkNvcmU9IiR7WFNSQ1JPT1R9IgotZXhwb3J0IEZFQVRVUkVfREVGSU5FUz1g
JFNES1JPT1QvdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMuc2ggJFNES1JPT1QgJDNgCitl
eHBvcnQgRkVBVFVSRV9ERUZJTkVTPWAkU0RLUk9PVC90b29scy9zY3JpcHRzL2ZlYXR1cmUtZGVm
aW5lczIwMTAuc2ggJFNES1JPT1QgJDNgCiBtYWtlIC1mICIkV2ViQ29yZS9EZXJpdmVkU291cmNl
cy5tYWtlIiAtaiAke05VTUNQVVN9IHx8IGV4aXQgMQpJbmRleDogV2ViS2l0TGlicmFyaWVzL0No
YW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRMaWJyYXJpZXMvQ2hhbmdlTG9nCShyZXZpc2lv
biAxNDczNzMpCisrKyBXZWJLaXRMaWJyYXJpZXMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTIgQEAKKzIwMTMtMDQtMDEgIFJvZ2VyIEZvbmcgIDxyb2dlcl9mb25nQGFwcGxl
LmNvbT4KKworICAgICAgICBVcGRhdGUgRkVBVFVSRV9ERUZJTkVTIHN0cmluZyBnZW5lcmF0aW9u
IGZvciBWUzIwMTAgc29sdXRpb24uCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD0xMTM3MzcuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgKiB3aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMyMDEwLnNo
OiBBZGRlZC4KKwogMjAxMy0wMy0yOSAgUm9nZXIgRm9uZyAgPHJvZ2VyX2ZvbmdAYXBwbGUuY29t
PgogCiAgICAgICAgIFVucmV2aWV3ZWQuIFJvbGxvdXQgcjE0NjgxOC4KSW5kZXg6IFdlYktpdExp
YnJhcmllcy93aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMyMDEwLnNoCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIFdlYktpdExpYnJhcmllcy93aW4vdG9vbHMvc2NyaXB0cy9mZWF0dXJlLWRlZmluZXMy
MDEwLnNoCShyZXZpc2lvbiAwKQorKysgV2ViS2l0TGlicmFyaWVzL3dpbi90b29scy9zY3JpcHRz
L2ZlYXR1cmUtZGVmaW5lczIwMTAuc2gJKHdvcmtpbmcgY29weSkKQEAgLTAsMCArMSwzMiBAQAor
IyEvdXNyL2Jpbi9iYXNoCisKKyMgQ29weXJpZ2h0IChDKSAyMDEwIEFwcGxlIEluYy4gQWxsIHJp
Z2h0cyByZXNlcnZlZC4KKyMKKyMgUmVkaXN0cmlidXRpb24gYW5kIHVzZSBpbiBzb3VyY2UgYW5k
IGJpbmFyeSBmb3Jtcywgd2l0aCBvciB3aXRob3V0CisjIG1vZGlmaWNhdGlvbiwgYXJlIHBlcm1p
dHRlZCBwcm92aWRlZCB0aGF0IHRoZSBmb2xsb3dpbmcgY29uZGl0aW9ucworIyBhcmUgbWV0Ogor
IyAxLiAgUmVkaXN0cmlidXRpb25zIG9mIHNvdXJjZSBjb2RlIG11c3QgcmV0YWluIHRoZSBhYm92
ZSBjb3B5cmlnaHQKKyMgICAgIG5vdGljZSwgdGhpcyBsaXN0IG9mIGNvbmRpdGlvbnMgYW5kIHRo
ZSBmb2xsb3dpbmcgZGlzY2xhaW1lci4KKyMgMi4gIFJlZGlzdHJpYnV0aW9ucyBpbiBiaW5hcnkg
Zm9ybSBtdXN0IHJlcHJvZHVjZSB0aGUgYWJvdmUgY29weXJpZ2h0CisjICAgICBub3RpY2UsIHRo
aXMgbGlzdCBvZiBjb25kaXRpb25zIGFuZCB0aGUgZm9sbG93aW5nIGRpc2NsYWltZXIgaW4gdGhl
CisjICAgICBkb2N1bWVudGF0aW9uIGFuZC9vciBvdGhlciBtYXRlcmlhbHMgcHJvdmlkZWQgd2l0
aCB0aGUgZGlzdHJpYnV0aW9uLgorIyAKKyMgVEhJUyBTT0ZUV0FSRSBJUyBQUk9WSURFRCBCWSBB
UFBMRSBJTkMuIEFORCBJVFMgQ09OVFJJQlVUT1JTIGBgQVMgSVMnJyBBTkQgQU5ZCisjIEVYUFJF
U1MgT1IgSU1QTElFRCBXQVJSQU5USUVTLCBJTkNMVURJTkcsIEJVVCBOT1QgTElNSVRFRCBUTywg
VEhFIElNUExJRUQKKyMgV0FSUkFOVElFUyBPRiBNRVJDSEFOVEFCSUxJVFkgQU5EIEZJVE5FU1Mg
Rk9SIEEgUEFSVElDVUxBUiBQVVJQT1NFIEFSRQorIyBESVNDTEFJTUVELiBJTiBOTyBFVkVOVCBT
SEFMTCBBUFBMRSBJTkMuIE9SIElUUyBDT05UUklCVVRPUlMgQkUgTElBQkxFIEZPUiBBTlkKKyMg
RElSRUNULCBJTkRJUkVDVCwgSU5DSURFTlRBTCwgU1BFQ0lBTCwgRVhFTVBMQVJZLCBPUiBDT05T
RVFVRU5USUFMIERBTUFHRVMKKyMgKElOQ0xVRElORywgQlVUIE5PVCBMSU1JVEVEIFRPLCBQUk9D
VVJFTUVOVCBPRiBTVUJTVElUVVRFIEdPT0RTIE9SIFNFUlZJQ0VTOworIyBMT1NTIE9GIFVTRSwg
REFUQSwgT1IgUFJPRklUUzsgT1IgQlVTSU5FU1MgSU5URVJSVVBUSU9OKSBIT1dFVkVSIENBVVNF
RCBBTkQgT04KKyMgQU5ZIFRIRU9SWSBPRiBMSUFCSUxJVFksIFdIRVRIRVIgSU4gQ09OVFJBQ1Qs
IFNUUklDVCBMSUFCSUxJVFksIE9SIFRPUlQKKyMgKElOQ0xVRElORyBORUdMSUdFTkNFIE9SIE9U
SEVSV0lTRSkgQVJJU0lORyBJTiBBTlkgV0FZIE9VVCBPRiBUSEUgVVNFIE9GIFRISVMKKyMgU09G
VFdBUkUsIEVWRU4gSUYgQURWSVNFRCBPRiBUSEUgUE9TU0lCSUxJVFkgT0YgU1VDSCBEQU1BR0Uu
CisKK2lmIFsgIiQyIiA9ICJjYWlybyIgXTsgdGhlbgorICAgIEZlYXR1cmVEZWZpbmVzPSQxL3Rv
b2xzL3ZzcHJvcHMvRmVhdHVyZURlZmluZXNDYWlyby5wcm9wcworZWxzZQorICAgIEZlYXR1cmVE
ZWZpbmVzPSQxL3Rvb2xzL3ZzcHJvcHMvRmVhdHVyZURlZmluZXMucHJvcHMKK2ZpCisKK2dyZXAg
IjxFTkFCTEVfIiAkRmVhdHVyZURlZmluZXMgfCBzZWQgJy9cLz4vZCcgfCBzZWQgJ3MvPFwvLio+
Ly8nIHwgc2VkICdzLzwuKj4vLycgfCB0ciAtZCAnXG4nCisK
</data>
<flag name="review"
          id="218036"
          type_id="1"
          status="+"
          setter="thorton"
    />
          </attachment>
      

    </bug>

</bugzilla>