|Summary:||[Qt] Install targets are not working correctly for modularized Qt and QtWebkit|
|Product:||WebKit||Reporter:||Kristian Amlie <kristian.amlie>|
|Component:||WebKit Qt||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Major||CC:||abarth, ademar, benjamin, cmarcelo, commit-queue, eric, menard, ossy, webkit.review.bot|
|Version:||528+ (Nightly build)|
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] Install-correct-header-files-for-webkit
Comment 2 Kristian Amlie 2011-04-01 06:04:47 PDT
Created attachment 87850 [details] Same-patch-with-whitespace-changes-removed 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] Install-correct-header-files-for-webkit 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] Install-correct-header-files-for-webkit 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: svg/W3C-SVG-1.1/animate-elem-46-t.svg
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>