Bug 60312 - DIST_SUBDIRS should never appear in a conditional

As it is documented like this:

If SUBDIRS is defined conditionally using Automake conditionals,
Automake will define DIST_SUBDIRS automatically from the possible
values of SUBDIRS in all conditions.

So we don't need to re-define DIST_SUBDIRS in Makefile.am unless
we use AC_SUBST to define SUBDIRS.

Patch from Quentin Glidic

Bug 50733 - Add font-file hash?

Add "hash" object which contains SHA256 hash value (so far) computed from the font file.

Use AM_MISSING_PROG instead of hardcoding missing

Revert "test: Use SH_LOG_COMPILER and AM_TESTS_ENVIRONMENT"

This reverts commit 2146b0307a3476892723104481f27f8484451c52.

That change introduces incompatibility and seems not working with
older releases of automake, including automake 1.12.2.

test: Use SH_LOG_COMPILER and AM_TESTS_ENVIRONMENT

TESTS_ENVIRONMENT is deprecated and should be reserved to the user to
override the test environment

<ext>_LOG_COMPILER is meant to contain the program that runs the test
with <ext> extension
AM_TESTS_ENVIRONMENT is meant to set the environment for the tests

  https://bugs.freedesktop.org/show_bug.cgi?id=60192

Signed-off-by: Quentin Glidic <sardemff7+git@sardemff7.net>

Use AM_MISSING_PROG instead of hardcoding missing

Modernize configure.ac

Bug 23757 - Add mode="delete" to <edit>

Add two edit mode, "delete" and "delete_all".
what values are being deleted depends on <test> as documented.
if the target object is same to what is tested, matching value there
will be deleted. otherwise all of values in the object will be deleted.
so this would means both edit mode will not take any expressions.

e.g.

Given that the testing is always true here, the following rules:

  <match>
    <test name="foo" compare="eq">
      <string>bar</string>
    </test>
    <edit name="foo" mode="delete"/>
  </match>

will removes "bar" string from "foo" object. and:

  <match>
    <test name="foo" compare="eq">
      <string>foo</string>
    </test>
    <edit name="bar" mode="delete"/>
  </match>

will removes all of values in "bar" object.

build: Only use PKG_INSTALLDIR if available

It's only in pkg-config 0.27 or newer, but 0.25 at least is still
fairly widespread.

Use the PKG_INSTALLDIR macro.

modified:   Makefile.am
modified:   configure.ac

Bug 59385 - Do the right thing for intermixed edit and test elements

This changes allows to have multiple mathcing rules in one <match> block
in the same order.
After this changes, the following thing will works as two matching rules:

<match>
<!-- rule 1 -->
<test name="family" compare="eq">
<string>foo</string>
</test>
<edit name="foo" mode="append">
<string>foo</string>
</edit>
<!-- rule 2 -->
<test name="foo" compare="eq">
<string>foo</string>
</test>
<edit name="foo" mode="append">
<string>bar</string>
</edit>
</match>

remove the unnecessary code

Add another approach to FC_PRGNAME for Solaris 10 or before

Patch from Raimund Steger

Fix installation on MinGW32

Patch from LRN

Fix mkstemp absence for some platform

Patch from LRN and modified to make more generic.

Add missing file descriptor to F_DUPFD_CLOEXEC

Patch from Matthieu Herrb

Fix readlink failure

As reported by Raimund Steger.

Better fix for 2fe5ddfd

Drop realpath() and fix breakage on Win32

Fix FC_PRGNAME default

As reported by Raimund Steger.

Fix fc-cache crash caused by looking up NULL object incorrectly

We were returning a skiplist node when looking up NULL!

Minor

Remove unused checks for common functions

The check results of these were never actually used.

Improve FcGetPrgname() to work on BSD

Bug 59379 - FC_PRGNAME

Can be used for per-application configuration.

Bug 50497 - RFE: Add OpenType feature tags support

Add FC_FONT_FEATURES to store the feature tags to be enabled.

Fix the build fail on MinGW

Reported at http://lists.freedesktop.org/archives/fontconfig/2013-January/004601.html

just warn at the runtime instead of the compile time. it somewhat works
on even MinGW since FcMakeTempfile() isn't used on Win32 so far.

Minor

Copy all values from pattern to font if the font doesn't have the element

Bug 59376 - FcFontRenderPrepare enhancement

