wine-commits October 2013

wine-commits@winehq.org

2 participants
757 discussions

Henri Verbeet : wined3d: Send stream frequency updates through the command stream.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: 264256970b8cde073987cb19258fb35596becb78 URL: http://source.winehq.org/git/wine.git/?a=commit;h=264256970b8cde073987cb192… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 3 08:54:11 2013 +0200 wined3d: Send stream frequency updates through the command stream. --- dlls/wined3d/cs.c | 35 +++++++++++++++++++++++++++++++++++ dlls/wined3d/device.c | 2 +- dlls/wined3d/wined3d_private.h | 2 ++ 3 files changed, 38 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/cs.c b/dlls/wined3d/cs.c index 6919e91..5add94d 100644 --- a/dlls/wined3d/cs.c +++ b/dlls/wined3d/cs.c @@ -34,6 +34,7 @@ enum wined3d_cs_op WINED3D_CS_OP_SET_DEPTH_STENCIL, WINED3D_CS_OP_SET_VERTEX_DECLARATION, WINED3D_CS_OP_SET_STREAM_SOURCE, + WINED3D_CS_OP_SET_STREAM_SOURCE_FREQ, }; struct wined3d_cs_present @@ -108,6 +109,14 @@ struct wined3d_cs_set_stream_source UINT stride; }; +struct wined3d_cs_set_stream_source_freq +{ + enum wined3d_cs_op opcode; + UINT stream_idx; + UINT frequency; + UINT flags; +}; + static void wined3d_cs_exec_present(struct wined3d_cs *cs, const void *data) { const struct wined3d_cs_present *op = data; @@ -355,6 +364,31 @@ void wined3d_cs_emit_set_stream_source(struct wined3d_cs *cs, UINT stream_idx, cs->ops->submit(cs); } +static void wined3d_cs_exec_set_stream_source_freq(struct wined3d_cs *cs, const void *data) +{ + const struct wined3d_cs_set_stream_source_freq *op = data; + struct wined3d_stream_state *stream; + + stream = &cs->state.streams[op->stream_idx]; + stream->frequency = op->frequency; + stream->flags = op->flags; + + device_invalidate_state(cs->device, STATE_STREAMSRC); +} + +void wined3d_cs_emit_set_stream_source_freq(struct wined3d_cs *cs, UINT stream_idx, UINT frequency, UINT flags) +{ + struct wined3d_cs_set_stream_source_freq *op; + + op = cs->ops->require_space(cs, sizeof(*op)); + op->opcode = WINED3D_CS_OP_SET_STREAM_SOURCE_FREQ; + op->stream_idx = stream_idx; + op->frequency = frequency; + op->flags = flags; + + cs->ops->submit(cs); +} + static void (* const wined3d_cs_op_handlers[])(struct wined3d_cs *cs, const void *data) = { /* WINED3D_CS_OP_PRESENT */ wined3d_cs_exec_present, @@ -366,6 +400,7 @@ static void (* const wined3d_cs_op_handlers[])(struct wined3d_cs *cs, const void /* WINED3D_CS_OP_SET_DEPTH_STENCIL */ wined3d_cs_exec_set_depth_stencil, /* WINED3D_CS_OP_SET_VERTEX_DECLARATION */ wined3d_cs_exec_set_vertex_declaration, /* WINED3D_CS_OP_SET_STREAM_SOURCE */ wined3d_cs_exec_set_stream_source, + /* WINED3D_CS_OP_SET_STREAM_SOURCE_FREQ */ wined3d_cs_exec_set_stream_source_freq, }; static void *wined3d_cs_st_require_space(struct wined3d_cs *cs, size_t size) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 6b6234b..603e5f5 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1308,7 +1308,7 @@ HRESULT CDECL wined3d_device_set_stream_source_freq(struct wined3d_device *devic if (device->recording) device->recording->changed.streamFreq |= 1 << stream_idx; else if (stream->frequency != old_freq || stream->flags != old_flags) - device_invalidate_state(device, STATE_STREAMSRC); + wined3d_cs_emit_set_stream_source_freq(device->cs, stream_idx, stream->frequency, stream->flags); return WINED3D_OK; } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 719aa87..c7b014e 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2492,6 +2492,8 @@ void wined3d_cs_emit_set_render_target(struct wined3d_cs *cs, UINT render_target void wined3d_cs_emit_set_scissor_rect(struct wined3d_cs *cs, const RECT *rect) DECLSPEC_HIDDEN; void wined3d_cs_emit_set_stream_source(struct wined3d_cs *cs, UINT stream_idx, struct wined3d_buffer *buffer, UINT offset, UINT stride) DECLSPEC_HIDDEN; +void wined3d_cs_emit_set_stream_source_freq(struct wined3d_cs *cs, UINT stream_idx, + UINT frequency, UINT flags) DECLSPEC_HIDDEN; void wined3d_cs_emit_set_vertex_declaration(struct wined3d_cs *cs, struct wined3d_vertex_declaration *declaration) DECLSPEC_HIDDEN; void wined3d_cs_emit_set_viewport(struct wined3d_cs *cs, const struct wined3d_viewport *viewport) DECLSPEC_HIDDEN;

