Does Windows x64 configuration build? about libnfs

[sahlberg]

Hi,

On Fri, Jul 25, 2014 at 4:28 PM, leonhui [email protected] wrote:

With some minor fixes I am able to build Windows x86 (Win32) configuration
with VS2010. It doesn't seem to use rpcgen.exe either.

With x64 configuration it runs rpcgen.exe overwriting the existing files.
Is this correct behavior?

That sounds wrong.
I don't build on Windows myself, so I have no way of testing/verifying it.
I know that the XBMC folks build on windows though, so it should work.

rpcgen.exe should not be used any more.
A long time ago I used the systems rpcgen tool to compile the protocol
definitions and then link with the rpc library.
For windows I ported/updated an old rpc library that the xbmc folks then
hosted and used.
In those days we used rpcgen.

Then there were other important platforms where there were just no rpcgen
or rpc library available at all (Android) easily
so several years ago I wrote my own rpc layer which I call ZDR inside
libnfs.
At that same time I also stopped using rpcgen during the build and instead
ship pre-compiled protocol files.

As such you should not need rpcgen.exe anymore at all.

As I do not have access to a windows build system,
Can you send me a patch that removes the use of rpcgen.exe from the project
files?

best regards
ronnie sahlberg

[leonhui]

I'll try to fix the Windows builds then. I just need to know if it still needs to compile/link against the oncrpc lib anymore since there is the ZDR layer?

One of the includes it needs from oncprc is <rpc/rpc.h>.

[sahlberg]

It does not need to link to the oncrpc library and should not link with
that library.

On Sun, Jul 27, 2014 at 1:50 PM, Leon [email protected] wrote:

I'll try to fix the Windows builds then. I just need to know if it still
needs to compile/link against the oncrpc lib anymore since there is the ZDR
layer?

One of the includes it needs from oncprc is .

—
Reply to this email directly or view it on GitHub
#91 (comment).

[sahlberg]

ping?

[sahlberg]

ping?

[sahlberg]

It builds on x64