Review Request 116625: Oxygen as default font

Martin Gräßlin mgraesslin at kde.org
Thu Mar 6 06:23:07 UTC 2014


-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140306/91f1318a/attachment.html>


More information about the Plasma-devel mailing list