Commit 3c1a34a2 authored by David Edmundson's avatar David Edmundson
Browse files

[startkde] Drop hack to manually set XDG_DATA_DIRS to the install root

Summary:
For normal distros it doesn't make sense, KDE_INSTALL_FULL_DATAROOTDIR
will be /usr/.

For devs using a prefix startplasma-dev.sh already does this for us only
for the dev setup.

For any other user, it should be their responsibility to adjust it.

It's currently broken anyway, if a user simply installs flatpak that sets this env
earlier in the boot, so this will do nothing. It implies no-oe relies on thisl

It's a relic from 2007 (bd38d59e11548f160efd0918ca5461536ac72349) that doesn't
make sense now.

BUG: 315452

Test Plan: Compiles

Reviewers: #plasma

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D26537
parent 4a65dcef
...@@ -196,14 +196,6 @@ void runEnvironmentScripts() ...@@ -196,14 +196,6 @@ void runEnvironmentScripts()
} }
} }
sourceFiles(scripts); sourceFiles(scripts);
// Make sure that the KDE prefix is first in XDG_DATA_DIRS and that it's set at all.
// The spec allows XDG_DATA_DIRS to be not set, but X session startup scripts tend
// to set it to a list of paths *not* including the KDE prefix if it's not /usr or
// /usr/local.
if (!qEnvironmentVariableIsSet("XDG_DATA_DIRS")) {
qputenv("XDG_DATA_DIRS", KDE_INSTALL_FULL_DATAROOTDIR ":/usr/share:/usr/local/share");
}
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment