Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 23 Jul 2016 16:51:09 +0000
From:      bugzilla-noreply@freebsd.org
To:        gnome@FreeBSD.org
Subject:   maintainer-approval requested: [Bug 211201] print/freetype2: update to 2.6.5, replace LCD_FILTERING with V40 code : [Attachment 172900] Proposed patch (since 412348 revision)
Message-ID:  <bug-211201-6497-HlxkUUiEGQ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-211201-6497@https.bugs.freebsd.org/bugzilla/>
References:  <bug-211201-6497@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
lightside <lightside@gmx.com> has asked gnome@FreeBSD.org for
maintainer-approval:
Bug 211201: print/freetype2: update to 2.6.5, replace LCD_FILTERING with V40
code
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D211201

Attachment 172900: Proposed patch (since 412348 revision)
https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D172900&action=3Dedit



--- Comment #25 from lightside <lightside@gmx.com> ---
Created attachment 172900
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D172900&action=
=3Dedit
Proposed patch (since 412348 revision)

(In reply to comment #23)
> SUBPIXEL_HINTING_MODE is unnecessary distraction in your case.
> The variable can still be kept for bmake simplification.
Looks like, I was right about OPTIONS_RADIO instead of OPTIONS_GROUP. I ren=
amed
BOTH option to V38_V40. The (1 | 2) =3D 3, by the way.
If you not agree, better to propose your version, I think. It may be usable=
 in
case of 3 or more such modes (excluding overall mode).

(In reply to comment #24)
> freetype2 is mainly configured via fontconfig, not via environment variab=
les.
> With V38 and V40 both OFF by default CONFIGURABLE maybe of debugging value
only.
Ok, I removed CONFIGURABLE option.

Thanks for reviews.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-211201-6497-HlxkUUiEGQ>