Wrapped Wow64* in begin..rescue.

[TomyLobo]

These functions don't seem to exist on Windows XP 32 bit, rendering support for it, which was broken since at least 0.6.1, probably since 0.6.0.

It shouldn't affect Windows versions where those functions exist.
And those where they don't exist would not work with 0.6.0-0.6.2 anyway :)

[djberg96]

I'm not inclined to support Windows XP since Microsoft dropped support in April. Are you using this in production?

[TomyLobo]

Nope, I had to downgrade to Puppet 3.6.2, which uses win32-eventlog 0.5.3.
That works until I need to upgrade Puppet for some reason.

I will test it if that's a requirement to getting it accepted.

Alternatively, someone suggested "calling RegOpenKeyEx with KEY_WOW64_64KEY", which, in addition to being more likely to work on Windows XP, also makes your code less stateful and thus more likely to be reentrant

[djberg96]

If RegOpenKeyEx with KEY_WOW64_64KEY is better, and it works, I'd be willing to merge such a patch!

[aogail]

I've just hit this incompatibility, also. Can this change be merged?

FWIW, XP is still widely used in production all over the place, and it's still supported by Microsoft. They just call it Windows Embedded POSReady 2009. It will continue to receive updates until 2019.

See: https://support.microsoft.com/en-us/lifecycle/search/default.aspx?sort=PN&alpha=Windows%20Embedded%20POSReady&Filter=FilterNO

[sean-sype-simmons]

@TomyLobo do you still need this?

[TomyLobo]

9 years later, Windows XP has been EOL for 10 years and needs to die, so I don't care :)