Re: [PATCH] Improve DejaVuSans detection

[Vladimir 'φ-coder/phcoder' Serbinenko]

On 18.01.2014 16:58, Andrey Borzenkov wrote:
> Look also in /usr/share/fonts/truetype, it is installed in this path on
> openSUSE.
> 
> Prefer DejaVuSansMono if found, on openSUSE DejaVuSans is proportional font.
> 
> Add detected path name to summary output.
> 
> ---
>  configure.ac | 13 ++++++++-----
>  1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 2e4cf3c..98f6012 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1378,11 +1378,13 @@ fi
>  
>  if test x"$starfield_excuse" = x; then
>     for ext in pcf pcf.gz bdf bdf.gz ttf ttf.gz; do
> -     for dir in . /usr/src /usr/share/fonts/X11/misc 
> /usr/share/fonts/truetype/ttf-dejavu /usr/share/fonts/dejavu; do
> -        if test -f "$dir/DejaVuSans.$ext"; then
> -          DJVU_FONT_SOURCE="$dir/DejaVuSans.$ext"
> -          break 2
> -        fi
> +     for dir in . /usr/src /usr/share/fonts/X11/misc 
> /usr/share/fonts/truetype/ttf-dejavu /usr/share/fonts/dejavu 
> /usr/share/fonts/truetype; do
> +     for font in DejaVuSansMono DejaVuSans; do
> +       if test -f "$dir/$font.$ext"; then
> +         DJVU_FONT_SOURCE="$dir/$font.$ext"
> +         break 2
> +       fi
> +     done
It's fine to use SansMono for terminal (I'd prefer unifont though) but
this change also changes the way various items in menu are displayed.
Monospace fonts are less dense so allow less info in menu on small screens.
>       done
>     done
>  
> @@ -1805,6 +1807,7 @@ echo grub-mount: No "($grub_mount_excuse)"
>  fi
>  if [ x"$starfield_excuse" = x ]; then
>  echo starfield theme: Yes
> +echo With DejaVuSans font from $DJVU_FONT_SOURCE
>  else
>  echo starfield theme: No "($starfield_excuse)"
>  fi
>

signature.asc
Description: OpenPGP digital signature