Francois Gouget : testbot: Pids can have more than 5 digits.

Alexandre Julliard julliard at winehq.org
Tue Aug 31 15:39:30 CDT 2021


Module: tools
Branch: master
Commit: f97f0e180a3f0fb2b9c7b1944eb4295603dc218e
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=f97f0e180a3f0fb2b9c7b1944eb4295603dc218e

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Aug 31 15:53:57 2021 +0200

testbot: Pids can have more than 5 digits.

Pids used to be a 16-bit value but that's no longer the case. Assume
they are an unsigned 32-bit value now.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/ddl/update46.sql       | 4 ++++
 testbot/ddl/winetestbot.sql    | 2 +-
 testbot/lib/WineTestBot/VMs.pm | 2 +-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/testbot/ddl/update46.sql b/testbot/ddl/update46.sql
new file mode 100644
index 0000000..f24130b
--- /dev/null
+++ b/testbot/ddl/update46.sql
@@ -0,0 +1,4 @@
+USE winetestbot;
+
+ALTER TABLE VMs
+  MODIFY ChildPid INT UNSIGNED NULL;
diff --git a/testbot/ddl/winetestbot.sql b/testbot/ddl/winetestbot.sql
index 3b5cda7..2d08524 100644
--- a/testbot/ddl/winetestbot.sql
+++ b/testbot/ddl/winetestbot.sql
@@ -52,7 +52,7 @@ CREATE TABLE VMs
   MissionCaps   VARCHAR(512)     NOT NULL,
   Status        ENUM('dirty', 'reverting', 'sleeping', 'idle', 'running', 'off', 'offline', 'maintenance') NOT NULL,
   Errors        INT(2)           NULL,
-  ChildPid      INT(5)           NULL,
+  ChildPid      INT UNSIGNED     NULL,
   ChildDeadline DATETIME         NULL,
   VirtURI       VARCHAR(64)      NOT NULL,
   VirtDomain    VARCHAR(32)      NOT NULL,
diff --git a/testbot/lib/WineTestBot/VMs.pm b/testbot/lib/WineTestBot/VMs.pm
index 2cc6c67..0b11d81 100644
--- a/testbot/lib/WineTestBot/VMs.pm
+++ b/testbot/lib/WineTestBot/VMs.pm
@@ -760,7 +760,7 @@ my @PropertyDescriptors = (
   CreateBasicPropertyDescriptor("MissionCaps", "Mission Capabilities", !1, !1, "A", 512),
   CreateEnumPropertyDescriptor("Status", "Current status", !1, 1, ['dirty', 'reverting', 'sleeping', 'idle', 'running', 'off', 'offline', 'maintenance']),
   CreateBasicPropertyDescriptor("Errors", "Errors", !1, !1, "N", 2),
-  CreateBasicPropertyDescriptor("ChildPid", "Child process id", !1, !1, "N", 5),
+  CreateBasicPropertyDescriptor("ChildPid", "Child process id", !1, !1, "N", 10),
   CreateBasicPropertyDescriptor("ChildDeadline", "Child Deadline", !1, !1, "DT", 19),
   CreateBasicPropertyDescriptor("VirtURI", "LibVirt URI of the VM", !1, 1, "A", 64),
   CreateBasicPropertyDescriptor("VirtDomain", "LibVirt Domain for the VM", !1, 1, "A", 32),




More information about the wine-cvs mailing list