<?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>37211</bug_id>
          
          <creation_ts>2010-04-07 08:46:16 -0700</creation_ts>
          <short_desc>[Qt] QtWebKit-2.0 needs to install a .prf file and add itself to the QTLIB, QT_CONFIG, and etc.</short_desc>
          <delta_ts>2011-02-28 09:33:37 -0800</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>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>53916</dup_id>
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Rodrigo Belem">rclbelem</reporter>
          <assigned_to name="Rodrigo Belem">rclbelem</assigned_to>
          <cc>ademar</cc>
    
    <cc>diegohcg</cc>
    
    <cc>hausmann</cc>
    
    <cc>jturcotte</cc>
    
    <cc>kenneth</cc>
    
    <cc>kristian.amlie</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>tonikitoo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>209652</commentid>
    <comment_count>0</comment_count>
    <who name="Rodrigo Belem">rclbelem</who>
    <bug_when>2010-04-07 08:46:16 -0700</bug_when>
    <thetext>The QtWebKit-2.0 (as a standalone package) needs to tell to qt where it is installed, its dependencies and that it is a qt lib, so it is possible for qt applications to add &quot;QT += webkit&quot; in the .pro files.

Currently it is hard-coded in the following files. 

qt-src/mkspecs/qconfig.pri -&gt; line 5
qt-src/mkspecs/features/qt.prf -&gt; lines 39, 69
qt-src/mkspecs/features/symbian/qt.prf -&gt; line 33

It is needed to take this responsibility to QtWebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>210641</commentid>
    <comment_count>1</comment_count>
    <who name="Jocelyn Turcotte">jturcotte</who>
    <bug_when>2010-04-09 02:05:40 -0700</bug_when>
    <thetext>The mechanism we want to use for 2.0 is to install over Qt&apos;s own install directory if you want the Qt += webkit syntax in your application&apos;s pro file.
Developers wanting to build against a QtWebKit outside of Qt&apos;s SDK can use INCLUDE+=... and LIBS+=... (though this is currenly tedious because of different binary file names on Windows)

One of the constraints why its difficult for QtWebKit to take responsibility of this is that qmake is bound to the SDK where it resides and that it is not clear which QtWebKit should be used when Qt += webkit is encountered.

The mechanism to compile Qt modules outside of Qt&apos;s source tree is still undetermined and should be looked at in the near future releases of Qt.
Until then we wanted to keep it simple for QtWebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>222308</commentid>
    <comment_count>2</comment_count>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2010-05-07 06:14:21 -0700</bug_when>
    <thetext>Belem, are you working on this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>222315</commentid>
    <comment_count>3</comment_count>
    <who name="Rodrigo Belem">rclbelem</who>
    <bug_when>2010-05-07 06:25:10 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; Belem, are you working on this?

I started to work on it and I made some changes to the qt prf files to support the possibility of adding an &quot;external&quot; qt module, but stopped to work on other stuff and I do not know if the qt guys have interest in this feature.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>222363</commentid>
    <comment_count>4</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-05-07 07:54:30 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; Belem, are you working on this?
&gt; 
&gt; I started to work on it and I made some changes to the qt prf files to support
&gt; the possibility of adding an &quot;external&quot; qt module, but stopped to work on other
&gt; stuff and I do not know if the qt guys have interest in this feature.

I don&apos;t think we want external module support right now. We need to get the module support &quot;inside&quot; working first :)

Removing from release blocker.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>358619</commentid>
    <comment_count>5</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2011-02-26 18:15:30 -0800</bug_when>
    <thetext>+Kristian. Is this bug obsoleted by the work at BUG 53916 ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>358910</commentid>
    <comment_count>6</comment_count>
    <who name="Kristian Amlie">kristian.amlie</who>
    <bug_when>2011-02-27 23:53:29 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; +Kristian. Is this bug obsoleted by the work at BUG 53916 ?

Yes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359113</commentid>
    <comment_count>7</comment_count>
    <who name="Ademar Reis">ademar</who>
    <bug_when>2011-02-28 09:33:37 -0800</bug_when>
    <thetext>(In reply to comment #6)
&gt; (In reply to comment #5)
&gt; &gt; +Kristian. Is this bug obsoleted by the work at BUG 53916 ?
&gt; 
&gt; Yes.

Closing as duplicate then.

*** This bug has been marked as a duplicate of bug 53916 ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>