1 0

Frédéric Delanoy : gdiplus: Use BOOL type where appropriate.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: 70bc6d5f7854b55a9febacb8f3883afaca6e6162 URL: http://source.winehq.org/git/wine.git/?a=commit;h=70bc6d5f7854b55a9febacb8f… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu Oct 3 00:08:58 2013 +0200 gdiplus: Use BOOL type where appropriate. --- dlls/gdiplus/graphics.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index bd9b70b..1b9d0e9 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -621,7 +621,7 @@ static ARGB transform_color(ARGB color, const ColorMatrix *matrix) return (a << 24) | (r << 16) | (g << 8) | b; } -static int color_is_gray(ARGB color) +static BOOL color_is_gray(ARGB color) { unsigned char r, g, b; @@ -944,12 +944,12 @@ static REAL intersect_line_scanline(const GpPointF *p1, const GpPointF *p2, REAL return (p1->X - p2->X) * (p2->Y - y) / (p2->Y - p1->Y) + p2->X; } -static INT brush_can_fill_path(GpBrush *brush) +static BOOL brush_can_fill_path(GpBrush *brush) { switch (brush->bt) { case BrushTypeSolidColor: - return 1; + return TRUE; case BrushTypeHatchFill: { GpHatch *hatch = (GpHatch*)brush; @@ -960,7 +960,7 @@ static INT brush_can_fill_path(GpBrush *brush) case BrushTypeTextureFill: /* Gdi32 isn't much help with these, so we should use brush_fill_pixels instead. */ default: - return 0; + return FALSE; } } @@ -1012,7 +1012,7 @@ static void brush_fill_path(GpGraphics *graphics, GpBrush* brush) } } -static INT brush_can_fill_pixels(GpBrush *brush) +static BOOL brush_can_fill_pixels(GpBrush *brush) { switch (brush->bt) { @@ -1021,9 +1021,9 @@ static INT brush_can_fill_pixels(GpBrush *brush) case BrushTypeLinearGradient: case BrushTypeTextureFill: case BrushTypePathGradient: - return 1; + return TRUE; default: - return 0; + return FALSE; } }

1 0

Frédéric Delanoy : kernel32: Use BOOL type where appropriate.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: baf491b80bf625d821929803a871a892bc216e7f URL: http://source.winehq.org/git/wine.git/?a=commit;h=baf491b80bf625d821929803a… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu Oct 3 00:08:57 2013 +0200 kernel32: Use BOOL type where appropriate. --- dlls/kernel32/process.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 6b4d9b4..afbdf72 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -84,7 +84,7 @@ typedef struct static DWORD shutdown_flags = 0; static DWORD shutdown_priority = 0x280; static BOOL is_wow64; -static const int is_win64 = (sizeof(void *) > sizeof(int)); +static const BOOL is_win64 = (sizeof(void *) > sizeof(int)); HMODULE kernel32_handle = 0; SYSTEM_BASIC_INFORMATION system_info = { 0 }; @@ -116,7 +116,7 @@ extern void SHELL_LoadRegistry(void); /*********************************************************************** * contains_path */ -static inline int contains_path( LPCWSTR name ) +static inline BOOL contains_path( LPCWSTR name ) { return ((*name && (name[1] == ':')) || strchrW(name, '/') || strchrW(name, '\\')); } @@ -128,7 +128,7 @@ static inline int contains_path( LPCWSTR name ) * Check if an environment variable needs to be handled specially when * passed through the Unix environment (i.e. prefixed with "WINE"). */ -static inline int is_special_env_var( const char *var ) +static inline BOOL is_special_env_var( const char *var ) { return (!strncmp( var, "PATH=", sizeof("PATH=")-1 ) || !strncmp( var, "PWD=", sizeof("PWD=")-1 ) ||

