117077 2013-05-31 08:37:20 -0700 [Qt] platformplugin install directory fix 2014-02-03 03:25:59 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID P2 Normal --- 0 mrjones webkit-unassigned oldest_to_newest 895898 0 mrjones 2013-05-31 08:37:20 -0700 platformplugin install directory fix 895904 1 203451 mrjones 2013-05-31 08:55:21 -0700 Created attachment 203451 Fixes build and install directory for platformplugin When building the platformplugin library the variable DESTDIR is defined. When Qt reads the platformplugin.pro file it defines DESTDIR in the Makefile. This has a couple of side affects. 1) The library libplatformplugin.so doesn't seem to be built when DESTDIR in the platformplugin.pro file is defined. 2) If the user has sufficient privileges (eg. root or sudo) then this would create an install directory in the location of $QT_INSTALL_PLUGINS/webkit with whatever location $QT_INSTALL_PLUGINS is defined as in the systems /. If the user tries to build to a temporary root (eg. /tmp) then this will ignore whatever INSTALL_ROOT or DESTDIR variable is given during the 'make install' phase. The patch adds the proper variables so that libplatformplugin.so gets built to the sources local 'release/' directory (or 'debug/' depending on build type) as that is the value of DESTDIR in the Makefile if it is not defined in platformplugin.pro. 975737 2 jturcotte 2014-02-03 03:25:59 -0800 === Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines. 203451 2013-05-31 08:55:21 -0700 2013-05-31 08:55:21 -0700 Fixes build and install directory for platformplugin platformplugin-install-directory-fix.patch text/plain 469 mrjones SW5kZXg6IFNvdXJjZS9XZWJLaXQvcXQvZXhhbXBsZXMvcGxhdGZvcm1wbHVnaW4vcGxhdGZvcm1w bHVnaW4ucHJvCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQvcXQvZXhhbXBsZXMvcGxhdGZv cm1wbHVnaW4vcGxhdGZvcm1wbHVnaW4ucHJvCShyZXZpc2lvbiAxNTA5OTcpCisrKyBTb3VyY2Uv V2ViS2l0L3F0L2V4YW1wbGVzL3BsYXRmb3JtcGx1Z2luL3BsYXRmb3JtcGx1Z2luLnBybwkod29y a2luZyBjb3B5KQpAQCAtMjAsNyArMjAsOCBAQAogICAgIH0KIH0KIAotREVTVERJUiA9ICQkW1FU X0lOU1RBTExfUExVR0lOU10vd2Via2l0Cit0YXJnZXQucGF0aCA9ICQkW1FUX0lOU1RBTExfUExV R0lOU10vd2Via2l0CitJTlNUQUxMUyArPSB0YXJnZXQKIAogU09VUkNFUyArPSBcCiAgICAgV2Vi UGx1Z2luLmNwcCBcCg==