user auth patches
Ralf Habacker
ralf.habacker at freenet.de
Fri Jun 15 14:10:37 PDT 2007
Havoc Pennington schrieb:
> Hi,
>
> Ralf Habacker wrote:
>> Unfortunally your patch breaks compiling of main.c because you have
>> removed the part
>> #ifdef DBUS_WIN
>> #include <dbus/dbus-sysdeps-win.h>
>> #endif
>>
>
> I took this out to break the code you mention, because any code
> relying on the above is definitely wrong. The purpose of the
> sysdeps-unix.h and sysdeps-win.h headers is that they are only
> included in platform-specific files. main.c is not a platform-specific
> file.
>
> Please study the pattern in the attached patch. The approach is:
> 1) add function in dbus-sysdeps.h which does the logical
> cross-platform operation
> 2) adapt cross-platform code to use said function
> 3) implement said function in sysdeps-unix.c or sysdeps-win.c
we were already here.
> 4) submit patch for review and commit afterward
this was discussed already on the mailing list.
> 5) wait for the maintainer of the other port to implement it for the
> other platform
>
now we are here
Thanks for your effort.
Ralf
> Havoc
More information about the dbus
mailing list