Review Request 116625: Oxygen as default font

[Martin Gräßlin]

> On March 6, 2014, 12:34 p.m., Aleix Pol Gonzalez wrote:
> > Is the ttf's the sources of the fonts? We shouldn't distribute binaries, no?
> > 
> > OTOH, I would say it's fine installing the font from frameworks integration, although it makes little difference since frameworks integration will be offering whatever has been put in the configs.
> 
> Sebastian Kügler wrote:
>     There are sources for the font (in the oxygen-fonts package), but to my knowledge, the preferred format for fonts is actually ttf.
>     
>     This would also be hardcoded as default in the config reading.

> There are sources for the font (in the oxygen-fonts package), but to my knowledge, the preferred format for fonts is actually ttf.

That's why I suggested to fix the oxygen-font package and runtime depend through CMake. I really don't want us to include binaries in kde-workspace.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116625/#review52240
-----------------------------------------------------------


On March 6, 2014, 11:29 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, 11:29 a.m.)
> 
> 
> Review request for Plasma and Àlex Fiestas.
> 
> 
> 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/85065cfd/attachment.html>