April 2022 - Wine-Commits - list.winehq.org

Giovanni Mascellani : vkd3d-shader/hlsl: Do not generate useless swizzles.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: c5c50a8b9bc7e3bcbb5caabb24b44ced59d0eb70 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=c5c50a8b9bc7e3bcbb5caab… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Apr 27 10:56:22 2022 +0200 vkd3d-shader/hlsl: Do not generate useless swizzles. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/hlsl_codegen.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index f31bfe9f..5f1d5997 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -246,6 +246,7 @@ static bool lower_broadcasts(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, v && src_type->type <= HLSL_CLASS_VECTOR && dst_type->type <= HLSL_CLASS_VECTOR && src_type->dimx == 1) { + struct hlsl_ir_node *replacement; struct hlsl_ir_swizzle *swizzle; struct hlsl_ir_expr *new_cast; @@ -255,11 +256,17 @@ static bool lower_broadcasts(struct hlsl_ctx *ctx, struct hlsl_ir_node *instr, v if (!(new_cast = hlsl_new_cast(ctx, cast->operands[0].node, dst_scalar_type, &cast->node.loc))) return false; list_add_after(&cast->node.entry, &new_cast->node.entry); - if (!(swizzle = hlsl_new_swizzle(ctx, HLSL_SWIZZLE(X, X, X, X), dst_type->dimx, &new_cast->node, &cast->node.loc))) - return false; - list_add_after(&new_cast->node.entry, &swizzle->node.entry); + replacement = &new_cast->node; - hlsl_replace_node(&cast->node, &swizzle->node); + if (dst_type->dimx != 1) + { + if (!(swizzle = hlsl_new_swizzle(ctx, HLSL_SWIZZLE(X, X, X, X), dst_type->dimx, replacement, &cast->node.loc))) + return false; + list_add_after(&new_cast->node.entry, &swizzle->node.entry); + replacement = &swizzle->node; + } + + hlsl_replace_node(&cast->node, replacement); return true; }

1 0

Zebediah Figura : configure: Allow specifying the Vulkan library path with VULKAN_LIBS.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: f34db84c62123f4c484a81cc0ab70aba8c55ef93 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=f34db84c62123f4c484a81c… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Apr 27 11:26:06 2022 -0500 configure: Allow specifying the Vulkan library path with VULKAN_LIBS. Do not define it for the vkd3d build, but restore support for overriding it as a configure argument. This partially reverts 99a3adcc19f2a64fad630948851616f52c00d553. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52915 Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure.ac | 1 + m4/check-vulkan.m4 | 7 ++++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index b386104a..7796ae8d 100644 --- a/configure.ac +++ b/configure.ac @@ -113,6 +113,7 @@ case $host_os in esac AC_ARG_VAR([SONAME_LIBVULKAN], [shared object name for the Vulkan library]) +AC_ARG_VAR([VULKAN_LIBS], [linker flags for the Vulkan library]) AS_IF([test "x$SONAME_LIBVULKAN" = "x"], [VKD3D_CHECK_VULKAN], [AC_DEFINE_UNQUOTED([SONAME_LIBVULKAN],["$SONAME_LIBVULKAN"],[Define to the shared object name of the Vulkan library.])]) diff --git a/m4/check-vulkan.m4 b/m4/check-vulkan.m4 index b2c3f8a1..f651a077 100644 --- a/m4/check-vulkan.m4 +++ b/m4/check-vulkan.m4 @@ -6,7 +6,8 @@ VKD3D_CHECK_SONAME([vulkan], [vkGetInstanceProcAddr],, [VKD3D_CHECK_SONAME([MoltenVK], [vkGetInstanceProcAddr], [AC_DEFINE_UNQUOTED([SONAME_LIBVULKAN], ["$ac_cv_lib_soname_MoltenVK"])], - [AC_MSG_ERROR([libvulkan and libMoltenVK not found.])]) - ]) - ]) + [AC_MSG_ERROR([libvulkan and libMoltenVK not found.])], + [$VULKAN_LIBS])], + [$VULKAN_LIBS])], + [$VULKAN_LIBS]) ])

1 0

