[krbdev.mit.edu #8662] Windows README does not document MFC requirement

[Sergey Ilinykh via RT]

It seems there is one more problem. I had to patch makefile to make it
compile


diff --git a/src/ccapi/test/Makefile.in b/src/ccapi/test/Makefile.in
index 85fe172..64e50fa 100644
--- a/src/ccapi/test/Makefile.in
+++ b/src/ccapi/test/Makefile.in
@@ -135,7 +135,7 @@ MORE_TESTS =    test_cc_context_release \
 ##### Linker
 LINK   = link
 LIBS    = -lkrb5
-##DOS##LIBS = $(CLIB) $(SLIB) advapi32.lib rpcrt4.lib user32.lib
ws2_32.lib $(CCLIB).lib
+##DOS##LIBS = $(CLIB) $(SLIB) advapi32.lib rpcrt4.lib user32.lib
ws2_32.lib ..\lib\win\srctmp\$(CCLIB).lib
 LFLAGS = /nologo $(LOPTS)

 all-mac:     setup-test-dir pingtest simple_lock_test build-base
build-tests link-tests copy-script success-message
@@ -192,10 +192,6 @@ $(OUTPRE)pingtest.exe: $(OBJECTS) $(PINGOBJS)
 #  deletes a.tmp and the following nmake actions correct LIB.
        echo %%PATH%% > a.tmp
     perl setlib.pl
-    if not exist a.tmp (
-        @echo Adding ..\lib\win\srctmp to LIB
-        set LIB=%%LIB%%;..\lib\win\srctmp
-        )
     $(LINK) $(linkdebug) /map:$(@B)1.map -out:$(*B)1.exe $(conflags)
$(PINGOBJS) $(LIBS)
        $(LINK) $(LFLAGS)    /map:$(@B)2.map /out:$(*B)2.exe $(conflags)
$(PINGOBJS) $(LIBS) $(conlibsdll)


Best Regards,
Sergey

2018-04-09 20:20 GMT+05:00 Greg Hudson via RT <rt-comment at krbdev.mit.edu>:

> The README file was originally written for MSVC++ 6.0 and was last
> comprehensively updated for Visual Studio 2010 (the version used to
> build the most recent KfW release).  I think perhaps MFC was more
> integral to Visual Studio in earlier releases, but I'd have to do more
> testing to be certain.  We should update the README file regardless.
>
> The build should work without MFC (skipping the Leash build) if you
> "set NO_LEASH=1" in the environment, as we do in the top level of
> appveyor.yml.
>