----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118865/#review99470 -----------------------------------------------------------
Closing as this review request is more than 2 years old. If it still applies to current Plasma please reopen this review request. Thanks - David Edmundson On June 21, 2014, 2:26 p.m., José Manuel Santamaría Lema wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118865/ > ----------------------------------------------------------- > > (Updated June 21, 2014, 2:26 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > Hi, > > I have been working a bit in kubuntu's plasma packaging, when I started > plasma from a fresh new account I noticed I couldn't see most icons in the > GUI's except for a few ones. > > So I checked the starkde output to try to find out what was wrong, I found a > line like this one: > static QPlatformTheme* QKdeTheme::createKdeTheme(): Unable to determine > KDEHOME > > Digging a bit more into the issue I also found out where this message comes > from. > File src/platformsupport/themes/genericunix/qgenericunixthemes.cpp (Qt 5.3.0) > lines 446-468: > QPlatformTheme *QKdeTheme::createKdeTheme() > { > // Check for version >= 4 and determine home folder from environment, > // defaulting to ~/.kde<version>, ~/.kde > const QByteArray kdeVersionBA = qgetenv("KDE_SESSION_VERSION"); > const int kdeVersion = kdeVersionBA.toInt(); > if (kdeVersion < 4) > return 0; > const QString kdeHomePathVar = QString::fromLocal8Bit(qgetenv("KDEHOME")); > if (!kdeHomePathVar.isEmpty()) > return new QKdeTheme(kdeHomePathVar, kdeVersion); > > const QString kdeVersionHomePath = QDir::homePath() + > QStringLiteral("/.kde") + QLatin1String(kdeVersionBA); > if (QFileInfo(kdeVersionHomePath).isDir()) > return new QKdeTheme(kdeVersionHomePath, kdeVersion); > > const QString kdeHomePath = QDir::homePath() + QStringLiteral("/.kde"); > if (QFileInfo(kdeHomePath).isDir()) > return new QKdeTheme(kdeHomePath, kdeVersion); > > qWarning("%s: Unable to determine KDEHOME", Q_FUNC_INFO); > return 0; > } > > So I'm inclined to think the ~/.kde directory should be created if it doesn't > exist, thats what the patch does. What do you think? > > > Diffs > ----- > > startkde/startkde.cmake ea0bdfe > > Diff: https://git.reviewboard.kde.org/r/118865/diff/ > > > Testing > ------- > > Applied a similar patch in a customized kubuntu package. With the patch the > ~/.kde directory is created and the icons can be seen. > > > Thanks, > > José Manuel Santamaría Lema > >
