<br><br><div class="gmail_quote">El 26 de marzo de 2012 13:38, Marvin <span dir="ltr"><<a href="mailto:testbot@testbot.winehq.org">testbot@testbot.winehq.org</a>></span> escribió:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<br>
While running your changed tests on Windows, I think I found new failures.<br>
Being a bot and all I'm not very good at pattern recognition, so I might be<br>
wrong, but could you please double-check?<br>
Full results can be found at<br>
<a href="http://testbot.winehq.org/JobDetails.pl?Key=17479" target="_blank">http://testbot.winehq.org/JobDetails.pl?Key=17479</a><br>
<br>
Your paranoid android.<br>
<br>
<br>
=== WNT4WSSP6 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W2KPROSP4 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== WXPPROSP3 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W2K3R2SESP2 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== WVISTAADM (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W2K8SE (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W7PRO (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W7PROX64 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== TEST64_W7SP1 (32 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== W7PROX64 (64 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
<br>
=== TEST64_W7SP1 (64 bit db) ===<br>
db.c:1079: Test failed: MsiViewModify failed<br>
</blockquote></div><br>In my checkout there was several errors before applying the path and I didn't noticed this one. The primary key 2 is used later in the test and changing it to a value that's not used later is enough. The diff with the respect of the previous patch is:<div>

<br><div><div>diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c</div><div>index f5a3c1e..4c91bd8 100644</div><div>--- a/dlls/msi/tests/db.c</div><div>+++ b/dlls/msi/tests/db.c</div><div>@@ -935,7 +935,7 @@ static void test_viewmodify(void)</div>

<div>     /* try merging a new record */</div><div>     hrec = MsiCreateRecord(3);</div><div> </div><div>-    r = MsiRecordSetInteger(hrec, 1, 2);</div><div>+    r = MsiRecordSetInteger(hrec, 1, 10);</div><div>     ok(r == ERROR_SUCCESS, "failed to set integer\n");</div>

<div>     r = MsiRecordSetString(hrec, 2, "pepe");</div><div>     ok(r == ERROR_SUCCESS, "failed to set string\n");</div><div><br></div><div><br></div>-- <br>Andoni Morales Alastruey<br><br>LongoMatch:The Digital Coach<br>

<a href="http://www.longomatch.ylatuya.es">http://www.longomatch.ylatuya.es</a><br>
</div></div>