Giovanni Mascellani : vkd3d-shader/hlsl: Simplify flow in add_array_load().
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: e2b4f01a9fa627d1fc533c27a93a8214c1466bc1 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=e2b4f01a9fa627d1fc533c2… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Apr 27 10:56:23 2022 +0200 vkd3d-shader/hlsl: Simplify flow in add_array_load(). Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/hlsl.y | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index 291f8392..968f36ae 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -587,13 +587,18 @@ static bool add_array_load(struct hlsl_ctx *ctx, struct list *instrs, struct hls const struct hlsl_type *expr_type = array->data_type; struct hlsl_type *data_type; struct hlsl_ir_constant *c; - struct hlsl_ir_node *mul; if (expr_type->type == HLSL_CLASS_ARRAY) { data_type = expr_type->e.array.type; + if (!(c = hlsl_new_uint_constant(ctx, hlsl_type_get_array_element_reg_size(data_type), &loc))) return false; + list_add_tail(instrs, &c->node.entry); + + if (!(index = hlsl_new_binary_expr(ctx, HLSL_OP2_MUL, index, &c->node))) + return false; + list_add_tail(instrs, &index->entry); } else if (expr_type->type == HLSL_CLASS_MATRIX) { @@ -604,8 +609,6 @@ static bool add_array_load(struct hlsl_ctx *ctx, struct list *instrs, struct hls else if (expr_type->type == HLSL_CLASS_VECTOR) { data_type = hlsl_get_scalar_type(ctx, expr_type->base_type); - if (!(c = hlsl_new_uint_constant(ctx, 1, &loc))) - return false; } else { @@ -616,12 +619,6 @@ static bool add_array_load(struct hlsl_ctx *ctx, struct list *instrs, struct hls return false; } - list_add_tail(instrs, &c->node.entry); - if (!(mul = hlsl_new_binary_expr(ctx, HLSL_OP2_MUL, index, &c->node))) - return false; - list_add_tail(instrs, &mul->entry); - index = mul; - return !!add_load(ctx, instrs, array, index, data_type, loc); }

1 0

Giovanni Mascellani : vkd3d-shader/hlsl: Do not allow NaNs in SM1.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: 4d181bc622a832e07fe768d248e35905d682e2d0 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=4d181bc622a832e07fe768d… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Apr 27 10:56:20 2022 +0200 vkd3d-shader/hlsl: Do not allow NaNs in SM1. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/hlsl_constant_ops.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_constant_ops.c b/libs/vkd3d-shader/hlsl_constant_ops.c index 2e16577f..d8787c21 100644 --- a/libs/vkd3d-shader/hlsl_constant_ops.c +++ b/libs/vkd3d-shader/hlsl_constant_ops.c @@ -278,10 +278,10 @@ static bool fold_div(struct hlsl_ctx *ctx, struct hlsl_ir_constant *dst, "Floating point division by zero."); } dst->value[k].f = src1->value[k].f / src2->value[k].f; - if (ctx->profile->major_version < 4 && isinf(dst->value[k].f)) + if (ctx->profile->major_version < 4 && !isfinite(dst->value[k].f)) { hlsl_error(ctx, &dst->node.loc, VKD3D_SHADER_ERROR_HLSL_DIVISION_BY_ZERO, - "Infinities are not allowed by the shader model."); + "Infinities and NaNs are not allowed by the shader model."); } break;

1 0

Giovanni Mascellani : vkd3d-shader/hlsl: Test the INT_MIN / -1 constant folding special case.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: 6b6ff48ece4320103f78ef4e6be5eb98b32bbafe URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=6b6ff48ece4320103f78ef4… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Apr 27 10:56:21 2022 +0200 vkd3d-shader/hlsl: Test the INT_MIN / -1 constant folding special case. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tests/arithmetic-int.shader_test | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/tests/arithmetic-int.shader_test b/tests/arithmetic-int.shader_test index c2eee2ba..8b6d6d7c 100644 --- a/tests/arithmetic-int.shader_test +++ b/tests/arithmetic-int.shader_test @@ -41,3 +41,19 @@ float4 main() : SV_TARGET return x % y; } + +[require] +shader model >= 4.0 + +[pixel shader] +float4 main() : SV_TARGET +{ + int x = -2147483648; + int y = -1; + + return x / y; +} + +[test] +draw quad +probe all rgba (-2147483648.0, -2147483648.0, -2147483648.0, -2147483648.0)

1 0

