Re: [PATCH] new property 'svn:tarballs'

steveking@gmx.ch wrote:

>>>If you really had take a look at the patch you wouldn't say something
>>>like that. With the patch I sent in you could compile it either way, the
>>>same way as APR.
>>>
>>>
>>>
>>>
>>No. Your patch produced a single, all-in-one DLL from all the Subversion
>>libraries. That's what was unacceptable. In the meantime, we got the
>>.dsp generator, and the extractor script (for generating DEF files) that
>>makes adding a SVN_DECLARE macro unnecessary. With these tools, we can
>>make building DLLs work the way it's supposed to.
>>
>>
>>
>
>The project file in the patch did that, yes. But that's about 1% of the
>patch. The rest of the patch modified the header files.
>
Exactly. And it modified them in the wrong way, like I said.

>If you wanted multiple dll's you only had to change the project file.
>
You aren't listening. There's no way your header modifications would let
you build a libsvn_repos DLL that statically linked libsvn_fs, for example.

>I won't go any further with this discussion since it's not going anywhere.
>
>
Hah. That's really very constructive.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org