From patchwork Sat Nov 23 17:31:19 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lukas Fleischer X-Patchwork-Id: 1388 Return-Path: Delivered-To: patchwork@archlinux.org Received: from apollo.archlinux.org (localhost [127.0.0.1]) by apollo.archlinux.org (Postfix) with ESMTP id 7F486153BF2B4 for ; Sat, 23 Nov 2019 17:30:41 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on apollo.archlinux.org X-Spam-Level: X-Spam-Status: No, score=-3.4 required=5.0 tests=DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1, MAILING_LIST_MULTI=-1,RCVD_IN_DNSWL_MED=-2.3,SPF_HELO_NONE=0.001, T_DMARC_POLICY_NONE=0.01 autolearn=unavailable autolearn_force=no version=3.4.2 X-Spam-BL-Results: [127.0.9.2] [127.0.0.4] Received: from orion.archlinux.org (orion.archlinux.org [IPv6:2a01:4f8:160:6087::1]) by apollo.archlinux.org (Postfix) with ESMTPS for ; Sat, 23 Nov 2019 17:30:41 +0000 (UTC) Received: from orion.archlinux.org (localhost [127.0.0.1]) by orion.archlinux.org (Postfix) with ESMTP id 897FA16D189084; Sat, 23 Nov 2019 17:30:26 +0000 (UTC) Received: from luna.archlinux.org (luna.archlinux.org [5.9.250.164]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: luna) by orion.archlinux.org (Postfix) with ESMTPSA id F23C016D188A4B; Sat, 23 Nov 2019 17:30:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=orion; t=1574530214; bh=+B5WSQLYuH0Uh65TfKquOKiaXORLfq+9dn498omwAJY=; h=From:To:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe; b=p1dDWT4MmN0v+lQCcas1ICTRBHSIzVNsDsjnLBVGhiVIj8aqxl03HEy8noPpvoF+r Inx6S8wSrZlMEA4519NZPdmJviIaozuh3QDLID95mC8jHZpizkNzaoofOCjPInWJ67 /LhLyFs2F/+GCWgYJzu3WA0Nf62TktfNGZV+dyZQPzG0RyVWPjaoBcdkaOM8mSzDFq 8PNGONOihqRQ0nG7o2fHon98BbnNd51UsjgckEG+zJ2sYcOegr90VPRd8HuPaAn89P AXEHMBZm0D82f57SvrNeNc+SmPA89oJjlTWt2OAor5Yb3YVKjjtwtgLVahaNy12xbO lBwPU9UUSrlLO2jQ23bDT43y4Uv7SwKBGmeecZmfynPDFOggve01XNwwPr1iFEtYu6 DDF7jxjFMrTBZFFWFm5Ki29X2LKeTN6gXgGAe39We243rmzdfzUS+VIj2KE9q+04lR nHuxjmnH6iC5hBji7EwEZaiFe9q+TTrkMJ5M3XUakm5IDh7WRR6cbXWZx+aCjWd8h2 Wl0fK8DM0UHbP2PImwQybf+n7iY2wSTyhvw/WmpAwoNvMrLI/aGwwYeYa3swXn74Tt xn3HALXd1BHHDaMzGuadH6eNRoISp4Fkg/y/8G3KloDb94kI4rAckv7ctqfpVIygh2 zrLa8wLmjgQ9HeyCiSB87L5c= Received: from luna.archlinux.org (luna.archlinux.org [127.0.0.1]) by luna.archlinux.org (Postfix) with ESMTP id 506FB2B794; Sat, 23 Nov 2019 17:30:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=luna2; t=1574530212; bh=+B5WSQLYuH0Uh65TfKquOKiaXORLfq+9dn498omwAJY=; h=From:To:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe; b=quTGx5fM1l15iPLEllO7lvLzev35ifkfJ0LYME5lQddeMkNx5wChFoZDpJE4aL2kJ pb8vR5BlCAvqbECUpIpUkXTSX+ql0J0xsnefRgcHvv9EIw3zJL15e3V4a4DmTR8fqO QCq307F4bkDFHzFgLKl/ywBZWlbE/3/C+7AznU/TbLRoEIB92/3BRoD7/nPV+ctbmX SMKXZ+EhtfRxg6I1sucEadBbptKIMA4CaieajO3lv53BeHgsDqGBNkYNOKQ1zXSCdN PwFlZ4q4xHtDhRGJTpKlTPGtv5NMdyABAv7GR0CnGL/cOcke+vz8E0fCKTbDw0tsaG 5IEC3b85GP7b8BYp09D7ySxsLJRYHw065h3s5zm3ptOehotXRdmj+cQA9iDJ+V93nl 4Huj3SSubCFmwuFcX2gGDEohCa9n0/izKmoY80A/9gh22rTaTSuT0zPi4qJl92HcxH UdsBab9pDbvD4iEQI4DNDOSubU7ysD5k2KHSBFXAOXmKCd7IyR3An5FZwHAbevgyPl 8WDUeiW/6PNYeANT+pavRCcdjI0n6/TL6XC32uKyL5YX/XcZr5vAzVIoKIOwPezx5v D43TWMC3p23nNABLFvJ1j77HLLVjeu55tdB+CPvqm3YMm4UTxhimu8JnbdgHO4k2o+ sCKcfEbkuM1R4ZdiM24aFvwU= Received: from luna.archlinux.org (luna.archlinux.org [127.0.0.1]) by luna.archlinux.org (Postfix) with ESMTP id 9913A2B789 for ; Sat, 23 Nov 2019 17:30:08 +0000 (UTC) Received: from orion.archlinux.org (orion.archlinux.org [88.198.91.70]) by luna.archlinux.org (Postfix) with ESMTPS for ; Sat, 23 Nov 2019 17:30:08 +0000 (UTC) Received: from orion.archlinux.org (localhost [127.0.0.1]) by orion.archlinux.org (Postfix) with ESMTP id 2252A16D1889CF for ; Sat, 23 Nov 2019 17:30:01 +0000 (UTC) Received: from localhost (unknown [72.138.14.22]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: lfleischer) by orion.archlinux.org (Postfix) with ESMTPSA id 1A45F16D1889CD for ; Sat, 23 Nov 2019 17:29:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=orion; t=1574530200; bh=+B5WSQLYuH0Uh65TfKquOKiaXORLfq+9dn498omwAJY=; h=From:To:Subject:Date; b=A+mgTQeBGH2i5bwHpzp4CQrH42/5z8awxD8sSeUYkIKkI++V249rDLoluYnnuHlNf qv1ywk9F14210OuZxF7r9bnwBPP+IDNXJf/R2SbvsP4ir2TvNSNTAffgilXvBsugjo hWcXEI5UZ5JAG9FH265iT1VX6Gbng0fDs5CD/VJxsSvxTYktQLrtyCsy0xQfrqZ7OL rzcNDlXTYOzhh9HP/Wl3PwKFJv9qzm9F55cEl8b6DBeWR//BIo6IH9ny6lNvM6yNu8 C0KWy4RzP0t6OarEKjnwXIqg77jtMPiVL/oqZhdRf0oykolc6XCVLpP9Rjblz2bQyo VHbKZbB8s72WM1Ept9s6+wqNvryek4il/CK4SGCzDbDKgHXjavDliXZMtooYFDB2Xz E4dCICHC49LtHHO3YSjHdysdS2Be7pFGfQmk7oRDSsCvRhi+FmXjaGuqqnmj4l9qnA oJGQwFxa1+t4YNhH32VwVJ5Up2bze0DVRJlj3Mzlkx0APAfEmKGIrgE7FK1t8+1/b1 2jr6y99+FihHpSRADMIMclHjNtxGgC7uIMKazv+7kYx8+1rw2YifWqoJ1xt8B6qsOD 3lX6j7ZfmzZge91tuH49jgbNchPgECvgxGsF9jkwFBbv9sPx7r4GYktXC7rXmVxd95 AyqE+vkju7dNVJa4zJtvxndk= From: Lukas Fleischer To: aur-dev@archlinux.org Subject: [PATCH 1/2] Upgrade Sharness to 1.1.0 Date: Sat, 23 Nov 2019 12:31:19 -0500 Message-Id: <20191123173120.11185-1-lfleischer@archlinux.org> X-Mailer: git-send-email 2.24.0 MIME-Version: 1.0 X-BeenThere: aur-dev@archlinux.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Arch User Repository \(AUR\) Development" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: aur-dev-bounces@archlinux.org Sender: "aur-dev" Signed-off-by: Lukas Fleischer --- test/sharness.sh | 193 +++++++++++++++++++++++++++++++++++++---------- 1 file changed, 153 insertions(+), 40 deletions(-) diff --git a/test/sharness.sh b/test/sharness.sh index 1d57ce9..ace5a5b 100644 --- a/test/sharness.sh +++ b/test/sharness.sh @@ -1,4 +1,4 @@ -#!/bin/sh +# Sharness test framework. # # Copyright (c) 2011-2012 Mathias Lafeldt # Copyright (c) 2005-2012 Git project @@ -22,10 +22,22 @@ SHARNESS_VERSION="1.0.0" export SHARNESS_VERSION # Public: The file extension for tests. By default, it is set to "t". -: ${SHARNESS_TEST_EXTENSION:=t} +: "${SHARNESS_TEST_EXTENSION:=t}" export SHARNESS_TEST_EXTENSION -# Reset TERM to original terminal if found, otherwise save orignal TERM +# Public: Root directory containing tests. Tests can override this variable, +# e.g. for testing Sharness itself. +if test -z "$SHARNESS_TEST_DIRECTORY" +then + SHARNESS_TEST_DIRECTORY=$(pwd) +else + # ensure that SHARNESS_TEST_DIRECTORY is an absolute path so that it + # is valid even if the current working directory is changed + SHARNESS_TEST_DIRECTORY=$(cd "$SHARNESS_TEST_DIRECTORY" && pwd) || exit 1 +fi +export SHARNESS_TEST_DIRECTORY + +# Reset TERM to original terminal if found, otherwise save original TERM [ "x" = "x$SHARNESS_ORIG_TERM" ] && SHARNESS_ORIG_TERM="$TERM" || TERM="$SHARNESS_ORIG_TERM" @@ -33,23 +45,45 @@ export SHARNESS_TEST_EXTENSION export SHARNESS_ORIG_TERM # Export SHELL_PATH -: ${SHELL_PATH:=$SHELL} +: "${SHELL_PATH:=$SHELL}" export SHELL_PATH +# if --tee was passed, write the output not only to the terminal, but +# additionally to the file test-results/$BASENAME.out, too. +case "$SHARNESS_TEST_TEE_STARTED, $* " in +done,*) + # do not redirect again + ;; +*' --tee '*|*' --verbose-log '*) + mkdir -p "$SHARNESS_TEST_DIRECTORY/test-results" + BASE="$SHARNESS_TEST_DIRECTORY/test-results/$(basename "$0" ".$SHARNESS_TEST_EXTENSION")" + + # Make this filename available to the sub-process in case it is using + # --verbose-log. + SHARNESS_TEST_TEE_OUTPUT_FILE="$BASE.out" + export SHARNESS_TEST_TEE_OUTPUT_FILE + + # Truncate before calling "tee -a" to get rid of the results + # from any previous runs. + : >"$SHARNESS_TEST_TEE_OUTPUT_FILE" + + (SHARNESS_TEST_TEE_STARTED="done" ${SHELL_PATH} "$0" "$@" 2>&1; + echo $? >"$BASE.exit") | tee -a "$SHARNESS_TEST_TEE_OUTPUT_FILE" + test "$(cat "$BASE.exit")" = 0 + exit + ;; +esac + # For repeatability, reset the environment to a known state. # TERM is sanitized below, after saving color control sequences. LANG=C LC_ALL=C -PAGER=cat +PAGER="cat" TZ=UTC EDITOR=: export LANG LC_ALL PAGER TZ EDITOR unset VISUAL CDPATH GREP_OPTIONS -# Line feed -LF=' -' - [ "x$TERM" != "xdumb" ] && ( [ -t 1 ] && tput bold >/dev/null 2>&1 && @@ -82,9 +116,14 @@ while test "$#" -ne 0; do chain_lint=; shift ;; --no-color) color=; shift ;; + --tee) + shift ;; # was handled already --root=*) root=$(expr "z$1" : 'z[^=]*=\(.*\)') shift ;; + --verbose-log) + verbose_log=t + shift ;; *) echo "error: unknown test option '$1'" >&2; exit 1 ;; esac @@ -109,18 +148,25 @@ if test -n "$color"; then say_color_pass=$(tput setaf 2) # green say_color_info=$(tput setaf 6) # cyan say_color_reset=$(tput sgr0) - say_color_="" # no formatting for normal text + say_color_raw="" # no formatting for normal text say_color() { test -z "$1" && test -n "$quiet" && return - eval "say_color_color=\$say_color_$1" + case "$1" in + error) say_color_color=$say_color_error ;; + skip) say_color_color=$say_color_skip ;; + warn) say_color_color=$say_color_warn ;; + pass) say_color_color=$say_color_pass ;; + info) say_color_color=$say_color_info ;; + *) say_color_color=$say_color_raw ;; + esac shift - printf "%s\\n" "$say_color_color$*$say_color_reset" + printf '%s%s%s\n' "$say_color_color" "$*" "$say_color_reset" } else say_color() { test -z "$1" && test -n "$quiet" && return shift - printf "%s\n" "$*" + printf '%s\n' "$*" } fi @@ -137,7 +183,7 @@ say() { say_color info "$*" } -test -n "$test_description" || error "Test script did not set test_description." +test -n "${test_description:-}" || error "Test script did not set test_description." if test "$help" = "t"; then echo "$test_description" @@ -146,7 +192,11 @@ fi exec 5>&1 exec 6<&0 -if test "$verbose" = "t"; then +if test "$verbose_log" = "t" +then + exec 3>>"$SHARNESS_TEST_TEE_OUTPUT_FILE" 4>&3 +elif test "$verbose" = "t" +then exec 4>&2 3>&1 else exec 4>/dev/null 3>/dev/null @@ -177,7 +227,7 @@ trap 'die' EXIT # implicitly by specifying the prerequisite name in calls to test_expect_success # or test_expect_failure. # -# $1 - Name of prerequiste (a simple word, in all capital letters by convention) +# $1 - Name of prerequisite (a simple word, in all capital letters by convention) # # Examples # @@ -214,7 +264,7 @@ test_have_prereq() { # prerequisites can be concatenated with ',' save_IFS=$IFS IFS=, - set -- $* + set -- $@ IFS=$save_IFS total_prereq=0 @@ -231,7 +281,7 @@ test_have_prereq() { negative_prereq= esac - total_prereq=$(($total_prereq + 1)) + total_prereq=$((total_prereq + 1)) case "$satisfied_prereq" in *" $prerequisite "*) satisfied_this_prereq=t @@ -242,7 +292,7 @@ test_have_prereq() { case "$satisfied_this_prereq,$negative_prereq" in t,|,t) - ok_prereq=$(($ok_prereq + 1)) + ok_prereq=$((ok_prereq + 1)) ;; *) # Keep a list of missing prerequisites; restore @@ -263,12 +313,12 @@ test_have_prereq() { # the text_expect_* functions instead. test_ok_() { - test_success=$(($test_success + 1)) - say_color "" "ok $test_count - $@" + test_success=$((test_success + 1)) + say_color "" "ok $test_count - $*" } test_failure_() { - test_failure=$(($test_failure + 1)) + test_failure=$((test_failure + 1)) say_color error "not ok $test_count - $1" shift echo "$@" | sed -e 's/^/# /' @@ -276,13 +326,13 @@ test_failure_() { } test_known_broken_ok_() { - test_fixed=$(($test_fixed + 1)) - say_color error "ok $test_count - $@ # TODO known breakage vanished" + test_fixed=$((test_fixed + 1)) + say_color error "ok $test_count - $* # TODO known breakage vanished" } test_known_broken_failure_() { - test_broken=$(($test_broken + 1)) - say_color warn "not ok $test_count - $@ # TODO known breakage" + test_broken=$((test_broken + 1)) + say_color warn "not ok $test_count - $* # TODO known breakage" } # Public: Execute commands in debug mode. @@ -351,7 +401,7 @@ test_run_() { } test_skip_() { - test_count=$(($test_count + 1)) + test_count=$((test_count + 1)) to_skip= for skp in $SKIP_TESTS; do case $this_test.$test_count in @@ -370,7 +420,7 @@ test_skip_() { of_prereq=" of $test_prereq" fi - say_color skip >&3 "skipping test: $@" + say_color skip >&3 "skipping test: $*" say_color skip "ok $test_count # skip $1 (missing $missing_prereq${of_prereq})" : true ;; @@ -468,6 +518,44 @@ test_expect_failure() { echo >&3 "" } +# Public: Run test commands and expect anything from them. Used when a +# test is not stable or not finished for some reason. +# +# When the test passed, an "ok" message is printed, but the number of +# fixed tests is not incremented. +# +# When it failed, a "not ok ... # TODO known breakage" message is +# printed, and the number of tests still broken is incremented. +# +# Failures from these tests won't cause --immediate to stop. +# +# Usually takes two arguments: +# $1 - Test description +# $2 - Commands to be executed. +# +# With three arguments, the first will be taken to be a prerequisite: +# $1 - Comma-separated list of test prerequisites. The test will be skipped if +# not all of the given prerequisites are set. To negate a prerequisite, +# put a "!" in front of it. +# $2 - Test description +# $3 - Commands to be executed. +# +# Returns nothing. +test_expect_unstable() { + test "$#" = 3 && { test_prereq=$1; shift; } || test_prereq= + test "$#" = 2 || error "bug in the test script: not 2 or 3 parameters to test_expect_unstable" + export test_prereq + if ! test_skip_ "$@"; then + say >&3 "checking unstable test: $2" + if test_run_ "$2" unstable; then + test_ok_ "$1" + else + test_known_broken_failure_ "$1" + fi + fi + echo >&3 "" +} + # Public: Run command and ensure that it fails in a controlled way. # # Use it instead of "! ". For example, when dies due to a @@ -560,7 +648,7 @@ test_expect_code() { shift "$@" exit_code=$? - if test $exit_code = $want_code; then + if test "$exit_code" = "$want_code"; then return 0 fi @@ -570,7 +658,7 @@ test_expect_code() { # Public: Compare two files to see if expected output matches actual output. # -# The TEST_CMP variable defines the command used for the comparision; it +# The TEST_CMP variable defines the command used for the comparison; it # defaults to "diff -u". Only when the test script was started with --verbose, # will the command's output, the diff, be printed to the standard output. # @@ -617,14 +705,14 @@ test_seq() { while test "$i" -le "$j" do echo "$i" || return - i=$(expr "$i" + 1) + i=$(("$i" + 1)) done } # Public: Check if the file expected to be empty is indeed empty, and barfs # otherwise. # -# $1 - File to check for emptyness. +# $1 - File to check for emptiness. # # Returns 0 if file is empty, 1 otherwise. test_must_be_empty() { @@ -636,6 +724,36 @@ test_must_be_empty() { fi } +# debugging-friendly alternatives to "test [-f|-d|-e]" +# The commands test the existence or non-existence of $1. $2 can be +# given to provide a more precise diagnosis. +test_path_is_file () { + if ! test -f "$1" + then + echo "File $1 doesn't exist. $2" + false + fi +} + +test_path_is_dir () { + if ! test -d "$1" + then + echo "Directory $1 doesn't exist. $2" + false + fi +} + +# Check if the directory exists and is empty as expected, barf otherwise. +test_dir_is_empty () { + test_path_is_dir "$1" && + if test -n "$(find "$1" -mindepth 1 -maxdepth 1)" + then + echo "Directory '$1' is not empty, it contains:" + ls -la "$1" + return 1 + fi +} + # Public: Schedule cleanup commands to be run unconditionally at the end of a # test. # @@ -723,7 +841,7 @@ test_done() { say_color warn "# still have $test_broken known breakage(s)" fi if test "$test_broken" != 0 || test "$test_fixed" != 0; then - test_remaining=$(( $test_count - $test_broken - $test_fixed )) + test_remaining=$((test_count - test_broken - test_fixed)) msg="remaining $test_remaining test(s)" else test_remaining=$test_count @@ -760,20 +878,15 @@ test_done() { esac } -# Public: Root directory containing tests. Tests can override this variable, -# e.g. for testing Sharness itself. -: ${SHARNESS_TEST_DIRECTORY:=$(pwd)} -export SHARNESS_TEST_DIRECTORY - # Public: Source directory of test code and sharness library. # This directory may be different from the directory in which tests are # being run. -: ${SHARNESS_TEST_SRCDIR:=$(cd $(dirname $0) && pwd)} +: "${SHARNESS_TEST_SRCDIR:=$(cd "$(dirname "$0")" && pwd)}" export SHARNESS_TEST_SRCDIR # Public: Build directory that will be added to PATH. By default, it is set to # the parent directory of SHARNESS_TEST_DIRECTORY. -: ${SHARNESS_BUILD_DIRECTORY:="$SHARNESS_TEST_DIRECTORY/.."} +: "${SHARNESS_BUILD_DIRECTORY:="$SHARNESS_TEST_DIRECTORY/.."}" PATH="$SHARNESS_BUILD_DIRECTORY:$PATH" export PATH SHARNESS_BUILD_DIRECTORY From patchwork Sat Nov 23 17:31:20 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lukas Fleischer X-Patchwork-Id: 1387 Return-Path: Delivered-To: patchwork@archlinux.org Received: from apollo.archlinux.org (localhost [127.0.0.1]) by apollo.archlinux.org (Postfix) with ESMTP id 0F2A9153BF266 for ; Sat, 23 Nov 2019 17:30:25 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on apollo.archlinux.org X-Spam-Level: X-Spam-Status: No, score=-3.4 required=5.0 tests=DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1, MAILING_LIST_MULTI=-1,RCVD_IN_DNSWL_MED=-2.3,SPF_HELO_NONE=0.001, T_DMARC_POLICY_NONE=0.01 autolearn=unavailable autolearn_force=no version=3.4.2 X-Spam-BL-Results: [127.0.0.4] [127.0.9.2] Received: from orion.archlinux.org (orion.archlinux.org [IPv6:2a01:4f8:160:6087::1]) by apollo.archlinux.org (Postfix) with ESMTPS for ; Sat, 23 Nov 2019 17:30:24 +0000 (UTC) Received: from orion.archlinux.org (localhost [127.0.0.1]) by orion.archlinux.org (Postfix) with ESMTP id 8463A16D189010; Sat, 23 Nov 2019 17:30:20 +0000 (UTC) Received: from luna.archlinux.org (luna.archlinux.org [IPv6:2a01:4f8:160:3033::2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: luna) by orion.archlinux.org (Postfix) with ESMTPSA id AD33816D188A47; Sat, 23 Nov 2019 17:30:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=orion; t=1574530213; bh=GFmgCoEx+ZWxDK+j6/f0CYh2/yiwRVZJG2BkpYt4BIU=; h=From:To:Subject:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe; b=KM4h89/WIlL8JTDXKEtr6nC3KHFLnfd3l9RSmgnM7dI4GnQOyzoALn2Q0EApIqG+j M0gUdkAbP5XN9s3XGagOCcui44VGDqj+iIm6/JVH6ogLsonM7L9NOze0zxSeu8K31x YXCeftZJaQcIxMdbcmG2/or3RJmZhO3aklUl3RJikX1sZDaXL22/2EBb0omsS+2GA6 WNX0myIa7pe/N2LP3grTVDLhuoXf+1uMuha9akD1mi8lopjwVwe10T8fg6IExQX73W xxnuNNhlZIf1JLBUvKJH+vbJzvtAa9ADyljS9yZHJ6P8l9PMUDWcSq73um6N7kMnFi 39YFf7xnKhv8+UjhbOEw4j0QrQO6bDtFVeuqSy3Pz7s6rOd8QJULZBJLI6z30SQYHH QYfaRwFeBquchkrPb3okvVIVFPodCFxl3vHouId9Z/DFrV4nWZCTx7udiW7lwq2RkN +xNYXDrlWwZpRBmRGa86bGeQYBi6XGK7qzTn9KTSpX0cx5bEdglFuRPq/VqGqrWxZ3 sBttAcQ1z0ukNyFXn/AiDiDY4jEOGNsQd5Kxjx0KWFBSnJwNuOkLhrwu4tTYBK5oZC 7pQmc5aTeewcKAn8xH0Zob5hdYJA3Bw8u9sARAzJcUGtpLqq35hqd7xXOXVyqduYx2 QmQNC1RzUEWOoTCXX3GgA+uY= Received: from luna.archlinux.org (luna.archlinux.org [127.0.0.1]) by luna.archlinux.org (Postfix) with ESMTP id 27CBC2B790; Sat, 23 Nov 2019 17:30:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=luna2; t=1574530212; bh=GFmgCoEx+ZWxDK+j6/f0CYh2/yiwRVZJG2BkpYt4BIU=; h=From:To:Subject:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe; b=m+YHwFBfh5VtR5irPST/k4WzlkVS/vss8K3mPebb/a4qo1pMuw9VjIzrLainb0+gj 6+n2wfJezGM5sgDj34Xbkh6gnceGG/SRHBiYJIQsQQH9iZjvt83VmE8c3+gg+RpGkS 1mslkTY2Vu6NDyM7fg0fKO+jhIXeYVy9cmcLtePEAXJvKF8xpsWQyrl+K9DDK9jckv OK1/kELN57Kj0PszbDFjHxk5qewRM4LmWMiCI22ZF6ZqE9fOkQ44JbMj9TNLkCrY/R XxyxL7ZT2CnTOhmUcUlRbIk0bhySXRfWyAa61JuIYNVOjoYPU28JMjbbWq8L1RNGa1 vb/UZa2MF3H933Wd1WTmFaDyWPii3Gj6VI6os7dRuPTL05tsj3hMbj2eP7tKmJsEN2 09R9s6lwM+hCsdAIBXGLmSFpk+2Qj3hg+C4Y4O2B2t3eRgcyqZ4er3RDbfDFZ6KAmC bR98wIEAMDSrNdEtVKSAVdSDOZBaum2NxDpl93MrapJYTDNxS54/mMhVCF3swVKHrm 0Mu227ih2iHWQEweshourCBGKZRSQj3AirIBJ+DdLpwVnHzjqI8ABAODiYMTWg8llc 3IH9Tbv/sMJtTxzRSEduDpZ6IZvM30JlshJJYZInkgjRPYdESaGGWU7UYupc1Iead8 ljM+Kc32HcqBuh91WWjFyPfg= Received: from luna.archlinux.org (luna.archlinux.org [127.0.0.1]) by luna.archlinux.org (Postfix) with ESMTP id DD7C52B398 for ; Sat, 23 Nov 2019 17:30:08 +0000 (UTC) Received: from orion.archlinux.org (orion.archlinux.org [IPv6:2a01:4f8:160:6087::1]) by luna.archlinux.org (Postfix) with ESMTPS for ; Sat, 23 Nov 2019 17:30:08 +0000 (UTC) Received: from orion.archlinux.org (localhost [127.0.0.1]) by orion.archlinux.org (Postfix) with ESMTP id 839AA16D1889D1 for ; Sat, 23 Nov 2019 17:30:04 +0000 (UTC) Received: from localhost (unknown [72.138.14.22]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: lfleischer) by orion.archlinux.org (Postfix) with ESMTPSA id E177B16D1889D0 for ; Sat, 23 Nov 2019 17:30:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=archlinux.org; s=orion; t=1574530204; bh=GFmgCoEx+ZWxDK+j6/f0CYh2/yiwRVZJG2BkpYt4BIU=; h=From:To:Subject:Date:In-Reply-To:References; b=F3cGgyCdiXno8rK4N+XPLPuGkZSiftZ8EWWB7TVnlverzKo4ECauSphjXA/XrG+NF bJkuAEM8iBlb+EiZKn+6R9WOJL9/6nKOGT3tF8cHyUgnWMReRoF+p651M5bwB9Gpei qmwq8qPCFiJVvtk5tg+cm6v35eEjsayZrz2cqD46kuurn3boRa3br3LsvIlD26fnPE ZWIG+7fhhh7rOPZWru9wNu0H0qL/UunyXQ2azzHM5YwmKCiCZhfp6Km1J6DY1lRZH/ XMbWJGn9jbKsCnv4lbFhjpUKlEFnJfOF62AaNf3j1l1ygV4Aebo3wOR9je6S/wilIY 1oIlfm0o3G5xzms8c2hZgg8OwGd1V+dO/OosguJxs8VQGJ0sX4h51WD9sOp2Khz8EJ BIMunyVBWQrURexzm4korWg0CnvLwe17SkieqeH7s9Myo1bNA4xWDCzEhvsyekoFTy Rk46GRHd4nfaBoMCHed9xXZj/eITwXvLinLjMGe5F0Blb2xw5RUS9MT2czhVqF84n+ +Q/rUMs2dJqOpEUyL7DveOm8LV9zSews/4MoyfGZEWWI1Wz6EM47wolG3UQzKOyS/L XYPepaZGl1Ecf2dSmvBDwbsZRmmk/ifbSP0F5d3TFrjf7wm5ipqRJhCmLgZaxkQw7W RAYcqxiykAkbrNCDFOrfo6NQ= From: Lukas Fleischer To: aur-dev@archlinux.org Subject: [PATCH 2/2] Don't require all Python database modules to be installed Date: Sat, 23 Nov 2019 12:31:20 -0500 Message-Id: <20191123173120.11185-2-lfleischer@archlinux.org> X-Mailer: git-send-email 2.24.0 In-Reply-To: <20191123173120.11185-1-lfleischer@archlinux.org> References: <20191123173120.11185-1-lfleischer@archlinux.org> MIME-Version: 1.0 X-BeenThere: aur-dev@archlinux.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Arch User Repository \(AUR\) Development" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: aur-dev-bounces@archlinux.org Sender: "aur-dev" We support multiple database backends. Don't require Python modules for all backends to be installed. Signed-off-by: Lukas Fleischer --- aurweb/db.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/aurweb/db.py b/aurweb/db.py index 0b58197..c6d4de1 100644 --- a/aurweb/db.py +++ b/aurweb/db.py @@ -1,5 +1,12 @@ -import mysql.connector -import sqlite3 +try: + import mysql.connector +except ImportError: + pass + +try: + import sqlite3 +except ImportError: + pass import aurweb.config