Giovanni Mascellani : vkd3d-shader/hlsl: Support matrix indexing.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: ff3ec5d4a54c4504c7cbf5c82ec535a74babb0a7 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=ff3ec5d4a54c4504c7cbf5c… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Apr 27 10:56:25 2022 +0200 vkd3d-shader/hlsl: Support matrix indexing. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/hlsl.y | 91 ++++++++++++++++++++++++++++++- tests/hlsl-initializer-matrix.shader_test | 8 +-- tests/hlsl-matrix-indexing.shader_test | 8 +-- 3 files changed, 96 insertions(+), 11 deletions(-) diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index d67ffb79..9eb007a3 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -581,6 +581,93 @@ static bool add_record_load(struct hlsl_ctx *ctx, struct list *instrs, struct hl return !!add_load(ctx, instrs, record, &c->node, field->type, loc); } +static struct hlsl_ir_expr *add_binary_arithmetic_expr(struct hlsl_ctx *ctx, struct list *instrs, + enum hlsl_ir_expr_op op, struct hlsl_ir_node *arg1, struct hlsl_ir_node *arg2, + const struct vkd3d_shader_location *loc); + +static struct hlsl_ir_node *add_matrix_scalar_load(struct hlsl_ctx *ctx, struct list *instrs, + struct hlsl_ir_node *matrix, struct hlsl_ir_node *x, struct hlsl_ir_node *y, + const struct vkd3d_shader_location *loc) +{ + struct hlsl_ir_node *major, *minor; + struct hlsl_ir_expr *mul, *add; + struct hlsl_ir_constant *four; + struct hlsl_ir_load *load; + struct hlsl_type *type = matrix->data_type, *scalar_type; + + scalar_type = hlsl_get_scalar_type(ctx, type->base_type); + + if (type->modifiers & HLSL_MODIFIER_ROW_MAJOR) + { + minor = x; + major = y; + } + else + { + minor = y; + major = x; + } + + if (!(four = hlsl_new_uint_constant(ctx, 4, loc))) + return NULL; + list_add_tail(instrs, &four->node.entry); + + if (!(mul = add_binary_arithmetic_expr(ctx, instrs, HLSL_OP2_MUL, &four->node, major, loc))) + return NULL; + + if (!(add = add_binary_arithmetic_expr(ctx, instrs, HLSL_OP2_ADD, &mul->node, minor, loc))) + return NULL; + + if (!(load = add_load(ctx, instrs, matrix, &add->node, scalar_type, *loc))) + return NULL; + + return &load->node; +} + +static bool add_matrix_index(struct hlsl_ctx *ctx, struct list *instrs, + struct hlsl_ir_node *matrix, struct hlsl_ir_node *index, const struct vkd3d_shader_location *loc) +{ + struct hlsl_type *mat_type = matrix->data_type, *ret_type; + struct vkd3d_string_buffer *name; + static unsigned int counter = 0; + struct hlsl_ir_load *load; + struct hlsl_ir_var *var; + unsigned int i; + + ret_type = hlsl_get_vector_type(ctx, mat_type->base_type, mat_type->dimx); + + name = vkd3d_string_buffer_get(&ctx->string_buffers); + vkd3d_string_buffer_printf(name, "<index-%u>", counter++); + var = hlsl_new_synthetic_var(ctx, name->buffer, ret_type, *loc); + vkd3d_string_buffer_release(&ctx->string_buffers, name); + if (!var) + return false; + + for (i = 0; i < mat_type->dimx; ++i) + { + struct hlsl_ir_store *store; + struct hlsl_ir_node *value; + struct hlsl_ir_constant *c; + + if (!(c = hlsl_new_uint_constant(ctx, i, loc))) + return false; + list_add_tail(instrs, &c->node.entry); + + if (!(value = add_matrix_scalar_load(ctx, instrs, matrix, &c->node, index, loc))) + return false; + + if (!(store = hlsl_new_store(ctx, var, &c->node, value, 0, *loc))) + return false; + list_add_tail(instrs, &store->node.entry); + } + + if (!(load = hlsl_new_var_load(ctx, var, *loc))) + return false; + list_add_tail(instrs, &load->node.entry); + + return true; +} + static bool add_array_load(struct hlsl_ctx *ctx, struct list *instrs, struct hlsl_ir_node *array, struct hlsl_ir_node *index, const struct vkd3d_shader_location loc) { @@ -602,9 +689,7 @@ static bool add_array_load(struct hlsl_ctx *ctx, struct list *instrs, struct hls } else if (expr_type->type == HLSL_CLASS_MATRIX) { - /* This needs to be lowered now, while we still have type information. */ - FIXME("Index of matrix type.\n"); - return false; + return add_matrix_index(ctx, instrs, array, index, &loc); } else if (expr_type->type == HLSL_CLASS_VECTOR) { diff --git a/tests/hlsl-initializer-matrix.shader_test b/tests/hlsl-initializer-matrix.shader_test index c4fafea7..790d402d 100644 --- a/tests/hlsl-initializer-matrix.shader_test +++ b/tests/hlsl-initializer-matrix.shader_test @@ -6,7 +6,7 @@ float4 main() : SV_TARGET } [test] -todo draw quad +draw quad probe all rgba (21, 22, 23, 0) @@ -18,7 +18,7 @@ float4 main() : SV_TARGET } [test] -todo draw quad +draw quad probe all rgba (21, 22, 23, 0) @@ -30,7 +30,7 @@ float4 main() : SV_TARGET } [test] -todo draw quad +draw quad probe all rgba (21, 22, 23, 24) @@ -42,7 +42,7 @@ float4 main() : SV_TARGET } [test] -todo draw quad +draw quad probe all rgba (21, 22, 31, 32) diff --git a/tests/hlsl-matrix-indexing.shader_test b/tests/hlsl-matrix-indexing.shader_test index 8057179e..a0a89829 100644 --- a/tests/hlsl-matrix-indexing.shader_test +++ b/tests/hlsl-matrix-indexing.shader_test @@ -11,7 +11,7 @@ uniform 0 float4 1.0 2.0 3.0 4.0 uniform 4 float4 5.0 6.0 7.0 8.0 uniform 8 float4 9.0 10.0 11.0 12.0 uniform 12 float4 13.0 14.0 15.0 16.0 -todo draw quad +draw quad probe all rgba (1.0, 2.0, 10.0, 15.0) [pixel shader] @@ -27,7 +27,7 @@ uniform 0 float4 1.0 2.0 3.0 4.0 uniform 4 float4 5.0 6.0 7.0 8.0 uniform 8 float4 9.0 10.0 11.0 12.0 uniform 12 float4 13.0 14.0 15.0 16.0 -todo draw quad +draw quad probe all rgba (1.0, 2.0, 10.0, 15.0) [pixel shader] @@ -43,7 +43,7 @@ uniform 0 float4 1.0 2.0 3.0 4.0 uniform 4 float4 5.0 6.0 7.0 8.0 uniform 8 float4 9.0 10.0 11.0 12.0 uniform 12 float4 13.0 14.0 15.0 16.0 -todo draw quad +draw quad probe all rgba (1.0, 5.0, 7.0, 12.0) [pixel shader] @@ -58,5 +58,5 @@ float4 main() : SV_TARGET [test] uniform 0 float4 1.0 2.0 3.0 0.0 uniform 4 float4 5.0 6.0 7.0 0.0 -todo draw quad +draw quad probe all rgba (1.0, 3.0, 6.0, 7.0)

