[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bug 786694 depends on bug 786693, which changed state. Bug 786693 Summary: wayland: fix fontconfig monitoring https://bugzilla.gnome.org/show_bug.cgi?id=786693 What|Removed |Added Status|NEW |RESOLVED Resolution|--- |OBSOLETE -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368835|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #369015|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368945|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368943|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368834|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368833|none|committed status|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #26 from Bastien Nocera --- Attachment 368833 pushed as 522640a - xsettings: Expose monitored settings through D-Bus Attachment 368834 pushed as 85e162f - xsettings: Use temporary variable for GTK+ modules dir Attachment 368835 pushed as 2b79fb8 - xsettings: Make it possible to override GTK+ modules path Attachment 368943 pushed as e514810 - xsettings: Make it possible to ignore llvmpipe Attachment 368945 pushed as 26e1e1e - tests: Rename work directory Attachment 369014 pushed as 8ec7856 - xsettings: Add tests for D-Bus interface Attachment 369015 pushed as 9dbb9e8 - tests: Fix concurrent launches of Xorg-reliant test cases -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #25 from Bastien Nocera--- Created attachment 369015 --> https://bugzilla.gnome.org/attachment.cgi?id=369015=edit tests: Fix concurrent launches of Xorg-reliant test cases -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368944|0 |1 is obsolete|| --- Comment #24 from Bastien Nocera --- Created attachment 369014 --> https://bugzilla.gnome.org/attachment.cgi?id=369014=edit xsettings: Add tests for D-Bus interface -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #23 from Bastien Nocera--- Created attachment 368945 --> https://bugzilla.gnome.org/attachment.cgi?id=368945=edit tests: Rename work directory There's more than just a power test now. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368931|0 |1 is obsolete|| --- Comment #22 from Bastien Nocera --- Created attachment 368944 --> https://bugzilla.gnome.org/attachment.cgi?id=368944=edit xsettings: Add tests for D-Bus interface -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #21 from Bastien Nocera--- Created attachment 368943 --> https://bugzilla.gnome.org/attachment.cgi?id=368943=edit xsettings: Make it possible to ignore llvmpipe So that we can test out vino support. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368836|0 |1 is obsolete|| --- Comment #20 from Bastien Nocera --- Created attachment 368931 --> https://bugzilla.gnome.org/attachment.cgi?id=368931=edit xsettings: Add tests for D-Bus interface -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #19 from Bastien Nocera--- Created attachment 368836 --> https://bugzilla.gnome.org/attachment.cgi?id=368836=edit xsettings: Add tests for D-Bus interface FIXME, EnableAnimations doesn't work because the test can't create a "boolean" property, and I don't know why. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #18 from Bastien Nocera--- Created attachment 368835 --> https://bugzilla.gnome.org/attachment.cgi?id=368835=edit xsettings: Make it possible to override GTK+ modules path -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368775|0 |1 is obsolete|| --- Comment #16 from Bastien Nocera --- Created attachment 368833 --> https://bugzilla.gnome.org/attachment.cgi?id=368833=edit xsettings: Expose monitored settings through D-Bus A number of things that gnome-settings-daemon monitors for all the GTK+ applications aren't currently available for GTK+ Wayland clients, only through XSettings. As we already monitor fontconfig configurations, enabled GTK+ modules and whether applications should use animations, export those through a D-Bus interface. See https://bugzilla.gnome.org/show_bug.cgi?id=786693 Based on patch by Martin Blanchard -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #17 from Bastien Nocera--- Created attachment 368834 --> https://bugzilla.gnome.org/attachment.cgi?id=368834=edit xsettings: Use temporary variable for GTK+ modules dir So that we can override it later. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368734|0 |1 is obsolete|| -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #15 from Bastien Nocera--- The patch above is completely untested, and the GTK+ patch needs rewriting. You might need to also apply the patch from bug 793721 for this one to apply. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #14 from Bastien Nocera--- Created attachment 368775 --> https://bugzilla.gnome.org/attachment.cgi?id=368775=edit xsettings: Expose monitored settings through D-Bus A number of things that gnome-settings-daemon monitors for all the GTK+ applications aren't currently available for GTK+ Wayland clients, only through XSettings. As we already monitor fontconfig configurations, enabled GTK+ modules and whether applications should use animations, export those through a D-Bus interface. See https://bugzilla.gnome.org/show_bug.cgi?id=786693 Based on patch by Martin Blanchard -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #13 from Bastien Nocera--- The XSettings in the xsettings plugin are either: - in GSettings, and GTK+ will read those directly under Wayland - hardcoded in gnome-settings-daemon, for older versions of GTK+, and hardcoded in newer versions of GTK+ 3.x - "computed" in the xsettings plugin Those last ones are what we're interested in here. They are: - fontconfig timestamps (Fontconfig/Timestamp) - whether to enable animations (used for remote displays, Gtk/EnableAnimations) - GTK+ modules list (Gtk/Modules) - whether gnome-shell (and its builtin menu) is running (Gtk/ShellShowsAppMenu) - a number of font and DPI settings (see xft_settings_set_xsettings()) That last two are handled directly in GTK+ in Wayland talking to the compositor and checking its capabilities, or in gdk/wayland/gdkscreen-wayland.c. Which leaves us with those first 3 settings (for now). So, on top of the fontconfig specific changes above, it would be nice if: - the D-Bus name showed that the settings are GTK+ specific, say "org.gtk.Settings" - also export Gtk/EnableAnimations and the modules list through D-Bus -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Bastien Nocerachanged: What|Removed |Added Attachment #368734|none|needs-work status|| --- Comment #12 from Bastien Nocera --- Review of attachment 368734: 1) Get rid of the array 2) Every time a timestamp change has happened, queue sending the new value for half-a-second (or a second?), reschedule if a new timestamp arrives in that time 3) Don't use g_get_real_time() ::: plugins/xsettings/gsd-xsettings-manager.c @@ +80,3 @@ +static const gchar introspection_xml[] = +"" +" " Is this really the name of the interface we want to use? What about GTK+ apps under other Wayland desktops? @@ +284,3 @@ GSettings *plugin_settings; FcMonitor *fontconfig_monitor; +GArray*fontconfig_timestamps; I've stared at the code for a long while, and can't figure out why you need to send a "changed" signal for each and every fontconfig timestamp change. Why do that? You could just keep the last one, and send it out when it changes, or aggregate/coalesce the changes into one event if the changes are too close together. You need to remember that this would wake up every single GTK+ program, so less notifications is better. @@ +1189,3 @@ GList *list, *l; const char *session; +gint64 timestamp = g_get_real_time (); You shouldn't user g_get_real_time(), this will break if the time on the machine is changed. Any reason why g_get_monotonic_time() can't be used? @@ +1453,3 @@ +manager = GNOME_XSETTINGS_MANAGER (user_data); + +if (manager->priv->dbus_connection == NULL) This isn't necessary, you get a connection in the arguments of the function above, and you don't use this variable anyway. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #11 from Bastien Nocera--- (In reply to Martin Blanchard from comment #10) > (In reply to Bastien Nocera from comment #8) > > It's too late to land this for GNOME 3.28. The gnome-shell patch will need > > reviewing before this can land, assuming the code is correct. > > May have missed something here, sorry: what is this gnome-shell patch you > are talking about? Never mind, the gtk+ patch rather. I'll review it later. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #10 from Martin Blanchard--- (In reply to Bastien Nocera from comment #8) > It's too late to land this for GNOME 3.28. The gnome-shell patch will need > reviewing before this can land, assuming the code is correct. May have missed something here, sorry: what is this gnome-shell patch you are talking about? -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Martin Blanchardchanged: What|Removed |Added Attachment #368196|0 |1 is obsolete|| --- Comment #9 from Martin Blanchard --- Created attachment 368734 --> https://bugzilla.gnome.org/attachment.cgi?id=368734=edit Introduce org.gnome.SettingsDaemon.FontConfig (timestamp as int64) Fourth version: removed a couple of extra whitespaces. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Martin Blanchardchanged: What|Removed |Added Attachment #365988|0 |1 is obsolete|| Bastien Nocera changed: What|Removed |Added CC||bugzi...@hadess.net --- Comment #7 from Martin Blanchard --- Created attachment 368188 --> https://bugzilla.gnome.org/attachment.cgi?id=368188=edit Introduce org.gnome.SettingsDaemon.FontConfig (timestamp as int64) Second version: time is represented as int64 and valued using g_get_real_time(). Comments and feedbacks welcome. --- Comment #8 from Bastien Nocera --- It's too late to land this for GNOME 3.28. The gnome-shell patch will need reviewing before this can land, assuming the code is correct. At a glance, you'll want to go through the patch and remove the unnecessary whitespace changes. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Martin Blanchardchanged: What|Removed |Added Attachment #365988|0 |1 is obsolete|| --- Comment #6 from Martin Blanchard --- Created attachment 368188 --> https://bugzilla.gnome.org/attachment.cgi?id=368188=edit Introduce org.gnome.SettingsDaemon.FontConfig (timestamp as int64) Second version: time is represented as int64 and valued using g_get_real_time(). Comments and feedbacks welcome. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #5 from Martin Blanchard--- (In reply to Matthias Clasen from comment #3) > Should maybe be 'u' instead of 'i' ? Not sure about what to do here: xsettings plugin is using an int value retrieved from time(), gtk+'s setting in uint, and GLib's time seems to be represented by int64... In my opinion, exposing an int64 from g_get_real_time() would be the best option, but wouldn't help on gtk+ side. The int type has only been kept for some sort of compatibility with the original xsetting. So should we simplify and opt for gtk+'s uint? -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 Matthias Clasenchanged: What|Removed |Added Attachment #365988|none|accepted-commit_now status|| --- Comment #3 from Matthias Clasen --- Review of attachment 365988: ::: plugins/xsettings/gsd-xsettings-manager.c @@ +81,3 @@ +"" +" " +"" Should maybe be 'u' instead of 'i' ? -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #2 from Matthias Clasen--- Thanks for working on this! -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
[Wayland-bugs] [Bug 786694] wayland: fix fontconfig monitoring
https://bugzilla.gnome.org/show_bug.cgi?id=786694 --- Comment #1 from Martin Blanchard--- Created attachment 365988 --> https://bugzilla.gnome.org/attachment.cgi?id=365988=edit Introduce org.gnome.SettingsDaemon.FontConfig Timestamp is exposed as the 'ConfigTimestamp' property of a (new) org.gnome.SettingsDaemon.FontConfig dbus interface. 'PropertiesChanged', from org.freedesktop.DBus.Properties, is emitted on update. It feels wrong to implement this in the xsettings plugin. Think it deserves a dedicated plugin; xsettings could rely on this new one. But needs more work... Comments and feedbacks welcome. -- You are receiving this mail because: You are on the CC list for the bug.___ wayland-bugs mailing list wayland-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-bugs