dns-sd dll implementation -- Needs work

[C.W. Betts]

What do you guys think? it's a wrapper for multicast DNS, aka Bonjour®.

I know there's probably a patches mailing list, but I put this here for feedback.

[Juan Lang]

Hi C.W.,
> I know there's probably a patches mailing list, but I put this here for feedback.

This is the appropriate place to ask about patches that aren't ready
to commit yet.

+#include <dns_sd.h>
At the least, you'll need to add configure checks for this header
file.  I imagine you'll need to load the library dynamically, too.  On
at least one of my systems, /usr/lib/libdns_sd.so is available, but
<dns_sd.h> is not.  I'm not sure how common that would be, but you
might want to provide a private copy of the .h file too.
--Juan

[Dmitry Timoshkov]

"C.W. Betts"  wrote:
> What do you guys think? it's a wrapper for multicast DNS, aka Bonjour®.

Looks like this is not a part of Windows or PSDK, so Wine shouldn't include
it either.

[Gert van den Berg]

On Tue, Feb 23, 2010 at 09:37, Dmitry Timoshkov  wrote:
> "C.W. Betts"  wrote:
> > What do you guys think? it's a wrapper for multicast DNS, aka Bonjour®.
>
> Looks like this is not a part of Windows or PSDK

Neither is OpenAL....

It might provide functionality as part of Wine that might be hard to
implement on top of Wine?

On OS X, it would also eliminate the need to install the Windows
version of Apple applications bundled with OS X in order to run things
that uses a single library provided by them. (e.g. Bonjour support in
VLC)

Gert

[Steven Edwards]

On Tue, Feb 23, 2010 at 7:08 AM, Gert van den Berg  wrote:
> On OS X, it would also eliminate the need to install the Windows
> version of Apple applications bundled with OS X in order to run things
> that uses a single library provided by them. (e.g. Bonjour support in
> VLC)

Do you really want to run the Windows version of VLC on OS X? I
believe there is a better case to be made with Safari/Quicktime/iTunes
as I believe they also install Bonjour services on Windows. If you
wanted to validate/compare how the native service works verses say a
wrapper then it would be of use.

Thanks

[Gert van den Berg]

On Tue, Feb 23, 2010 at 18:57, Steven Edwards  wrote:
> Do you really want to run the Windows version of VLC on OS X? I
> believe there is a better case to be made with Safari/Quicktime/iTunes
> as I believe they also install Bonjour services on Windows. If you
> wanted to validate/compare how the native service works verses say a
> wrapper then it would be of use.

Pidgin might be another example...

What I really meant is that it should at least be evaluated... (I
didn't realize at the time that Bonjour for Windows has a seperate
installer available...)

It might have a valid use if conflicts exist between Windows Bonjour
under Wine and native bonjour / other Zeroconf implementations...
(such as listening on the same port) (If such a confilct exists, it
should be possible to run either a ZeroConf implementation natively or
to run it under Wine, but not both. This can be problematic, since two
of Wine's largest platforms, Ubuntu and OS X installs ZeroConf
implementations by default)

(I'm not familiar enough with Bonjour / ZeroConf to really understand
which parts of it is wrapped by this DLL) (It seems that DNS Service
Discovery is able to function without mDNS)

Just a note: According to Wikipedia, Photoshop CS3 is among the
applications using Bonjour under Windows:
http://blogs.adobe.com/jnack/2007/01/cs3_doesnt_inst.html

Gert

[C.W. Betts]

On Feb 23, 2010, at 12:31 PM, Gert van den Berg wrote:
> Just a note: According to Wikipedia, Photoshop CS3 is among the
> applications using Bonjour under Windows:
> http://blogs.adobe.com/jnack/2007/01/cs3_doesnt_inst.html
>
> Gert

And I read a bugzilla report that CS3 was having issues because of it.

From what I can tell, DNS SD uses obscure structs and memory is allocated / deallocated using it's own functions.  This wrapper wraps each function.

I did edit the configure.ac file, but couldn't get autoconf to create a good configure file.
Patch v2 attached