1 0

Francisco Casas : vkd3d-shader/hlsl: Support all complex initalizers.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: vkd3d Branch: master Commit: 246ff145110672b8de1c374335162dd0466687ad URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=246ff145110672b8de1c374… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Wed Apr 27 10:56:24 2022 +0200 vkd3d-shader/hlsl: Support all complex initalizers. Signed-off-by: Francisco Casas <fcasas(a)codeweavers.com> Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d-shader/hlsl.c | 79 +++++++++++++ libs/vkd3d-shader/hlsl.h | 2 + libs/vkd3d-shader/hlsl.y | 123 ++++++--------------- libs/vkd3d-shader/hlsl_codegen.c | 2 +- tests/hlsl-initializer-flatten.shader_test | 10 +- .../hlsl-initializer-invalid-arg-count.shader_test | 4 +- tests/hlsl-initializer-local-array.shader_test | 4 +- tests/hlsl-initializer-nested.shader_test | 4 +- tests/hlsl-initializer-static-array.shader_test | 4 +- tests/hlsl-initializer-struct.shader_test | 6 +- 10 files changed, 129 insertions(+), 109 deletions(-) Diff: https://source.winehq.org/git/vkd3d.git/?a=commitdiff;h=246ff145110672b8de1…

1 0

Jeremy White : Fix a bug in the new config file setup.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: tools Branch: master Commit: 9cc41efd7bcd1d3f0bc865c612befd6290d48cec URL: https://source.winehq.org/git/tools.git/?a=commit;h=9cc41efd7bcd1d3f0bc865c… Author: Jeremy White <jwhite(a)codeweavers.com> Date: Wed Apr 27 15:25:09 2022 -0500 Fix a bug in the new config file setup. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- gitlab/gitlab-to-mail/gitlabtomail.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gitlab/gitlab-to-mail/gitlabtomail.py b/gitlab/gitlab-to-mail/gitlabtomail.py index 250f87a..dfe72e2 100755 --- a/gitlab/gitlab-to-mail/gitlabtomail.py +++ b/gitlab/gitlab-to-mail/gitlabtomail.py @@ -55,7 +55,7 @@ def send_email(message): with smtplib.SMTP(settings.SMTP_DOMAIN, settings.SMTP_PORT) as smtp: smtp.starttls() - if hasattr(settings, 'SMTP_USER') and hasattr(settings, 'SMTP_PASSWORD'): + if settings.SMTP_USER is not None and settings.SMTP_PASSWORD is not None: smtp.login(settings.SMTP_USER, settings.SMTP_PASSWORD) smtp.sendmail(message['From'], message['To'], encoded_message)

