[Fontconfig] [PATCH] Do not remove UUID file when a scanned directory is empty

Akira TAGOH akira at tagoh.org
Mon Nov 5 07:52:04 UTC 2018


a few more comments for your changes.

Please keep doc for FcDirCacheCreateUUID but describe deprecated. this
causes make check fails. also good to use FC_PATH_MAX instead of
PATH_MAX. PATH_MAX seems not defined on GNU/Hurd. see abdb6d65. there
are one more PATH_MAX in the existing code which I missed correcting
when merging. that would be appreciated if you can update it too.
For last one, well, I know this is ugly hack but I did to get tests
working though, please sync up FcConfig structure to test/test-conf.c,
particularly if you are going to change the member of them or adding a
member earlier like this. this is the reason why the CI pipeline on
GitLab failed.
This is needed to build the virtual FcFontSet for testing though,
dunno if I should add an API to access instead of accessing the
structure directly, because I don't expect to use it for non-testing
purpose.

Anyway, otherwise that looks good. please submit a MR for that.

On Fri, Nov 2, 2018 at 3:15 AM Keith Packard <keithp at keithp.com> wrote:
> We might consider creating a new element that resets the directory list
> to empty?
>
>         <reset-dirs/>
>         <dir>~/.fonts</dir>
>
> would remove all system-specified font directories and add ~/.fonts.

That could be used anytime anywhere. doesn't it make a trouble?

>
> --
> -keith


Thanks,
-- 
Akira TAGOH


More information about the Fontconfig mailing list