D16530: Drop X11 core fonts code from startkde/startplasma

2018-11-08 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R120:5374513266ee: Drop X11 core fonts code from startkde/startplasma (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16530?vs=44499&id=4515

D16530: Drop X11 core fonts code from startkde/startplasma

2018-11-08 Thread David Edmundson
davidedmundson added a comment. It's always tricky when you remove old stuff; we don't want to risk any changes that could affect a usecase no matter how obscure. It's not something to rush or do lightly. However, your rationale makes sense. There's an additional pieces of information

D16530: Drop X11 core fonts code from startkde/startplasma

2018-11-08 Thread Stefan Brüns
bruns added a comment. In D16530#355078 , @fvogt wrote: > Looks good to me, but I don't know enough about old X11 stuff like this to accept it. @fvogt - maybe you can poke a coworker? Someone else? REPOSITORY R120 Plasma Workspace

D16530: Drop X11 core fonts code from startkde/startplasma

2018-11-06 Thread Fabian Vogt
fvogt added a comment. Looks good to me, but I don't know enough about old X11 stuff like this to accept it. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D16530 To: bruns, #plasma, fvogt, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezha

D16530: Drop X11 core fonts code from startkde/startplasma

2018-11-06 Thread Stefan Brüns
bruns added a comment. Friendly reminder ... REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D16530 To: bruns, #plasma, fvogt, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16530: Drop X11 core fonts code from startkde/startplasma

2018-10-30 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Plasma, fvogt. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Qt and GTK based applications have switched to using fontconfig for fonts exclusively, so this c