1 0

Jeremy White : Add a todo for db.py cleanup.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: tools Branch: master Commit: 96295d51e76b97cdf9d9e6ddbf9a78fd202f0bcd URL: https://source.winehq.org/git/tools.git/?a=commit;h=96295d51e76b97cdf9d9e6d… Author: Jeremy White <jwhite(a)codeweavers.com> Date: Wed Apr 27 09:11:07 2022 -0500 Add a todo for db.py cleanup. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- gitlab/gitlab-to-mail/README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/gitlab/gitlab-to-mail/README.md b/gitlab/gitlab-to-mail/README.md index 9f3f15b..b1d966a 100644 --- a/gitlab/gitlab-to-mail/README.md +++ b/gitlab/gitlab-to-mail/README.md @@ -17,5 +17,6 @@ * Run getmail in a loop similar to the one above / as a cron job. ## TODOs: + * Make db.py into a class that allows it to be passed the database path * Maybe have replies from the mailing list be 'unresolved'? * Super difficult: have responses from the mailing list generate the diff line_range junk

1 0

Jeremy White : Make the database location configurable.
by Alexandre Julliard April 27, 2022

April 27, 2022

Module: tools Branch: master Commit: 195e2f6d6f18e63b7bef4f2f9bbf88a21bf0d855 URL: https://source.winehq.org/git/tools.git/?a=commit;h=195e2f6d6f18e63b7bef4f2… Author: Jeremy White <jwhite(a)codeweavers.com> Date: Tue Apr 26 13:29:02 2022 -0500 Make the database location configurable. Rather than being implicity based on argv[1] Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- gitlab/gitlab-to-mail/db.py | 7 ++++++- gitlab/gitlab-to-mail/settings_example.py | 3 +++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/gitlab/gitlab-to-mail/db.py b/gitlab/gitlab-to-mail/db.py index 118d3c7..472026c 100644 --- a/gitlab/gitlab-to-mail/db.py +++ b/gitlab/gitlab-to-mail/db.py @@ -1,8 +1,13 @@ from sqlalchemy import create_engine, Column, Integer, Unicode, DateTime, Date from sqlalchemy.orm import declarative_base, sessionmaker +import importlib +import os import sys +sys.path.append(os.getcwd()) +settings = importlib.import_module(f"settings_{sys.argv[1]}") + Base = declarative_base() @@ -25,7 +30,7 @@ class GlobalState(Base): integer_value = Column(Integer) -engine = create_engine(f"sqlite:///db_{sys.argv[1]}.sqlite", echo=False) +engine = create_engine(f"sqlite:///{settings.DATABASE}", echo=False) Base.metadata.create_all(engine) Session = sessionmaker(bind=engine) session = Session() diff --git a/gitlab/gitlab-to-mail/settings_example.py b/gitlab/gitlab-to-mail/settings_example.py index fca80a2..1a42216 100644 --- a/gitlab/gitlab-to-mail/settings_example.py +++ b/gitlab/gitlab-to-mail/settings_example.py @@ -1,3 +1,6 @@ +# Location of the sqlite database file, which will be auto created if not found +DATABASE= "/home/wine/example/db_example.sqlite" + # if True mails are dumped to mail_dump mbox file instead of being sent out DEBUG = False

1 0