On Thu, 2010-10-07 at 17:31 +0200, Michał Górny wrote:

> This option is useful to disable all possible font encodings at once,
> without fine-graining the calling ebuilds for a list of encodings
> supported by each font package.
> 
> The option is parsed before all other encoding options, so it
> basically
> sets defaults for all of them. Afterwards, any encoding can be
> re-enabled back using '--enable-<encoding>' (much like
> '--disable-all-encodings --enable-iso8859-2').


Forgive me as I am not familiar with the font module. If I understand,
all encodings are enabled by default,
which forced you to hunt down the list of encodings to disable. With
this patch, you can control the
"default" encoding enablement. You can have all encodings disabled by
default, allowing you
to only specify the ones you know and care about.

Have you tested permutations of enable/disable all-encodings and
enable/disable iso8859-2 for example?

This patch is modifying the public interface of the font-util package,
so it will require a version bump
and a release. The "Minimum version" in the patch comment need to
reflect this new value (1.2)

Any package using font-util and the new configure option will need to
upgrade this statement:

        m4_ifndef([XORG_FONT_MACROS_VERSION],
                  [m4_fatal([must install X.Org font-util 1.1 or later before 
running autoconf/autogen])])
        XORG_FONT_MACROS_VERSION(1.1)

Coincidentally, another font-util patch requiring a version bump has
been posted on the list.
http://lists.x.org/archives/xorg-devel/2010-October/013772.html
They should be part of the same version bump.

Acked-by: Gaetan Nadon <[email protected]>


Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to