File: ffs_icon_loader.patch

package info (click to toggle)
freefilesync 12.0-2
  • links: PTS, VCS
  • area: main
  • in suites: bookworm
  • size: 8,432 kB
  • sloc: cpp: 64,801; ansic: 431; makefile: 217
file content (35 lines) | stat: -rw-r--r-- 2,073 bytes parent folder | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
Summary: Solve libglib2.0 problem in sketchy manner
Date: 2022-05-24
Version: 11.21
Author: bgstack15
Message:
Warning! I have absolutely no idea what I am doing. Check https://freefilesync.org/forum/viewtopic.php?t=8780 for future developments.
In Devuan Ceres, starting in 2021-08 with libglib2.0-0=2.68.3-2 the file icon_loader.cpp fails to compile. By removing the static_cast, this file can compile. The built application throws all sorts of GLib-GObject-CRITICIAL messages but the application operates.
Downgrade to "deb [check-valid-until=no] https://snapshot.debian.org/archive/debian/20210801T144205Z/ unstable main non-free contrib" in /etc/apt/sources.list and install the exact libglib version necessary to compile without this patch.
$ sudo apt-get -V install libglib2.0-0=2.66.8-1 libglib2.0-bin=2.66.8-1 libglib2.0-data=2.66.8-1 libglib2.0-dev=2.66.8-1 libglib2.0-dev-bin=2.66.8-1 libglib2.0-0:i386=2.66.8-1
Zenju tried addressing this for me, but it still does not compile, even with g++-12. So I still have to omit this macro call.

UPDATE by Fab Stz on 2022-08-08:
Preprocessor output for `::g_object_ref(gicon); ` is:
    ::((typename std::remove_reference<decltype (gicon)>::type) (g_object_ref) (gicon));
Since commit https://gitlab.gnome.org/GNOME/glib/-/commit/5b2bee3f539056b42c802608f9f00cc9ddd64b79
that is shipped starting from 2.67.0, the macro g_object_ref changed and is
also defined when __cplusplus is defined in some circumstances.

Removing the scope resolution operator `::` permits the code to build.
There remains to check whether this is the correct fix.

--- a/FreeFileSync/Source/base/icon_loader.cpp
+++ b/FreeFileSync/Source/base/icon_loader.cpp
@@ -227,7 +227,11 @@
     //the remaining icon types won't block!
     assert(GDK_IS_PIXBUF(gicon) || G_IS_THEMED_ICON(gicon) || G_IS_EMBLEMED_ICON(gicon));
 
+#if (GLIB_CHECK_VERSION (2, 67, 0))
+    g_object_ref(gicon);                   //pass ownership
+#else
     ::g_object_ref(gicon);                 //pass ownership
+#endif
     return FileIconHolder(gicon, maxSize); //
 
 }