D14124: Fix qtplugins in KStandardDirs

2018-07-15 Thread Antonio Rojas
This revision was automatically updated to reflect the committed changes.
Closed by commit R239:3ab2e2f5592d: Fix qtplugins in KStandardDirs (authored by 
arojas).

REPOSITORY
  R239 KDELibs4Support

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D14124?vs=37785=37794

REVISION DETAIL
  https://phabricator.kde.org/D14124

AFFECTED FILES
  src/kdecore/kstandarddirs.cpp

To: arojas, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D14124: Fix qtplugins in KStandardDirs

2018-07-15 Thread David Faure
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  Interestingly it works for me with and without this patch, because of 
"%lib/plugins/" later on in the code, but indeed at that point installdir is 
/d/kde/inst/kde_frameworks/lib64/plugins/plugins/ which looks wrong.

REPOSITORY
  R239 KDELibs4Support

REVISION DETAIL
  https://phabricator.kde.org/D14124

To: arojas, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D14124: Fix qtplugins in KStandardDirs

2018-07-15 Thread Antonio Rojas
arojas created this revision.
arojas added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
arojas requested review of this revision.

REVISION SUMMARY
  PLUGIN_INSTALL_DIR already contains the /plugins/ suffix, don't duplicate it

TEST PLAN
  'kf5-config --path qtplugins' returns the right thing

REPOSITORY
  R239 KDELibs4Support

REVISION DETAIL
  https://phabricator.kde.org/D14124

AFFECTED FILES
  src/kdecore/kstandarddirs.cpp

To: arojas, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns