[Fontconfig-bugs] [Bug 90148] Don't warn if cachedir isn't specified

[bugzilla-daemon at freedesktop.org]

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

--- Comment #4 from Behdad Esfahbod <freedesktop at behdad.org> ---
(In reply to Akira TAGOH from comment #3)
> (In reply to Behdad Esfahbod from comment #2)
> > (In reply to Akira TAGOH from comment #1)
> > > 
> > > I'm not sure what's the purpose of the warning/error for that though, if it
> > > is to detect the installation error,
> > 
> > It might be ok for the system-wide config file.  I suppose the aim is to
> > make sure systems set global cachedir, as to prevent every user to have
> > their own cache of all the system fonts.  But we should definitely skip the
> > warning for custom config files.
> 
> Right. but there are no way to know if it is custom or not. hence:

It kinda is.  If FONTCONFIG_FILE / FONTCONFIG_PATH are not set, and we were
initializing default config, we should warn.


> > > is it a good idea to add an API like FcInitLoadConfigFromMemory()
> > 
> > That API is certainly desirable.
> 
> the above and the following suggestion. we can use it internally and move
> warnings out of it.
> 
> > > FcInitWithMinimalConfig() or so may be more useful perhaps.
> > > 
> > > Any thought?
> > 
> > What would go into the minimal config?
> > 
> > I discussed this in detail in my blog post.  I don't think there's a clear
> > line between minimal config and full config.
> 
> read it. well, I meant to make it convenient but may be better leave it to
> users then. so just reading the config from memory may be good.

My current thinking is that for most use cases, reading full config (but not
fonts) might be what user wants.  The major exception would be font viewers,
which probably can do without any config.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/fontconfig-bugs/attachments/20150514/aa21d4a1/attachment.html>