Win32 builds broken after r15435

[Jay Modi <jaymode@…>]

The Win32 builds seem to have been partially broken by r15435. The first issue I noticed is that the typedef on line 89 needs a semicolon. However, this leads to even more problems, dealing with multiple definitions. The cure for this is to remove the typedef and then include compat.h but this causes linking errors. These linking errors are due to some other issue, as I have removed the entire r15435 changes from my compat.h and get the same errors.

I have attached a patch file to add the include.

[Jay Modi <jaymode@…>]

Patch to add include to util.h

[Jay Modi <jaymode@…>]

Patch for libmythtv/remoteutils.cpp. Needs compat.h for usleep() in Win32

[Jay Modi <jaymode@…>]

These patches may be unnecessary and it seems the builds actually broke with r15478. I believe the issue I was experiencing when I had opened the ticket was due to the use of a --cpu=k8 flag in the configure script, while using the win32-packager.pl script.

[jaymode@…]

Replying to danielk: I was incorrect. r15435 does break the Win32 builds when sse and mmx are enabled. The --cpu flag needs to be specified due to the cpu auto detection in the configure script being broken(per Andrei Tanas) in order to enable sse and mmx. With the --cpu=k8 flag enabled, the util_win32.patch is needed. The remoteutil.patch should be ignored for now.

r15478 breaks the builds with this patch, but that is a separate issue.

[Nigel]

(In [15547]) Prevent error on Windows builds when MMX is defined. See #4497

[danielk]

Jay, what patches on this ticket are still relevant and why? I don't have a windows box, so I need a little direction here...

[Jay Modi <jaymode@…>]

Neither patch is relevant. The remoteutil.patch is irrelevant and is not needed. The util_win32.patch has been applied by Nigel in r15547. So this ticket can be closed.