----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116625/#review52225 -----------------------------------------------------------
I don't like that the fonts are copied as binaries into our repository. I would prefer that oxygen-fonts is adjusted to: * have a CMake system * install a OxygenFontsConfig.cmake * find it in kde-workspace * make it a RUNTIME dependency - Martin Gräßlin On March 6, 2014, 2:08 a.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116625/ > ----------------------------------------------------------- > > (Updated March 6, 2014, 2:08 a.m.) > > > Review request for Plasma. > > > Repository: kde-workspace > > > Description > ------- > > Make Oxygen the default font > > This has three portions: the fonts itself, installation of them including > fontconfig bits, and setting font defaults in kdeglobals. > > Oxygen Font 0.4 Imported from oxygen-font repository. > > The startkde portion contains the bits to write out a kdeglobals default > file if it doesn't exist with font settings applied. Usually, with install > prefix set to /usr, the installed oxygen font is found automatically by > fontconfig. If we're installed to a different prefix, we need to point > fontconfig at the font. We do that by linking it from either > XDG_DATA_HOME or ~/.fonts/ and updating fontconfig with it. The latter is > irrelevant for systems that install into /usr. > > > Diffs > ----- > > CMakeLists.txt 0135bb1f7475862451775928adf5dc20167424e0 > fonts/CMakeLists.txt PRE-CREATION > fonts/oxygen/COPYING-GPL+FE.txt PRE-CREATION > fonts/oxygen/GPL.txt PRE-CREATION > fonts/oxygen/Oxygen-Sans-Bold.ttf PRE-CREATION > fonts/oxygen/Oxygen-Sans.ttf PRE-CREATION > fonts/oxygen/OxygenMono-Regular.ttf PRE-CREATION > fonts/oxygen/README PRE-CREATION > startkde.cmake 53d1bc6a2098f634c5f386f95e1a1c504b554303 > > Diff: https://git.reviewboard.kde.org/r/116625/diff/ > > > Testing > ------- > > Logged into session with clean config. Font settings are applied correctly > throughout. > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
