Re: [PATCH] Improve DejaVuSans detection

["Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1676 bytes --]

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
> 



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 274 bytes --]