After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 753743 - glibtop_machine ABI changes in libgtop 2.31.90 break darwin build
glibtop_machine ABI changes in libgtop 2.31.90 break darwin build
Status: RESOLVED FIXED
Product: libgtop
Classification: Core
Component: darwin
2.31.x
Other Mac OS
: Normal normal
: ---
Assigned To: libgtop maintainers
libgtop maintainers
Depends on:
Blocks:
 
 
Reported: 2015-08-18 04:38 UTC by David Evans
Modified: 2016-01-28 16:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch to fix darwin build (793 bytes, patch)
2015-08-18 09:22 UTC, David Evans
committed Details | Review

Description David Evans 2015-08-18 04:38:01 UTC
Building on

MacPorts 2.3.3
Mac OS X 10.10.5 (Yosemite)
Xcode 6.4
Build version 6E35b
Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)
Target: x86_64-apple-darwin14.5.0
Thread model: posix

Build fails as follows:

:info:build make[3]: Entering directory `/opt/local/var/macports/build/_opt_macports_users_devans_GNOME-3_unstable_dports_gnome_libgtop/libgtop/work/libgtop-2.31.90/sysdeps/darwin'
:info:build   CC       nosuid.lo
:info:build   CC       siglist.lo
:info:build   CC       sysinfo.lo
:info:build   CC       open.lo
:info:build open.c:70:17: error: incomplete definition of type 'struct _glibtop_machine'
:info:build         server->machine->uid = getuid ();
:info:build         ~~~~~~~~~~~~~~~^
:info:build ../../glibtop.h:35:16: note: forward declaration of 'struct _glibtop_machine'
:info:build typedef struct _glibtop_machine glibtop_machine;
Comment 1 Benoît Dejean 2015-08-18 06:40:02 UTC
Hello David, are you building from git ? It just misses a #include <glibtop_machine.h> in open.c and may be in some other files.
Comment 2 David Evans 2015-08-18 09:19:29 UTC
Hi, Benoît.  Actually I'm building from the 2.31.90 tar ball but the difference shouldn't be significant. Your diagnosis is correct.  Attached is my patch that adds the missing #include in the right places.

With this patch, libgtop builds without error and cursory testing with gnome-system-monitor 3.17.90 looks good.
Comment 3 David Evans 2015-08-18 09:22:05 UTC
Created attachment 309446 [details] [review]
Proposed patch to fix darwin build
Comment 5 Benoît Dejean 2015-08-18 14:45:33 UTC
This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.