This revision was automatically updated to reflect the committed changes.
Closed by commit R258:d8017f7b9744: Check if plymouth file is known to
update-alternatives before setting (authored by leszeklesner, committed by
aacid).
REPOSITORY
R258 Plymouth KCM
CHANGES SINCE LAST UPDATE
https://
leszeklesner updated this revision to Diff 14776.
leszeklesner added a comment.
Fixed qDebug typo.
REPOSITORY
R258 Plymouth KCM
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D5942?vs=14772&id=14776
REVISION DETAIL
https://phabricator.kde.org/D5942
AFFECTED FILES
src/helper.c
fvogt added inline comments.
INLINE COMMENTS
> helper.cpp:70
> +QByteArray data;
> +qDebug() << "Running update-initramfs --list default.plymouth now";
> +checkProcess.start("update-alternatives --list default.plymouth");
Should be `update-alternatives` here.
REPOSITORY
mart accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R258 Plymouth KCM
REVISION DETAIL
https://phabricator.kde.org/D5942
To: leszeklesner, mart
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, lu
leszeklesner created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
If the plymouth theme is not known to update-alternatives trying to execute
update-alternatives --set will always fail.
As plymouth kc
