D11447: No longer export QT_QPA_PLATFORM env variable to the session

[Fabian Vogt]

fvogt added a comment.


  In D11447#229334 <https://phabricator.kde.org/D11447#229334>, @graesslin wrote:
  
  > In D11447#229128 <https://phabricator.kde.org/D11447#229128>, @fvogt wrote:
  >
  > > If `KWorkSpace::detectPlatform(argc, argv);` were changed to edit `argc` and `argv` to pass `-platform foo` instead of setting `QT_QPA_PLATFORM`, the `qunsetenv` calls would become unnecessary.
  >
  >
  > but that sounds rather dangerous. I don't want to mess with the command line arguments
  
  
  Why is that more dangerous than environment variables?
  If argc and argv are passed to `QApplication`, you already know how it'll be parsed.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  unset-qt-qpa-platform

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

To: graesslin, #plasma, davidedmundson
Cc: fvogt, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180319/2e6e5bae/attachment.html>