Bug 101997 - Consider moving fontconfig-monitor into fontconfig tree
Summary: Consider moving fontconfig-monitor into fontconfig tree
Status: RESOLVED MOVED
Alias: None
Product: fontconfig
Classification: Unclassified
Component: library (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: fontconfig-bugs
QA Contact: Behdad Esfahbod
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-01 10:46 UTC by Behdad Esfahbod
Modified: 2018-08-20 21:51 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Behdad Esfahbod 2017-08-01 10:46:49 UTC
The code at:
https://github.com/behdad/fontconfig-monitor
Comment 1 Akira TAGOH 2017-08-01 14:48:44 UTC
Do we want to add a glib dep to fontconfig?
just wonder if the sort of this feature should goes into the toolkits like GTK+ and Qt or whatever. so it can be integrated to their mainloop easily.
Comment 2 Behdad Esfahbod 2017-08-01 14:53:17 UTC
Oh I meant just moving the code into fontconfig tree under its own subdirectory.  Maybe if user has glib, build it as fc-monitor, with a useful output format.  Didn't mean to merge it into the library.
Comment 3 Behdad Esfahbod 2017-08-08 20:04:54 UTC
Code updated and improved.  See:
https://github.com/behdad/fc-monitor

I can just put it in fontconfig/fc-monitor, with its separate build Makefile as is.
Comment 4 Akira TAGOH 2017-08-09 02:08:51 UTC
Sure. though I'm not sure if we should build it because it doesn't do more than checking updates and we don't have a way to stop that in apps so far. so people doesn't have any advantage to run it.

We may merge https://cgit.freedesktop.org/~tagoh/fontconfig/commit/?h=disable-auto-update as well?
Comment 5 Behdad Esfahbod 2017-08-11 02:33:07 UTC
(In reply to Akira TAGOH from comment #4)
> Sure. though I'm not sure if we should build it because it doesn't do more
> than checking updates and we don't have a way to stop that in apps so far.
> so people doesn't have any advantage to run it.
> 
> We may merge
> https://cgit.freedesktop.org/~tagoh/fontconfig/commit/?h=disable-auto-update
> as well?

What problem is that trying to solve?  How do you see it used?  Ie. by Gtk+?
Comment 6 Behdad Esfahbod 2017-08-11 02:33:37 UTC
(In reply to Akira TAGOH from comment #4)
> Sure. though I'm not sure if we should build it because it doesn't do more
> than checking updates and we don't have a way to stop that in apps so far.
> so people doesn't have any advantage to run it.
> 
> We may merge
> https://cgit.freedesktop.org/~tagoh/fontconfig/commit/?h=disable-auto-update
> as well?

I prefer that we fix scanning speed and remove the blocking lock instead of adding bandaid API.
Comment 7 Akira TAGOH 2017-08-14 03:44:17 UTC
I don't think we can get rid of the lock. improving the scan speed would just make things hard to see. but can't be zero unless making the update process to be singleton as I suggested before. improving the scan speed itself would be very useful though, it isn't the essential solution to remove the lock. need an extra idea to avoid breaking a cache and detect it before writing otherwise we can't simply do that.
Comment 8 Behdad Esfahbod 2017-08-17 01:41:53 UTC
(In reply to Akira TAGOH from comment #7)
> I don't think we can get rid of the lock. improving the scan speed would
> just make things hard to see. but can't be zero unless making the update
> process to be singleton as I suggested before. improving the scan speed
> itself would be very useful though, it isn't the essential solution to
> remove the lock. need an extra idea to avoid breaking a cache and detect it
> before writing otherwise we can't simply do that.

Ok, let's continue on the email thread.  I'll comment there.
Comment 9 GitLab Migration User 2018-08-20 21:51:34 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/89.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.