<?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>131228</bug_id>
          
          <creation_ts>2014-04-04 08:33:55 -0700</creation_ts>
          <short_desc>[GTK][CMAKE] build-webkit&apos;s parameter --no-webkit1 is not honored</short_desc>
          <delta_ts>2014-04-09 05:54:57 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</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="Víctor M. Jáquez L.">vjaquez</reporter>
          <assigned_to name="Víctor M. Jáquez L.">vjaquez</assigned_to>
          <cc>bunhere</cc>
    
    <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>mrobinson</cc>
    
    <cc>rakuco</cc>
    
    <cc>sergio</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>997573</commentid>
    <comment_count>0</comment_count>
    <who name="Víctor M. Jáquez L.">vjaquez</who>
    <bug_when>2014-04-04 08:33:55 -0700</bug_when>
    <thetext>When building webkit with

$ build-webkit --gtk --no-webkit1

GtkLaunch and the webkit1 libraries are built.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997616</commentid>
    <comment_count>1</comment_count>
      <attachid>228602</attachid>
    <who name="Víctor M. Jáquez L.">vjaquez</who>
    <bug_when>2014-04-04 10:34:23 -0700</bug_when>
    <thetext>Created attachment 228602
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997631</commentid>
    <comment_count>2</comment_count>
      <attachid>228602</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2014-04-04 10:55:36 -0700</bug_when>
    <thetext>Comment on attachment 228602
Patch

WebKit1 supports GTK3, so I&apos;m not sure I understand the patch. We are going to remove WebKit1 soon, so perhaps we can just close this bug and wait for it to go away?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997642</commentid>
    <comment_count>3</comment_count>
    <who name="Víctor M. Jáquez L.">vjaquez</who>
    <bug_when>2014-04-04 11:14:31 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 228602 [details])
&gt; WebKit1 supports GTK3, so I&apos;m not sure I understand the patch. We are going to remove WebKit1 soon, so perhaps we can just close this bug and wait for it to go away?

This patch removes the unconditional compilation of WebKit1 when compiling for GTK3.

Sure, we can close this bug and wait, or commit it and then removes its logic. Just I wanted to spot the issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>998929</commentid>
    <comment_count>4</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2014-04-08 17:47:37 -0700</bug_when>
    <thetext>WebKit1GTK+ has been removed.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>228602</attachid>
            <date>2014-04-04 10:34:23 -0700</date>
            <delta_ts>2014-04-09 05:54:57 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-131228-20140404173402.patch</filename>
            <type>text/plain</type>
            <size>1431</size>
            <attacher name="Víctor M. Jáquez L.">vjaquez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTY2Nzc2CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvT3B0
aW9uc0dUSy5jbWFrZSBiL1NvdXJjZS9jbWFrZS9PcHRpb25zR1RLLmNtYWtlCmluZGV4IGU1M2Rl
ZWRjZTc2OTJiYTE4NjkxOGMzNGM0ZmMzZDUzMjM1NGI4OWMuLjc4MTM4ZDQxYTk3Y2UyZDBlNmE1
YThjMTlmYzkyZTVmNzY4ZTU1ZTUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9jbWFrZS9PcHRpb25zR1RL
LmNtYWtlCisrKyBiL1NvdXJjZS9jbWFrZS9PcHRpb25zR1RLLmNtYWtlCkBAIC05Myw3ICs5Myw2
IEBAIFdFQktJVF9PUFRJT05fRU5EKCkKIAogc2V0KEVOQUJMRV9XRUJDT1JFIE9OKQogc2V0KEVO
QUJMRV9JTlNQRUNUT1IgT04pCi1zZXQoRU5BQkxFX1dFQktJVCBPTikKIGlmIChOT1QgVVNFX0dU
SzIpCiAgICAgc2V0KFdFQktJVEdUS19BUElfVkVSU0lPTiAzLjApCiAgICAgc2V0KEdUS19BUElf
VkVSU0lPTiAzLjApCkBAIC0xMDUsNiArMTA0LDcgQEAgZWxzZSAoKQogICAgIHNldChHVEtfQVBJ
X1ZFUlNJT04gMi4wKQogICAgIHNldChFTkFCTEVfUExVR0lOX1BST0NFU1MgT0ZGKQogICAgIHNl
dChFTkFCTEVfV0VCS0lUMiBPRkYpCisgICAgc2V0KEVOQUJMRV9XRUJLSVQgT04pCiBlbmRpZiAo
KQogCiAjIFRoZXNlIGFyZSB1c2VkIHRvIGdlbmVyYXRlIHRoZSBwa2ctY29uZmlnIGZpbGVzLgpk
aWZmIC0tZ2l0IGEvQ2hhbmdlTG9nIGIvQ2hhbmdlTG9nCmluZGV4IDYwMDAzMGRmZmNjMzVhYWUy
NmU5OGE0ODUxODdlMTA5YzQ5ODU4MDguLjc3Y2M1NzY4MGYzZTBiYWY4MDBkZDkwYTRhNjEzMzk2
MmJmNGFhMzAgMTAwNjQ0Ci0tLSBhL0NoYW5nZUxvZworKysgYi9DaGFuZ2VMb2cKQEAgLTEsMyAr
MSwxNiBAQAorMjAxNC0wNC0wNCAgVsOtY3RvciBNYW51ZWwgSsOhcXVleiBMZWFsICA8dmphcXVl
ekBpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdW0NNQUtFXSBidWlsZC13ZWJraXQncyBwYXJh
bWV0ZXIgLS1uby13ZWJraXQxIGlzIG5vdCBob25vcmVkCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMzEyMjgKKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBIb25vdXIgdGhlIHBhcmFtZXRlciAtLW5vLXdlYmtp
dDEuIFByZXZpb3VzbHkgV2ViS2l0MSB3YXMgYnVpbHQKKyAgICAgICAgdW5jb25kaXRpb25hbGx5
LgorCisgICAgICAgICogU291cmNlL2NtYWtlL09wdGlvbnNHVEsuY21ha2U6IGVuYWJsZSB3ZWJr
aXQxIG9ubHkgd2hlbiBidWlsZGluZworICAgICAgICB3aXRoIEdUSzIuCisKIDIwMTQtMDQtMDIg
IFJhcGhhZWwgS3VibyBkYSBDb3N0YSAgPHJhcGhhZWwua3Viby5kYS5jb3N0YUBpbnRlbC5jb20+
CiAKICAgICAgICAgW0dUS11bQ01ha2VdIExvb2sgZm9yIGdseC5oIGluIE9QRU5HTF9JTkNMVURF
X0RJUi4K
</data>

          </attachment>
      

    </bug>

</bugzilla>