1 0

Frédéric Delanoy : itss: Use BOOL type where appropriate.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: 239c160452dd8494a739b3bf9074a1539504b4aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=239c160452dd8494a739b3bf9… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu Oct 3 00:08:56 2013 +0200 itss: Use BOOL type where appropriate. --- dlls/itss/chm_lib.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/itss/chm_lib.c b/dlls/itss/chm_lib.c index c9bbcdf..c1be36e 100644 --- a/dlls/itss/chm_lib.c +++ b/dlls/itss/chm_lib.c @@ -1424,8 +1424,8 @@ int chm_enumerate_dir(struct chmFile *h, UChar *cur; unsigned int lenRemain; - /* set to 1 once we've started */ - int it_has_begun=0; + /* set to TRUE once we've started */ + BOOL it_has_begun = FALSE; /* the current ui */ struct chmUnitInfo ui; @@ -1493,7 +1493,7 @@ int chm_enumerate_dir(struct chmFile *h, if (! it_has_begun) { if (ui.length == 0 && strncmpiW(ui.path, prefixRectified, prefixLen) == 0) - it_has_begun = 1; + it_has_begun = TRUE; else continue;

1 0

Frédéric Delanoy : gdi32/tests: Use BOOL type where appropriate.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: 3b2f85e0b63c6fb70e26efe6cbe480128200eea0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3b2f85e0b63c6fb70e26efe6c… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu Oct 3 00:08:55 2013 +0200 gdi32/tests: Use BOOL type where appropriate. --- dlls/gdi32/tests/bitmap.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdi32/tests/bitmap.c b/dlls/gdi32/tests/bitmap.c index 6b52356..dfa8711 100644 --- a/dlls/gdi32/tests/bitmap.c +++ b/dlls/gdi32/tests/bitmap.c @@ -1564,7 +1564,7 @@ static COLORREF get_nearest( int r, int g, int b ) return (r*r + g*g + b*b < (255-r)*(255-r) + (255-g)*(255-g) + (255-b)*(255-b)) ? 0x000000 : 0xffffff; } -static int is_black_pen( COLORREF fg, COLORREF bg, int r, int g, int b ) +static BOOL is_black_pen( COLORREF fg, COLORREF bg, int r, int g, int b ) { if (fg == 0 || bg == 0xffffff) return RGB(r,g,b) != 0xffffff && RGB(r,g,b) != bg; return RGB(r,g,b) == 0x000000 || RGB(r,g,b) == bg;

1 0

Julian Rüger : po: Update German translation.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: wine Branch: master Commit: 11431e2f9a74a71443514db3f24986bf3ec0c1c7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11431e2f9a74a71443514db3f… Author: Julian Rüger <jr98(a)gmx.net> Date: Wed Oct 2 23:41:02 2013 +0200 po: Update German translation. --- po/de.po | 55 +++++++++++++++++-------------------------------------- 1 files changed, 17 insertions(+), 38 deletions(-) diff --git a/po/de.po b/po/de.po index 9efd3a4..36a2ec7 100644 --- a/po/de.po +++ b/po/de.po @@ -8552,17 +8552,12 @@ msgstr "" "Übertragung abgeschlossen ist und versuchen Sie es erneut." #: winmm.rc:52 winmm.rc:125 -#, fuzzy -#| msgid "" -#| "The current MIDI Mapper setup refers to a MIDI device that is not " -#| "installed on the system. Use MIDI Mapper to edit the setup." msgid "" "The current MIDI Mapper setup refers to a MIDI device that is not installed " "on the system." msgstr "" "Die gegenwärtigen MIDI-Mapper Einstellungen beziehen sich auf ein MIDI-" -"Gerät, das nicht installiert ist. Benutzen Sie den MIDI-Mapper, um dies zu " -"ändern." +"Gerät, das nicht installiert ist." #: winmm.rc:53 msgid "" @@ -9409,22 +9404,16 @@ msgstr "" "einer gerufenen Prozedur werden von der aufrufenden geerbt.\n" #: cmd.rc:40 -#, fuzzy -#| msgid "" -#| "CD <dir> is the short version of CHDIR. It changes the current\n" -#| "default directory.\n" msgid "" "CD <directory> is the short version of CHDIR. It changes the current\n" "default directory.\n" msgstr "" -"CD <Ver> ist die Kurzform von CHDIR. Es wechselt das aktuelle\n" +"CD <Verzeichnis> ist die Kurzform von CHDIR. Es wechselt das aktuelle\n" "Verzeichnis.\n" #: cmd.rc:41 -#, fuzzy -#| msgid "CHDIR <dir> changes the current default directory.\n" msgid "CHDIR <directory> changes the current default directory.\n" -msgstr "CHDIR Wechselt in ein Verzeichnis.\n" +msgstr "CHDIR <Verzeichnis> wechselt in ein Verzeichnis.\n" #: cmd.rc:43 msgid "CLS clears the console screen.\n" @@ -9432,7 +9421,7 @@ msgstr "CLS löscht den Bildschirminhalt der Konsole.\n" #: cmd.rc:45 msgid "COPY <filename> copies a file.\n" -msgstr "COPY Kopiert eine Datei.\n" +msgstr "COPY <Dateiname> kopiert eine Datei.\n" #: cmd.rc:46 msgid "CTTY changes the input/output device.\n" @@ -9444,7 +9433,7 @@ msgstr "DATE zeigt oder ändert das Systemdatum.\n" #: cmd.rc:48 msgid "DEL <filename> deletes a file or set of files.\n" -msgstr "DEL löscht eine oder mehrere Dateien.\n" +msgstr "DEL <Dateiname> löscht eine oder mehrere Dateien.\n" #: cmd.rc:49 msgid "DIR lists the contents of a directory.\n" @@ -9473,14 +9462,9 @@ msgstr "" #: cmd.rc:61 msgid "ERASE <filename> deletes a file or set of files.\n" -msgstr "ERASE löscht die angegebenen Dateien.\n" +msgstr "ERASE <Dateiname> löscht die angegebenen Dateien.\n" #: cmd.rc:68 -#, fuzzy -#| msgid "" -#| "The FOR command is used to execute a command for each of a set of files.\n" -#| "\n" -#| "Syntax: FOR %variable IN (set) DO command\n" msgid "" "The FOR command is used to execute a command for each of a set of files.\n" "\n" @@ -9488,10 +9472,13 @@ msgid "" "\n" "The % sign must be doubled when using FOR in a batch file.\n" msgstr "" -"Der FOR Befehl wird genutzt, um einen Befehl für eine Menge von Dateien\n" -"einzeln auszuführen.\n" +"Der FOR-Befehl wird genutzt, um einen Befehl für jede Datei in einer Menge\n" +"von Dateien einzeln auszuführen.\n" "\n" -"Syntax: FOR %Variable IN (set) DO Befehl\n" +"Syntax: FOR %Variable IN (Menge) DO Befehl\n" +"\n" +"Das %-Zeichen muss doppelt angegeben werden, wenn FOR in Batchdateien\n" +"verwendet wird.\n" #: cmd.rc:80 msgid "" @@ -9523,8 +9510,8 @@ msgid "" "HELP <command> shows brief help details on a topic.\n" "HELP without an argument shows all CMD built-in commands.\n" msgstr "" -"HELP <Kommando> zeigt eine kurze Hilfe zu einem Thema.\n" -"HELP ohne Argumente zeigt alle CMD Kommandos.\n" +"HELP <Befehl> zeigt eine kurze Hilfe zu einem Thema.\n" +"HELP ohne Argumente zeigt alle eingebauten CMD-Befehle.\n" #: cmd.rc:93 msgid "" @@ -9688,16 +9675,12 @@ msgid "RENAME <filename> renames a file.\n" msgstr "RENAME <Dateiname> benennt die angegebenen Dateien um.\n" #: cmd.rc:158 -#, fuzzy -#| msgid "RD <dir> is the short version of RMDIR. It deletes a subdirectory.\n" msgid "RD <directory> is the short version of RMDIR. It deletes a directory.\n" -msgstr "RD ist die Kurzform von RMDIR.\n" +msgstr "RD <Verzeichnis> ist die Kurzform von RMDIR. Löscht ein Verzeichnis.\n" #: cmd.rc:159 -#, fuzzy -#| msgid "RMDIR <dir> deletes a subdirectory.\n" msgid "RMDIR <directory> deletes a directory.\n" -msgstr "RMDIR löscht das angegebene Verzeichnis.\n" +msgstr "RMDIR <Verzeichnis> löscht das angegebene Verzeichnis.\n" #: cmd.rc:203 msgid "" @@ -9874,15 +9857,11 @@ msgstr "" "Ab diesem Punkt wird die vorherige Umgebung wiederhergestellt.\n" #: cmd.rc:245 -#, fuzzy -#| msgid "" -#| "PUSHD <directoryname> saves the current directory onto a\n" -#| "stack, and then changes the current directory to the supplied one.\n" msgid "" "PUSHD <directory> saves the current directory onto a\n" "stack, and then changes the current directory to the supplied one.\n" msgstr "" -"PUSHD <Verzeichnisname> speichert das aktuelle Verzeichnis auf einem\n" +"PUSHD <Verzeichnis> speichert das aktuelle Verzeichnis auf einem\n" "Stack und wechselt das Arbeitsverzeichnis zu dem angegebenen.\n" #: cmd.rc:247

1 0

Francois Gouget : testbot: Rework the job scheduler and add an 'off' VM state.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: tools Branch: master Commit: 2ab859b9a8dbba9a5ab8c7d295aebecfdd31fdc5 URL: http://source.winehq.org/git/tools.git/?a=commit;h=2ab859b9a8dbba9a5ab8c7d2… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Oct 3 12:23:54 2013 +0200 testbot: Rework the job scheduler and add an 'off' VM state. The old scheduler would limit the number of VMs in the running state to $MaxRunningVMs but would allow dirty base VMs to accumulate without bound. The new scheduler makes sure that no more than $MaxActiveVMs VMs are using memory, CPU or I/O resources, whether they are reverting, sleeping, running or dirty. Furthermore the new scheduler will power off any dirty VM that it cannot revert right away to make sure it won't waste resources. The run tasks also no longer power off the VMs when done. They now leave that task to the scheduler which will decide what to do with the VM. The new scheduler can revert VMs while others are running. This case gets its own limit, $MaxRevertWhileRunningVMs, so the reverts do not disturb the tests. To better exploit this it can look ahead and prepare the VMs needed for a job's next steps. And like the previous scheduler, when idle it will start a set of VMs for the next job, trying to pick the most likely to be needed: first the build VM, then base VMs, etc. This can also gets its own limit, $MaxVMsWhenIdle, so developpers can set the TestBot to not use resources unless needed. This can lead to having idle VMs that turn out to not be needed, for instance if the new job targets non base VMs. When that happens the scheduler can shut down some idle VMs to make room for more useful ones. --- testbot/bin/Engine.pl | 12 +- testbot/bin/WineRunTask.pl | 2 - testbot/ddl/update26.sql | 4 + testbot/lib/WineTestBot/Config.pm | 17 ++- testbot/lib/WineTestBot/Jobs.pm | 262 +++++++++++++++++++++++++++---------- testbot/lib/WineTestBot/VMs.pm | 16 ++- 6 files changed, 221 insertions(+), 92 deletions(-) Diff: http://source.winehq.org/git/tools.git/?a=commitdiff;h=2ab859b9a8dbba9a5ab8…

1 0

Francois Gouget : testbot: Restrict the use of WineTestBot:: Log in the Jobs module.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: tools Branch: master Commit: 67333a424676b24d86b12e8a669b79a52f62a580 URL: http://source.winehq.org/git/tools.git/?a=commit;h=67333a424676b24d86b12e8a… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Oct 3 12:18:19 2013 +0200 testbot: Restrict the use of WineTestBot::Log in the Jobs module. --- testbot/lib/WineTestBot/Jobs.pm | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm index 48e74f9..86bb0c3 100644 --- a/testbot/lib/WineTestBot/Jobs.pm +++ b/testbot/lib/WineTestBot/Jobs.pm @@ -54,7 +54,6 @@ use WineTestBot::Config; use WineTestBot::Branches; use WineTestBot::Engine::Notify; use WineTestBot::WineTestBotObjects; -use WineTestBot::Log; use vars qw(@ISA @EXPORT); @@ -206,7 +205,8 @@ sub Cancel } elsif (defined $Task->ChildPid) { - LogMsg "Canceling the ". join("/", $self->Id, $Step->No, $Task->No) . " task\n"; + require WineTestBot::Log; + WineTestBot::Log::LogMsg("Canceling the " . join("/", $self->Id, $Step->No, $Task->No) . " task\n"); kill("TERM", $Task->ChildPid); $Task->Status("canceled"); $Task->ChildPid(undef);

1 0

Francois Gouget : testbot: Some spelling fixes and wording tweaks.
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: tools Branch: master Commit: 0c79e60d6777279b311d8adc34cca0afd2aa6c87 URL: http://source.winehq.org/git/tools.git/?a=commit;h=0c79e60d6777279b311d8adc… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Oct 3 12:16:48 2013 +0200 testbot: Some spelling fixes and wording tweaks. --- testbot/bin/Engine.pl | 4 ++-- testbot/lib/WineTestBot/Jobs.pm | 2 +- testbot/lib/WineTestBot/VMs.pm | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/testbot/bin/Engine.pl b/testbot/bin/Engine.pl index 2438298..03575f6 100755 --- a/testbot/bin/Engine.pl +++ b/testbot/bin/Engine.pl @@ -73,7 +73,7 @@ It has to contend with three main scenarios: - The Engine being restarted. Any task started just before that is still running should still have its process and powered on VM and should be left alone so it can complete normally. If a task died unexpectedly while the - Engine was done it's ok to requeue it. + Engine was down it's ok to requeue it. - The Engine startup after a reboot. All task processes will be dead and need to be requeued. But the VMs are likely to be hosted on a separate machine so it is quite possible that they will still be running. Hopefully any running @@ -82,7 +82,7 @@ It has to contend with three main scenarios: - A shutdown of the Engine and its tasks / VMs. In this case it's used to kill the running tasks and requeue them, and/or shut down the VMs. -In all cases we only trust that a VM status field is still valid: +In all cases we only trust that a VM status field is still valid if: - It is 'running' and used by a task that is still running. - It is 'reverting' or 'sleeping', is powered on and the revert process is still running. diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm index 35514a7..48e74f9 100644 --- a/testbot/lib/WineTestBot/Jobs.pm +++ b/testbot/lib/WineTestBot/Jobs.pm @@ -399,7 +399,7 @@ VM Statuses. The number of VMs running on the host must be kept under $MaxRunningVMs. The rational behind this limit is that the host may not be able to run more VMs simultaneously, typically due to memory or CPU constraints. Also note that -this limit must be respected even if there are more than one hypervisor running +this limit must be respected even if there is more than one hypervisor running on the host. =item * diff --git a/testbot/lib/WineTestBot/VMs.pm b/testbot/lib/WineTestBot/VMs.pm index 37d1c4f..8069c90 100644 --- a/testbot/lib/WineTestBot/VMs.pm +++ b/testbot/lib/WineTestBot/VMs.pm @@ -535,7 +535,7 @@ sub RunRevert # Note that if the child process completes quickly (typically due to some # error), it may set ChildPid to undef before we get here. So we may end up - # with non-reverting VM for which ChildPid is set. That's ok because + # with non-reverting VMs for which ChildPid is set. That's ok because # ChildPid should be ignored anyway if Status is not 'reverting' or # 'sleeping'. $self->ChildPid($Pid);

1 0

Francois Gouget : patches: Remove bits that are unneeded now that it is integrated with the WineTestBot .
by Alexandre Julliard 03 Oct '13

03 Oct '13

Module: tools Branch: master Commit: 3a572e9fdedb04bf355111b3dbb5b13af3124ebd URL: http://source.winehq.org/git/tools.git/?a=commit;h=3a572e9fdedb04bf355111b3… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Oct 3 12:18:04 2013 +0200 patches: Remove bits that are unneeded now that it is integrated with the WineTestBot. --- patches/doc/INSTALL.txt | 12 +------ patches/filter-testbot | 83 ----------------------------------------------- patches/notify-bot | 35 -------------------- 3 files changed, 2 insertions(+), 128 deletions(-) diff --git a/patches/doc/INSTALL.txt b/patches/doc/INSTALL.txt index 86b3dca..c1570ac 100644 --- a/patches/doc/INSTALL.txt +++ b/patches/doc/INSTALL.txt @@ -58,16 +58,8 @@ Apache setup: - Run expire regularly to delete old patches: 0 0 * * * $HOME/tools/patches/expire -v $HOME/patches -To integrate with the WineTestBot also do the following: -- Run the notify-bot script regularly to notify the WineTestBot when there are - new patches to test. - */5 * * * * $HOME/tools/patches/notify-bot <winetestbot-email-address> -- Arrange for the filter-testbot script to be fed the WineTestBot emails. One - way to do so is to add the following lines to the .procmailrc file: - :0 bfw - * ^X-TestBot-Results: - | $HOME/tools/patches/filter-testbot - +For integration with the WineTestBot, see the corresponding section in the +WineTestBot documentation. 5. Setup for the regressions site diff --git a/patches/filter-testbot b/patches/filter-testbot deleted file mode 100755 index 844090c..0000000 --- a/patches/filter-testbot +++ /dev/null @@ -1,83 +0,0 @@ -#!/usr/bin/perl -w -# -# Process a testbot result email and generate the .testfail and .testbot -# result files that the patches site uses / links to. -# -# Copyright 2010 Alexandre Julliard -# -# This library is free software; you can redistribute it and/or -# modify it under the terms of the GNU Lesser General Public -# License as published by the Free Software Foundation; either -# version 2.1 of the License, or (at your option) any later version. -# -# This library is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU -# Lesser General Public License for more details. -# -# You should have received a copy of the GNU Lesser General Public -# License along with this library; if not, write to the Free Software -# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA - -use strict; - -my $step = 0; -my $patch = 0; -my $failed = 0; - -while (<>) -{ - if (/^--- BEGIN GENERAL ---$/) - { - $step = 1; - next; - } - if (/^--- BEGIN NEW_ERRORS ---$/) - { - $step = 2; - next; - } - if (/^--- BEGIN FULL_LOGS ---$/) - { - $step = 3; - next; - } - if (/^--- END GENERAL ---$/) - { - die "no patch id found" unless $patch > 0; - if ($failed) - { - open TESTFAIL, ">patches/$patch.testfail" or die "cannot create $patch.testfail"; - } - open TESTBOT, ">patches/$patch.testbot" or die "cannot create $patch.testbot"; - $step = 0; - next; - } - if (/^--- END/) - { - $step = 0; - next; - } - if ($step == 1 && /^Patch: (\d+)/) - { - $patch = $1; - next; - } - if ($step == 1 && /^Test-Result:\s+Failed/) - { - $failed = 1; - next; - } - if ($step == 2 && $failed) - { - print TESTFAIL $_; - next; - } - if ($step == 3) - { - print TESTBOT $_; - next; - } -} -close TESTFAIL if $failed; -close TESTBOT; diff --git a/patches/notify-bot b/patches/notify-bot deleted file mode 100755 index 9676c42..0000000 --- a/patches/notify-bot +++ /dev/null @@ -1,35 +0,0 @@ -#!/bin/sh -# -# Send an email to the WineTestBot to let it know of new patches. -# -# Copyright 2010 Alexandre Julliard -# -# This library is free software; you can redistribute it and/or -# modify it under the terms of the GNU Lesser General Public -# License as published by the Free Software Foundation; either -# version 2.1 of the License, or (at your option) any later version. -# -# This library is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU -# Lesser General Public License for more details. -# -# You should have received a copy of the GNU Lesser General Public -# License along with this library; if not, write to the Free Software -# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA - -notify_addr="$1" -subject="New patches available at http://source.winehq.org/patches" -test -n "$notify_addr" || exit 0 - -old=`cat $HOME/.next-patch` -next=$old -while test -f /home/winehq/opt/source/patches/data/$next -do - next=`expr $next + 1` -done -if test $next -gt $old -then - echo $next >$HOME/.next-patch - echo "The latest patch is `expr $next - 1`" | Mail -s "$subject" "$notify_addr" -fi

1 0

← Newer
1
...
66
67
68
69
70
71
72
...
76
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits October 2013