msi: Implement MSIMODIFY_MERGE function in TABLE_modify.

Andoni Morales ylatuya at gmail.com
Mon Mar 26 07:05:59 CDT 2012


El 26 de marzo de 2012 13:38, Marvin <testbot at testbot.winehq.org> escribió:

> Hi,
>
> While running your changed tests on Windows, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> Full results can be found at
> http://testbot.winehq.org/JobDetails.pl?Key=17479
>
> Your paranoid android.
>
>
> === WNT4WSSP6 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W2KPROSP4 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === WXPPROSP3 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W2K3R2SESP2 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === WVISTAADM (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W2K8SE (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W7PRO (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W7PROX64 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === TEST64_W7SP1 (32 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === W7PROX64 (64 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>
> === TEST64_W7SP1 (64 bit db) ===
> db.c:1079: Test failed: MsiViewModify failed
>

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:

diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c
index f5a3c1e..4c91bd8 100644
--- a/dlls/msi/tests/db.c
+++ b/dlls/msi/tests/db.c
@@ -935,7 +935,7 @@ static void test_viewmodify(void)
     /* try merging a new record */
     hrec = MsiCreateRecord(3);

-    r = MsiRecordSetInteger(hrec, 1, 2);
+    r = MsiRecordSetInteger(hrec, 1, 10);
     ok(r == ERROR_SUCCESS, "failed to set integer\n");
     r = MsiRecordSetString(hrec, 2, "pepe");
     ok(r == ERROR_SUCCESS, "failed to set string\n");


-- 
Andoni Morales Alastruey

LongoMatch:The Digital Coach
http://www.longomatch.ylatuya.es
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20120326/6b1ab406/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-msi-implement-MSIMODIFY_MERGE-function-in-TABLE_modi.patch
Type: text/x-patch
Size: 2907 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20120326/6b1ab406/attachment.bin>


More information about the wine-devel mailing list