Bug 57621

Summary: [Qt] Install targets are not working correctly for modularized Qt and QtWebkit
Product: WebKit Reporter: Kristian Amlie <kristian.amlie>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Severity: Major CC: abarth, ademar, benjamin, cmarcelo, commit-queue, eric, menard, ossy, webkit.review.bot
Priority: P3 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Description Flags
Same-patch-with-whitespace-changes-removed none

Description Kristian Amlie 2011-04-01 06:01:25 PDT
See attached patch for details regarding the fix.
Comment 1 Kristian Amlie 2011-04-01 06:03:12 PDT
Created attachment 87849 [details]
Comment 2 Kristian Amlie 2011-04-01 06:04:47 PDT
Created attachment 87850 [details]

For easier reading, here is the same patch with all whitespace-only changes removed.
Comment 3 Benjamin Poulain 2011-04-05 05:43:30 PDT
Anyone has any idea if this is correct? My knowledge of qmake magic is limited :(
Comment 4 Kristian Amlie 2011-04-08 06:04:36 PDT
FYI, I will be away for the next two weeks, so I will not respond during that time.
Comment 5 Eric Seidel (no email) 2011-04-10 15:50:13 PDT
This seems like an easy review for a qt reviewer.  I think you meant to obsolete the first patch and mark the second one for review/cq.
Comment 6 Alexis Menard (darktears) 2011-04-11 06:53:21 PDT
Down-prioritize to P3 because the modularization project is a bit unknown state to us. We can revisit those patches after QtWebKit will find its way to Qt source tree.
Comment 7 Simon Hausmann 2011-04-29 05:03:02 PDT
Comment on attachment 87849 [details]

r=me . Holding off the cq+ per Alexis' comment.
Comment 8 Caio Marcelo de Oliveira Filho 2011-05-05 08:28:42 PDT
Why not removing the other references to QTDIR_build then?
Comment 9 WebKit Commit Bot 2011-05-05 13:14:04 PDT
Comment on attachment 87849 [details]

Clearing flags on attachment: 87849

Committed r85870: <http://trac.webkit.org/changeset/85870>
Comment 10 WebKit Commit Bot 2011-05-05 13:14:09 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 WebKit Review Bot 2011-05-05 16:06:43 PDT
http://trac.webkit.org/changeset/85870 might have broken GTK Linux 32-bit Debug
The following tests are not passing:
Comment 12 Ademar Reis 2011-05-10 13:31:22 PDT
Revision r85870 cherry-picked into qtwebkit-2.2 with commit b8f0053 <http://gitorious.org/webkit/qtwebkit/commit/b8f0053>