Ensure we find the uninstalled fontconfig header

Patch from Colin Walters.

Resepct $NOCONFIGURE

Patch from Colin Walters.

Bump version to 2.10.91

Update the date in README properly

Make linker happy

Add atomic ops for Solaris

Patch from Raimund Steger.

Bug 29312 - RFE: feature to indicate which characters are missing to satisfy the language support

Add fc-validate to check the language coverage in a font.

Fix a typo in the manpages template

Add pthread test

Not enabled by default since it requires config and fonts.

Fix memory corruption!

In FcStrListCreate() we were increasing reference count of set,
however, if set had a const reference (which is the case for list
of languages), and with multiple threads, the const ref (-1) was
getting up to 1 and then a decrease was destroying the set.  Ouch.

Here's the valgrind error, which took me quite a few hours of
running to catch:

==4464== Invalid read of size 4
==4464==    at 0x4E58FF3: FcStrListNext (fcstr.c:1256)
==4464==    by 0x4E3F11D: FcConfigSubstituteWithPat (fccfg.c:1508)
==4464==    by 0x4E3F8F4: FcConfigSubstitute (fccfg.c:1729)
==4464==    by 0x4009FA: test_match (simple-pthread-test.c:53)
==4464==    by 0x400A6E: run_test_in_thread (simple-pthread-test.c:68)
==4464==    by 0x507EE99: start_thread (pthread_create.c:308)
==4464==  Address 0x6bc0b44 is 4 bytes inside a block of size 24 free'd
==4464==    at 0x4C2A82E: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==4464==    by 0x4E58F84: FcStrSetDestroy (fcstr.c:1236)
==4464==    by 0x4E3F0C6: FcConfigSubstituteWithPat (fccfg.c:1507)
==4464==    by 0x4E3F8F4: FcConfigSubstitute (fccfg.c:1729)
==4464==    by 0x4009FA: test_match (simple-pthread-test.c:53)
==4464==    by 0x400A6E: run_test_in_thread (simple-pthread-test.c:68)
==4464==    by 0x507EE99: start_thread (pthread_create.c:308)

Thread test is running happily now.  Will add the test in a moment.

Fix a build fail on mingw

Regarding the change of 596931c8b4a7a35cbff9c33437d3cd44395d9c3f

missing header file to declare _mkdir

Bug 47705 - Using O_CLOEXEC

Fix pthreads setup

As reported by Raimund Steger.

Oops, add the actual file

Add 10-scale-bitmap-fonts.conf and enable by default

Clean up the unused variable

Remove FcInit() calls from tools

Library is supposed to automatically initialize itself.  If it doesn't,
it's a bug.

Don't use blanks for fc-query

fc-query is supposed to be config-independent.

Missing header file for _mkdir declaration

Minor

Ugh, add Tools.mk

Second try to make Sun CPP happy

Really fix cross-compiling and building of tools this time

Work around Sun CPP

According to Raimund Steger:

> [...]
> diff --git a/src/Makefile.am b/src/Makefile.am
> index dc082b7..57c34a2 100644
> [...]
> +fcobjshash.gperf: fcobjshash.gperf.h fcobjs.h
> +    $(AM_V_GEN) $(CPP) -I$(top_srcdir) $< | $(GREP) '^[^#]' | awk ' \
> +    /CUT_OUT_BEGIN/ { no_write=1; next; }; \
> +    /CUT_OUT_END/ { no_write=0; next; }; \
> +    { if (!no_write) print; next; }; \
> +    ' - > $@.tmp && \
> +    mv -f $@.tmp $@

Sun Studio CPP seems to insert whitespace in a different way than GCC's CPP.

GCC generates in src/fcobjshash.gperf:

[...]
"family", FC_FAMILY_OBJECT
"familylang", FC_FAMILYLANG_OBJECT
[...]

Sun Studio generates:

[...]
 "family" , FC_FAMILY_OBJECT
 "familylang" , FC_FAMILYLANG_OBJECT
[...]

leading to:

[...]
Making all in src
gmake[2]: Entering directory `/home/rs/src/fontconfig-git/fontconfig/src'
  GEN    fcobjshash.gperf
  GEN    fcobjshash.h
Key link: " " = " ", with key set "".
1 input keys have identical hash values,
use option -D.
gmake[2]: *** [fcobjshash.h] Error 1
gmake[2]: Leaving directory `/home/rs/src/fontconfig-git/fontconfig/src'
gmake[1]: *** [all-recursive] Error 1
gmake[1]: Leaving directory `/home/rs/src/fontconfig-git/fontconfig'
gmake: *** [all] Error 2

...maybe we could tuck in an additional sed to remove the whitespace, like:

[...]
fcobjshash.gperf: fcobjshash.gperf.h fcobjs.h
    $(AM_V_GEN) $(CPP) -I$(top_srcdir) $< | \
    $(SED) 's/^\s*//;s/\s*,\s*/,/;' | \
    $(GREP) '^[^#]' | \
    $(AWK) '/CUT_OUT_BEGIN/,/CUT_OUT_END/ { next; }; { print; };' \
    > $@.tmp && \
    mv -f $@.tmp $@
[...]

though I'm not sure what kind of guarantee CPP can give us/what easier option I might have missed...

Fix build around true/false

Trying to fix distcheck

Doesn't work though :(.  Building docs is very fragile...

At least, if docbook is present, distcheck passes now.

Fix more warnings.

Linux build and mingw32 cross build warning-free now.

Use CC_FOR_BUILD to generate source files

Previously we were failing if CROSS_COMPILING and the generated headers
were not present.  It works just fine now.

One caveat: the fix is not fully correct since config.h is being
included in the files built with CC_FOR_BUILD, but config.h has config
for the host system, not the build system.  Should be fine though.

Fix build and warnings on win32

Minor

Fix compiler warnings

Remove FcSharedStr*

Fixup fcobjs.c

Ouch!

Warn about undefined/invalid attributes during config parsing

Make fcobjs.c thread-safe

With this, the library should be threadsafe as far as my analysis goes!

Make FcDirCacheDispose() threadsafe

Make cache hash threadsafe

This concludes my first pass at making fontconfig threadsafe.  Now to
testing and actually fixing it!

Make random-state initialization threadsafe

Add a big cache lock

Not used yet.

Make cache refcounting threadsafe

Minor

Make FcCacheIsMmapSafe() threadsafe

Minor

Make default-FcConfig threadsafe

Minor

Refactor; contain default config in fccfg.c

Make FcDefaultFini() threadsafe

Make FcInitDebug() idempotent

Make FcGetDefaultLang and FcGetDefaultLangs thread-safe

Make refcounts, patterns, charsets, strings, and FcLang thread-safe

Add thread-safety primitives

Add build stuff for threadsafety primitives

Copied over from HarfBuzz.

Fix build stuff

Remove shared-str pool

We used to have a shared-str pool.  Removed to make thread-safety
work easier.  My measurements show that the extra overhead is not
significant by any means.

Switch .gitignore to git.mk

Use a static perfect hash table for object-name lookup

The hash table is generated by gperf.  For runtime element types, we use
a append-only linked list.

A bit clumsy, but I think I got it right.

Deprecate FcName(Un)RegisterObjectTypes / FcName(Un)RegisterConstants

These never worked as intended.  The problem is, if Fontconfig tries to
read config files when these new types / constants are not registered,
it errs.  As a result, no defined types / constants are usable from
config files.  Which makes these really useless.  Xft was the only user
of this API and even there it's not really used.  Just kill it.

One inch closer to thread-safety since we can fix the object-type hash
table at compile time.

Whitespace

Minor

Don't crash in FcPatternFormat() with NULL pattern

Don't crash in FcPatternDestroy with NULL pattern

Add NULL check

Refuse to set value to unsupported types during config too

Make FC_DBG_OBJTYPES debug messages into warnings

And remove FC_DBG_OBJTYPES since it has no use now.

Warn if <name target="font"> appears in <match target="pattern">

Adjust docs for recent changes

Initialize matrix during name parsing

Before:
$ fc-match sans:matrix=2 -v | grep matrix
matrix: [2 6.95183e-310; 0 0](s)

After:
$ fc-match sans:matrix=2 -v | grep matrix
matrix: [2 0; 0 1](s)

Make tests run on Windows

Enable fcarch assert checks even when FC_ARCHITECTURE is explicitly given

After all, if the asserts fail, the resulting library simply can't be
working correctly.

Remove unneeded stuff

Unbreak build when FC_ARCHITECTURE is defined