Mercurial > hg
annotate tests/test-diff-binary-file.t @ 26375:3686fa2b8eee
windows: insert file positioning call between reads and writes
fopen() and fdopen() have a unique-to-Windows requirement that
transitions between read and write operations in files opened
in modes r+, w+, and a+ perform a file positioning call
(fsetpos, fseek, or rewind) in between. While the MSDN docs don't
say what will happen if this is not done, observations reveal
that Python raises an IOError with errno 0. Furthermore, I
/think/ this behavior isn't deterministic. But I can reproduce
it reliably with subsequent patches applied that open revlogs
in a+ mode and perform both reads and writes.
This patch introduces a proxy class for file handles opened
in r+, w+, and a+ mode on Windows. The class intercepts calls
and audits whether a file positioning function has been called
between read and write operations. If not, a dummy, no-op seek
to the current file position is performed. This appears to be
sufficient to "trick" Windows into allowing transitions between
read and writes without raising errors.
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Sun, 27 Sep 2015 18:46:53 -0700 |
parents | 0db6810e84e8 |
children | 186f2afe9919 |
rev | line source |
---|---|
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
1 $ hg init a |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
2 $ cd a |
16350
4f795f5fbb0b
tests: make tests work if directory contains special characters
Thomas Arendsen Hein <thomas@intevation.de>
parents:
12151
diff
changeset
|
3 $ cp "$TESTDIR/binfile.bin" . |
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
4 $ hg add binfile.bin |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
5 $ hg ci -m 'add binfile.bin' |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
6 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
7 $ echo >> binfile.bin |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
8 $ hg ci -m 'change binfile.bin' |
4104
0934fef871f3
add test for diffing identical binary files
Alexis S. L. Carvalho <alexis@cecm.usp.br>
parents:
diff
changeset
|
9 |
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
10 $ hg revert -r 0 binfile.bin |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
11 $ hg ci -m 'revert binfile.bin' |
23924
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
12 $ hg cp binfile.bin nonbinfile |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
13 $ echo text > nonbinfile |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
14 $ hg ci -m 'make non-binary copy of binary file' |
4104
0934fef871f3
add test for diffing identical binary files
Alexis S. L. Carvalho <alexis@cecm.usp.br>
parents:
diff
changeset
|
15 |
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
16 $ hg diff --nodates -r 0 -r 1 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
17 diff -r 48b371597640 -r acea2ab458c8 binfile.bin |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
18 Binary file binfile.bin has changed |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
19 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
20 $ hg diff --nodates -r 0 -r 2 |
4104
0934fef871f3
add test for diffing identical binary files
Alexis S. L. Carvalho <alexis@cecm.usp.br>
parents:
diff
changeset
|
21 |
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
22 $ hg diff --git -r 0 -r 1 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
23 diff --git a/binfile.bin b/binfile.bin |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
24 index 37ba3d1c6f17137d9c5f5776fa040caf5fe73ff9..58dc31a9e2f40f74ff3b45903f7d620b8e5b7356 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
25 GIT binary patch |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
26 literal 594 |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
27 zc$@)J0<HatP)<h;3K|Lk000e1NJLTq000mG000mO0ssI2kdbIM00009a7bBm000XU |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
28 z000XU0RWnu7ytkO2XskIMF-Uh9TW;VpMjwv0005-Nkl<ZD9@FWPs=e;7{<>W$NUkd |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
29 zX$nnYLt$-$V!?uy+1V%`z&Eh=ah|duER<4|QWhju3gb^nF*8iYobxWG-qqXl=2~5M |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
30 z*IoDB)sG^CfNuoBmqLTVU^<;@nwHP!1wrWd`{(mHo6VNXWtyh{alzqmsH*yYzpvLT |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
31 zLdY<T=ks|woh-`&01!ej#(xbV1f|pI*=%;d-%F*E*X#ZH`4I%6SS+$EJDE&ct=8po |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
32 ziN#{?_j|kD%Cd|oiqds`xm@;oJ-^?NG3Gdqrs?5u*zI;{nogxsx~^|Fn^Y?Gdc6<; |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
33 zfMJ+iF1J`LMx&A2?dEwNW8ClebzPTbIh{@$hS6*`kH@1d%Lo7fA#}N1)oN7`gm$~V |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
34 z+wDx#)OFqMcE{s!JN0-xhG8ItAjVkJwEcb`3WWlJfU2r?;Pd%dmR+q@mSri5q9_W- |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
35 zaR2~ECX?B2w+zELozC0s*6Z~|QG^f{3I#<`?)Q7U-JZ|q5W;9Q8i_=pBuSzunx=U; |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
36 z9C)5jBoYw9^?EHyQl(M}1OlQcCX>lXB*ODN003Z&P17_@)3Pi=i0wb04<W?v-u}7K |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
37 zXmmQA+wDgE!qR9o8jr`%=ab_&uh(l?R=r;Tjiqon91I2-hIu?57~@*4h7h9uORK#= |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
38 gQItJW-{SoTm)8|5##k|m00000NkvXXu0mjf3JwksH2?qr |
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
39 |
4104
0934fef871f3
add test for diffing identical binary files
Alexis S. L. Carvalho <alexis@cecm.usp.br>
parents:
diff
changeset
|
40 |
12151
cc5b71840148
tests: unify test-diff-binary-file
Adrian Buehlmann <adrian@cadifra.com>
parents:
8167
diff
changeset
|
41 $ hg diff --git -r 0 -r 2 |
4104
0934fef871f3
add test for diffing identical binary files
Alexis S. L. Carvalho <alexis@cecm.usp.br>
parents:
diff
changeset
|
42 |
21790
3fbef7ac26f0
diff: add nobinary config to suppress git-style binary diffs
Stephen Lee <sphen.lee@gmail.com>
parents:
16913
diff
changeset
|
43 $ hg diff --config diff.nobinary=True --git -r 0 -r 1 |
3fbef7ac26f0
diff: add nobinary config to suppress git-style binary diffs
Stephen Lee <sphen.lee@gmail.com>
parents:
16913
diff
changeset
|
44 diff --git a/binfile.bin b/binfile.bin |
3fbef7ac26f0
diff: add nobinary config to suppress git-style binary diffs
Stephen Lee <sphen.lee@gmail.com>
parents:
16913
diff
changeset
|
45 Binary file binfile.bin has changed |
3fbef7ac26f0
diff: add nobinary config to suppress git-style binary diffs
Stephen Lee <sphen.lee@gmail.com>
parents:
16913
diff
changeset
|
46 |
23924
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
47 $ hg diff --git -r 2 -r 3 |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
48 diff --git a/binfile.bin b/nonbinfile |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
49 copy from binfile.bin |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
50 copy to nonbinfile |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
51 index 37ba3d1c6f17137d9c5f5776fa040caf5fe73ff9..8e27be7d6154a1f68ea9160ef0e18691d20560dc |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
52 GIT binary patch |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
53 literal 5 |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
54 Mc$_OqttjCF00uV!&;S4c |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
55 |
0db6810e84e8
diff: use binary diff when copy source is binary
Martin von Zweigbergk <martinvonz@google.com>
parents:
21790
diff
changeset
|
56 |
16913
f2719b387380
tests: add missing trailing 'cd ..'
Mads Kiilerich <mads@kiilerich.com>
parents:
16350
diff
changeset
|
57 $ cd .. |