]> git.michaelhowe.org Git - packages/o/openafs.git/commit
Windows: Remove RXAFS* from afsrpc.dll
authorJeffrey Altman <jaltman@your-file-system.com>
Wed, 27 Feb 2013 20:07:57 +0000 (15:07 -0500)
committerJeffrey Altman <jaltman@your-file-system.com>
Thu, 28 Feb 2013 10:54:24 +0000 (02:54 -0800)
commite2ef4ce23d0cd0eddadfb39a39a29593093f0b6f
tree750c7d0f29660948d3803d09ebf9e3c3c5d1dbfe
parentd6cfcf46a8f3ec03e2f805aca6e5012eb6633803
Windows: Remove RXAFS* from afsrpc.dll

afsrpc.dll is supposed to be the multi-threaded dynamic loadable
version of the RX package.  It has also included the RXAFS and
RXAFSCB client functions from src/fsint.  Including the RXAFS* is
nice in theory but in practice the afsd_service.exe must link to
afsint.lib anyway in order to obtain access to RXAFSCB_ExecuteRequest()
which is part of the server portion.  Linking to both results in
multiple instances of the same symbols in which case afsint.lib
must be linked to first and the dynamic version of the functions
are ignored.

The only other module that used RXAFS* functions from afsrpc.dll
is the adminutil library from the libadmin package.  adminutil
doesn't implement the server functions but it can just as easily
link against afsint.lib.

Removing RXAFS* from afsrpc.dll results in a library that only
contains RX, RXSTATS, core xdr and rxkad.

This patchset also fixes some minor bugs:

 1. $(UTILOBJS) were not being linked to the library due to
    a typo.

 2. Always use multi-thread safe versions of afsutil.lib

Change-Id: I5cc6d485a763d5a17735abd35d193fb7b0d280e5
Reviewed-on: http://gerrit.openafs.org/9300
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/WINNT/afsd/NTMakefile
src/libadmin/adminutil/NTMakefile
src/libafsrpc/NTMakefile
src/libafsrpc/afsrpc.def