Hi, On Wed, Feb 21, 2018 at 08:20:34AM -0500, Selva Nair wrote: > > Thanks for investigating and finding the root cause. "This is not what > > the documentation says...!" -> I need to actually *run* these things, not > > just test compile. > > When I say painful builds, I mean edit build[*], copy to Windows, run > in a few configurations, look through the event viewer, rinse rather > repeat. And finally when it seems to start working connect to a my > work VPN, remove the registry keys and try to break it etc..
Indeed, that takes quite a bit of time.
> FYI, I don't just "test compile", and leave it at that.
I did not want to imply that you did - sorry if it came across that way.
*I* only test compiled, and was then happy merging your patch ("I'm sure
Selva tested this, it is according to documentation and the code looks
good, and compiles fine") - but as I learned, there are more caveats on
Windows. So, when we do more basic changes (new API, etc), I need to
do actual test installations on my windows zoo, like, win7/32, win7/64,
win10/64... I usually do this already today if I suspect breakage, but
in this case, it came as a surprise.
gert
--
now what should I write here...
Gert Doering - Munich, Germany [email protected]
signature.asc
Description: PGP signature
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
