Re: [TSVN] TSVN for Win64 - first results

Stefan Fuhrmann wrote:

> as promised, I spent some time trying to port
> TSVN to AMD64. My experiences so far:
>
> * most standard string operations like strncpy are
> considered "unsecure" (independent from target
> platform). Disabling warning 4996 is a must ..

But 4996 is "deprecated" according to my version of MSDN? Why is that
called "unsecure" now?

> * compiling SVN and TSVN for x86 requires only
> a few tweaks (no surprise here)

Great!

> * setting the projects up for AMD64 takes some
> time
> * finally, I got BDB, SVN libraries and most other
> libraries to compile "o.k." or even "fine"

What's the difference between "o.k." and "fine"?

> * OpenSSL doesn't compile yet

I just checked the openssl docs, and according to them it _should_
compile just fine. But you might have to adjust the config files (I
don't think that ./configure will run on Windows).

> * most modules in general (.DLL, .EXE) don't link yet

Does that mean it doesn't link because of missing modules? I mean you
just said that openssl doesn't compile yet.

> * in general: even in SVN there are many warnings
> of possibly lossy size_t => int conversions

I noticed that too, that's why I disabled the 64-bit compatibility warnings.

> Most important, though, TortoiseSVN.DLL compiles
> and links fine. According to the MSDN library entry
> for "InprocServer", it will not require modified
> registry settings.

Great!

> Because I had no time yet to set up a WinXP64 box,
> I have no idea whether the code actually works.

At least, it does on WinXP32.

> @steveKing: Does it make sense to provide it
> somewhere in the download section? Just in case,
> someone wants to give it a spin ..

You mean just the dll? I don't think that would help much without an
installer which also sets all the requred registry keys.

About the openssl and the other libs: I don't think it's necessary to
compile TortoiseProc with 64-bit support. The dll should be enough
because AFAIK Win64 can run 32-bit apps just fine?

> The attached patch makes TSVN compatible with
> 32 and 64 bit WinAPI. Also, it provides some fixes
> for "paranoid" 64 bit compatibility warnings.

Committed your patch in revision 1774.

Maybe we should contact Brian C. Barnes (brian.barnes@amd.com) and ask
him if he's willing to try a testrun? I mean he's the one who first
asked for 64-bit Windows support.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org