Bug 117077

Summary: [Qt] platformplugin install directory fix
Product: WebKit Reporter: Matthew Jones <mrjones>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fixes build and install directory for platformplugin none

Matthew Jones
Reported 2013-05-31 08:37:20 PDT
platformplugin install directory fix
Attachments
Fixes build and install directory for platformplugin (469 bytes, patch)
2013-05-31 08:55 PDT, Matthew Jones
no flags
Matthew Jones
Comment 1 2013-05-31 08:55:21 PDT
Created attachment 203451 [details] 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.
Jocelyn Turcotte
Comment 2 2014-02-03 03:25:59 PST
=== 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.
Note You need to log in before you can comment on or make changes to this bug.