On Tue, 2005-02-08 at 13:34 +0100, Tom Parker wrote: >Getting errors on NetworkManagerDevice.c, because I don't appear to have >a "pci/types.h". No idea where the heck this would be (on a Debian >system), but then worked out that we only needed a few types from it and >so wrote a patch. > >Index: src/NetworkManagerDevice.c >=================================================================== >RCS file: /cvs/gnome/NetworkManager/src/NetworkManagerDevice.c,v >retrieving revision 1.91 >diff -u -r1.91 NetworkManagerDevice.c >--- src/NetworkManagerDevice.c 7 Feb 2005 23:04:05 -0000 1.91 >+++ src/NetworkManagerDevice.c 8 Feb 2005 12:32:11 -0000 >@@ -3600,7 +3600,10 @@ > /**************************************/ > /* Ethtool capability detection */ > /**************************************/ >-#include <pci/types.h> >+typedef unsigned long long u64; >+typedef unsigned int u32; >+typedef unsigned short u16; >+typedef unsigned char u8; > #include <linux/sockios.h> > #include <linux/ethtool.h>
Hmm. Why not port the code to the glib typedefs (e.g. guint32) instead? While those typedefs are correct (AFAIK) on all the important platforms that Linux runs on today, there's no guarantee that in the future e.g. "int" will be 32 bits. On such a platform the code will probably start failing mysteriously.
signature.asc
Description: This is a digitally signed message part
_______________________________________________ NetworkManager-list mailing list NetworkManager-list@gnome.org http://mail.gnome.org/mailman/listinfo/networkmanager-list
