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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx> 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
_______________________________________________
Fontconfig mailing list
Fontconfig@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/fontconfig




[Index of Archives]     [Fedora Fonts]     [Fedora Users]     [Fedora Cloud]     [Kernel]     [Fedora Packaging]     [Fedora Desktop]     [PAM]     [Gimp Graphics Editor]     [Yosemite News]

  Powered by Linux