search

dsuni / svndumpsanitizer

A program aspiring to be a more advanced version of svndumpfilter
https://miria.homelinuxserver.org/svndumpsanitizer/
GNU General Public License v3.0
48 stars 15 forks source link

Output dump file broken when it's printed to STDOUT #9

Closed kleuter closed 9 years ago

kleuter commented 9 years ago

It's correct when printed with -o option

PS. Thank you for the tool, it's unique and works great

dsuni commented 9 years ago

I'm unable to reproduce this one. When using my test dump files, the results are identical when svndumpsanitizer is invoked using "-o" or stdout is directed to a file. I would be interested in looking at a diff between two files created in identical ways, but with different ways of output.

Notice that if you just output to the screen, and copy-paste from there the result will be incorrect, because you will get the status messages (printed to stderr) mixed in there as well.

kleuter commented 9 years ago

I compiled the app with VS 2013 under Windows 7, 32-bit Can this be an issue?

kleuter commented 9 years ago

Just reproduced: When using STDOUT, dump size 441 481 030 bytes When using -o, dump size 436 927 750 bytes

dsuni commented 9 years ago

It's possible that this is a windows issue - it wouldn't be the first time it behaved... ehm... in a special manner. I really have no clue what it could be, though. Since I don't have a windows machine to test on (and don't want one) I'll probably just document the issue, given that an easy workaround exists.

dsuni commented 9 years ago

Thanks for the work. :-)

kleuter commented 9 years ago

My pleasure