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.
> 
It's not right solution. Proportional fonts must be preffered for
themes. Only terminal requires monospace fonts. You need to set
gfxterm_font to unifont name.
> 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
>       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