[tor-commits] [tor/master] Make test_glob() pass even when run as root.
commit 7b4716a7624b73e60f70485ac577ef6cf884e67c Author: Nick Mathewson Date: Wed Aug 12 20:36:23 2020 -0400 Make test_glob() pass even when run as root. Previously the test relied on not being able to look inside 000 directories, which is a thing root _can_ do. Bug not in any released Tor version. --- src/test/test_util.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/test/test_util.c b/src/test/test_util.c index 230f2c7034..0e2550d5c5 100644 --- a/src/test/test_util.c +++ b/src/test/test_util.c @@ -4622,11 +4622,14 @@ test_util_glob(void *ptr) TEST("\\*"); EXPECT_EMPTY(); - // test forbidden directory - tor_asprintf(, "%s"PATH_SEPARATOR"*"PATH_SEPARATOR"*", dirname); - results = tor_glob(pattern); - tor_free(pattern); - tt_assert(!results); + if (getuid() != 0) { +// test forbidden directory, if we're not root. +// (Root will be able to see this directory anyway.) +tor_asprintf(, "%s"PATH_SEPARATOR"*"PATH_SEPARATOR"*", dirname); +results = tor_glob(pattern); +tor_free(pattern); +tt_assert(!results); + } #endif #undef TEST ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 34919368cebb1e635d10694d8ec42911db909130 Merge: 309eafbc3c 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] Merge branch 'maint-0.4.2' into release-0.4.2
commit feecf46e6deca4493c46b53e84872a76fff67b80 Merge: b6fb925e99 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into release-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 34919368cebb1e635d10694d8ec42911db909130 Merge: 309eafbc3c 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.4' into release-0.4.4
commit 8bd79bd75e760e7493c2fc7dcf5b38d453ea9f13 Merge: a2e64b3e85 9d77ed7be2 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.4' into release-0.4.4 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.3' into maint-0.4.4
commit 9d77ed7be2da18bef3ab1258f277c527347a94a7 Merge: f98b1988ec 34919368ce Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.4.3' into release-0.4.3
commit 146e4cfa95c3d0cd26f0804e186b2d445a3f771c Merge: f94ad3d32a 34919368ce Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.3' into release-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] Merge branch 'maint-0.3.5' into release-0.3.5
commit d605622aded18c2d0c78f81c3c75735d4cf982ff Merge: ac5a06d42c aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into release-0.3.5 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 34919368cebb1e635d10694d8ec42911db909130 Merge: 309eafbc3c 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.3.5] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.4.4'
commit effc9739bfb3c91171be47673c67ec2bd05c2758 Merge: 9ef991dbe3 9d77ed7be2 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.4' .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.4.3' into maint-0.4.4
commit 9d77ed7be2da18bef3ab1258f277c527347a94a7 Merge: f98b1988ec 34919368ce Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.4.3' into maint-0.4.4
commit 9d77ed7be2da18bef3ab1258f277c527347a94a7 Merge: f98b1988ec 34919368ce Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 07f0210a0525423f8cbbb41d00df0aa2d40fedf5 Merge: 02bb719a94 aeafb7f44f Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 34919368cebb1e635d10694d8ec42911db909130 Merge: 309eafbc3c 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 34919368cebb1e635d10694d8ec42911db909130 Merge: 309eafbc3c 07f0210a05 Author: Nick Mathewson Date: Wed Aug 12 20:21:04 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 52 insertions(+), 16 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.3.5] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] CI: Turn on stem with 044 and later.
commit 41a7ab96e578e58f277b34f5735d35a2a168da09 Author: Nick Mathewson Date: Wed Aug 12 20:13:18 2020 -0400 CI: Turn on stem with 044 and later. --- .gitlab-ci.yml | 7 +++ scripts/ci/ci-driver.sh | 34 +++--- 2 files changed, 26 insertions(+), 15 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8ed615687b..c77e78764d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -37,6 +37,7 @@ variables: paths: - artifacts/ +# This template .apt-template: | export LC_ALL=C.UTF-8 echo Etc/UTC > /etc/timezone @@ -90,9 +91,7 @@ debian-minimal: - ./scripts/ci/ci-driver.sh -# TODO: This one just takes too long to finish right now! -# Maybe we need to divide the call to ./src/test/test into a few segments, -# that all end in similar amount of time? +# TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing <<: *debian-template @@ -128,6 +127,6 @@ debian-integration: variables: CHECK: "no" CHUTNEY: "yes" -# STEM: "yes" -- currently failing on <044. +STEM: "yes" script: - ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 6faffd7b40..527bba82e5 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -267,18 +267,26 @@ TOR_VERSION=$(grep -m 1 AC_INIT configure.ac | sed -e 's/.*\[//; s/\].*//;') # Use variables like these when we need to behave differently depending on # Tor version. Only create the variables we need. TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no # These are the currently supported Tor versions; no need to work with anything # ancient in this script. case "$TOR_VERSION" in 0.3.*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no ;; 0.4.[012].*) TOR_VER_AT_LEAST_043=no +TOR_VER_AT_LEAST_044=no +;; +0.4.3.*) +TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=no ;; *) TOR_VER_AT_LEAST_043=yes +TOR_VER_AT_LEAST_044=yes ;; esac @@ -427,21 +435,25 @@ fi if [[ "${STEM}" = "yes" ]]; then start_section "Stem" - # This shold probably be part some test-stem make target. - if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ - python3 "${STEM_PATH}/run_tests.py" \ - --tor src/app/tor \ - --integ --test control.controller \ - --test control.base_controller \ - --test process \ - --log TRACE \ - --log-file stem.log ; then - hooray "Stem tests have succeeded" - else + if [[ "${TOR_VER_AT_LEAST_044}" = 'yes' ]]; then + # This shold probably be part some test-stem make target. + if runcmd timelimit -p -t 520 -s USR1 -T 30 -S ABRT \ + python3 "${STEM_PATH}/run_tests.py" \ + --tor src/app/tor \ + --integ --test control.controller \ + --test control.base_controller \ + --test process \ + --log TRACE \ + --log-file stem.log ; then + hooray "Stem tests have succeeded" + else error "Stem output:" runcmd tail -1000 "${STEM_PATH}"/test/data/tor_log runcmd grep -v "SocketClosed" stem.log | tail -1000 FAILED_TESTS="${FAILED_TESTS} stem" + fi + else + skipping "Stem: broken with <= 0.4.3. See bug tor#40077" fi end_section "Stem" fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Improve comments in .gitlab-ci.yml
commit aeafb7f44f07136101c06ea2986b295ef9915c9c Author: Nick Mathewson Date: Wed Aug 12 20:20:58 2020 -0400 Improve comments in .gitlab-ci.yml --- .gitlab-ci.yml | 27 ++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c77e78764d..176c71f8f0 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -58,12 +58,18 @@ variables: <<: *artifacts-template variables: DEBIAN_FRONTEND: "noninteractive" + # TODO: Using "cache" in this way speeds up our downloads. It would be + # even better, though, to start with a pre-upgraded debian image. + # + # TODO: Will we have to do this differently once we have more than one + # debian version that we're using? cache: key: apt paths: - apt-cache before_script: - *apt-template +# Install patches unconditionally. - apt-get install automake build-essential @@ -77,20 +83,28 @@ variables: pkg-config python3 zlib1g-dev +# Install patches that we only need for some use cases. - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi - if [ "$STEM" = yes ]; then apt-get install timelimit; fi - if [ "$CC" = clang ]; then apt-get install clang; fi +# TODO: This next line should not be debian-only. - if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +# TODO: This next line should not be debian-only. - if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi +# Minmal check on debian: just make, make check. +# debian-minimal: image: debian:stable <<: *debian-template script: - ./scripts/ci/ci-driver.sh - +# +# Run "make check" with a hardened clang on debian stable. This takes +# care of a hardening check, and a compile-with-clang check. +# # TODO: This will be faster once we merge #40098 and #40099. debian-hardened: image: debian:testing @@ -101,6 +115,8 @@ debian-hardened: script: - ./scripts/ci/ci-driver.sh +# +# Distcheck on debian stable debian-distcheck: image: debian:stable <<: *debian-template @@ -110,6 +126,8 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh +# +# Documentation tests on debian stable: doxygen and asciidoc. debian-docs: image: debian:stable <<: *debian-template @@ -121,6 +139,13 @@ debian-docs: script: - ./scripts/ci/ci-driver.sh +# +# Integration tests on debian stable: chutney and stem. +# +# TODO: It would be cool if this target didn't have to re-build tor, and +# could instead re-use Tor from debian-minimal. That can be done +# with the 'artifacts' mechanism, in theory, but it would be good to +# avoid having to have a system with hundreds of artifacts. debian-integration: image: debian:stable <<: *debian-template ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Changelog draft for 0.4.4.4-rc
commit 9b3e721d74fa6c40d1dabb678054de277d0e3f9a Author: Nick Mathewson Date: Wed Aug 12 14:52:23 2020 -0400 Changelog draft for 0.4.4.4-rc --- ChangeLog | 56 + changes/bug31036| 3 --- changes/bug34086| 3 --- changes/bug40072| 4 changes/bug40076| 5 - changes/bug40083| 5 - changes/bug40095| 4 changes/ticket33747 | 7 --- changes/ticket40081 | 6 -- changes/ticket6198 | 3 --- 10 files changed, 56 insertions(+), 40 deletions(-) diff --git a/ChangeLog b/ChangeLog index 65891fefa9..7ef8c543c3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,59 @@ +Changes in version 0.4.4.4-rc - 2020-08-12 + Tor 0.4.4.4-rc is the first release candidate in its series. It fixes + several bugs in previous versions, including some that caused annoying + behavior for relay and bridge operators. + + o Minor features (security): +- Channels using obsolete versions of the Tor link protocol are no + longer allowed to circumvent address-canonicity checks. (This is + only a minor issue, since such channels have no way to set ed25519 + keys, and therefore should always be rejected for circuits that + specify ed25519 identities.) Closes ticket 40081. + + o Minor features (defense in depth): +- Wipe more data from connection address fields before returning + them to the memory heap. Closes ticket 6198. + + o Minor bugfixes (correctness, buffers): +- Fix a correctness bug that could cause an assertion failure if we + ever tried using the buf_move_all() function with an empty input + buffer. As far as we know, no released versions of Tor do this. + Fixes bug 40076; bugfix on 0.3.3.1-alpha. + + o Minor bugfixes (linux seccomp2 sandbox): +- Fix startup crash with seccomp sandbox enabled when tor tries to + open the data directory. Patch from Daniel Pinto. Fixes bug 40072; + bugfix on 0.4.4.3-alpha-dev. + + o Minor bugfixes (onion service v3): +- Remove a BUG() warning that could trigger in certain unlikely + edge-cases. Fixes bug 34086; bugfix on 0.3.2.1-alpha. + + o Minor bugfixes (rate limiting, bridges, pluggable transports): +- On a bridge, treat all connections from an ExtORPort as remote by + default for the purposes of rate-limiting. Previously, bridges + would treat the connection as local unless they explicitly + received a "USERADDR" command. ExtORPort connections still count + as local if there is a USERADDR command with an explicit local + address. Fixes bug 33747; bugfix on 0.2.5.1-alpha. + + o Minor bugfixes (relay, self-testing): +- When starting up as a relay, if we haven't been able to verify + that we're reachable, only launch reachability tests at most once + a minute. Previously, we had been launching tests up to once a + second, which was needlessly noisy. Fixes bug 40083; bugfix + on 0.2.8.1-alpha. + + o Minor bugfixes (testing): +- When running the subsystem order check, use the python binary + configured with the PYTHON environment variable. Fixes bug 40095; + bugfix on 0.4.4.1-alpha. + + o Minor bugfixes (windows): +- Fix a bug that prevented Tor from starting if its log file grew + above 2GB. Fixes bug 31036; bugfix on 0.2.1.8-alpha. + + Changes in version 0.4.4.3-alpha - 2020-07-27 Tor 0.4.4.3-alpha fixes several annoyances in previous versions, including one affecting NSS users, and several affecting the Linux diff --git a/changes/bug31036 b/changes/bug31036 deleted file mode 100644 index d9921dba43..00 --- a/changes/bug31036 +++ /dev/null @@ -1,3 +0,0 @@ - o Minor bugfixes (windows): -- Fix a bug that prevented Tor from starting if its log file - grew above 2GB. Fixes bug 31036; bugfix on 0.2.1.8-alpha. diff --git a/changes/bug34086 b/changes/bug34086 deleted file mode 100644 index 245992f8f4..00 --- a/changes/bug34086 +++ /dev/null @@ -1,3 +0,0 @@ - o Minor bugfixes (onion service v3): -- Remove a BUG() warning that could trigger in certain unlikely edge-cases. - Fixes bug 34086; bugfix on 0.3.2.1-alpha. diff --git a/changes/bug40072 b/changes/bug40072 deleted file mode 100644 index 2b82f3f18b..00 --- a/changes/bug40072 +++ /dev/null @@ -1,4 +0,0 @@ - o Minor bugfixes (linux seccomp2 sandbox): -- Fix startup crash with seccomp sandbox enabled when tor tries to - open the data directory. Patch from Daniel Pinto. Fixes bug 40072; - bugfix on 0.4.4.3-alpha-dev. diff --git a/changes/bug40076 b/changes/bug40076 deleted file mode 100644 index 9ef5969ae8..00 --- a/changes/bug40076 +++ /dev/null @@ -1,5 +0,0 @@ - o Minor bugfixes (correctness, buffers): -- Fix a correctness bug that could cause an assertion failure if we ever - tried using the buf_move_all() function with an empty input. - As far as we know, no released versions of Tor
[tor-commits] [tor/release-0.4.4] small edits to 0444-rc changelog
commit a2e64b3e85da186eaaf87fff939cc1c5b36a2f5e Author: Nick Mathewson Date: Wed Aug 12 19:53:27 2020 -0400 small edits to 0444-rc changelog --- ChangeLog | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 7ef8c543c3..4e584e8fad 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,4 @@ -Changes in version 0.4.4.4-rc - 2020-08-12 +Changes in version 0.4.4.4-rc - 2020-08-13 Tor 0.4.4.4-rc is the first release candidate in its series. It fixes several bugs in previous versions, including some that caused annoying behavior for relay and bridge operators. @@ -45,7 +45,7 @@ Changes in version 0.4.4.4-rc - 2020-08-12 on 0.2.8.1-alpha. o Minor bugfixes (testing): -- When running the subsystem order check, use the python binary +- When running the subsystem order check, use the Python binary configured with the PYTHON environment variable. Fixes bug 40095; bugfix on 0.4.4.1-alpha. ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [stem/master] Present a safer pkill command
commit daed7bca9f930e03a89ce8a4254e02a5310a0fc1 Author: Damian Johnson Date: Wed Aug 12 15:41:46 2020 -0700 Present a safer pkill command pkill's '-x' flag picks a process via an exact match rather than a regex. Caught thanks to uokf... 17:16 < uokf> I'd like to make a suggestion to change https://stem.torproject.org/faq.html#how-do-i-reload-my-torrc 17:17 < uokf> it says pkill -sighup tor 17:17 < uokf> but it should say pkill -x -sighup tor 17:17 < uokf> pkill sends the signal to all matching pids 17:18 <@arma> so if you have a process called 'extractor' then it'll hit that one too? 17:18 <@arma> that seems like a good bug. can you open a ticket for stem? i think atagar likes his tickets in github but i'm not sure. --- docs/faq.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/faq.rst b/docs/faq.rst index a7333fc7..50c9786a 100644 --- a/docs/faq.rst +++ b/docs/faq.rst @@ -444,7 +444,7 @@ Tor is configured through its `torrc you need to either restart Tor or issue a **HUP** for the changes to be reflected. To issue a HUP you can either... - * Run **pkill -sighup tor**. + * Run **pkill -x -sighup tor**. * Send Tor a **HUP** signal through its control port... :: ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [translation/support-portal] https://gitweb.torproject.org/translation.git/commit/?h=support-portal
commit cc219f7e1201432e9c140402d2e081f8efe12e3e Author: Translation commit bot Date: Wed Aug 12 21:48:09 2020 + https://gitweb.torproject.org/translation.git/commit/?h=support-portal --- contents+ar.po | 6 ++ 1 file changed, 6 insertions(+) diff --git a/contents+ar.po b/contents+ar.po index 3d6ed391d7..331f1521d6 100644 --- a/contents+ar.po +++ b/contents+ar.po @@ -8901,6 +8901,12 @@ msgid "" "[Facebook](https://www.facebook.com/notes/protect-the-graph/making-; "connections-to-facebook-more-secure/1526085754298237/)." msgstr "" +"Ùت٠اÙاعت٠اد Ø£ÙضÙا عÙ٠خد٠ات Onion ÙÙدردشة اÙخاÙÙØ© ٠٠اÙبÙاÙات اÙÙصÙÙØ© " +"Ù٠شارÙØ© اÙÙ ÙÙات Ø ÙاÙتÙاع٠اÙآ٠٠بÙ٠اÙصØÙÙÙÙ Ù٠صادرÙÙ Ù Ø«Ù [SecureDrop] " +"(https://securedrop.org/) Ø£Ù [OnionShare] (https://onionshare.org/ ) " +"ÙتØدÙثات اÙبرا٠ج اÙØ£Ùثر أ٠اÙÙا Ùطر٠أÙثر أ٠اÙÙا ÙÙÙصÙ٠إÙÙ Ù ÙاÙع اÙÙÙب " +"اÙØ´ÙÙرة Ù Ø«Ù [Facebook] (https://www.facebook.com/notes/protect-the-graph; +"/making-connections-to-facebook-more-secure/1526085754298237 /)." #: https//support.torproject.org/onionservices/onionservices-2/ #: (content/onionservices/onionservices-2/contents+en.lrquestion.description) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [translation/support-portal] https://gitweb.torproject.org/translation.git/commit/?h=support-portal
commit 4e269312f72252b3e52ba382828bdf73533885f1 Author: Translation commit bot Date: Wed Aug 12 18:48:11 2020 + https://gitweb.torproject.org/translation.git/commit/?h=support-portal --- contents+ar.po | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/contents+ar.po b/contents+ar.po index d829b7fa39..3d6ed391d7 100644 --- a/contents+ar.po +++ b/contents+ar.po @@ -8850,6 +8850,8 @@ msgid "" "Websites that are only accessible over Tor are called \"onions\" and end in " "the TLD .onion." msgstr "" +"تس٠٠٠ÙاÙع اÙÙÙب اÙت٠ÙÙ Ù٠اÙÙصÙ٠إÙÙÙا عبر Tor \"onions\" ÙتÙتÙ٠بÙTLD " +".onion." #: https//support.torproject.org/onionservices/onionservices-1/ #: (content/onionservices/onionservices-1/contents+en.lrquestion.description) @@ -8857,11 +8859,13 @@ msgid "" "For example, the DuckDuckGo onion is " "[https://3g2upl4pq6kufc4m.onion](https://3g2upl4pq6kufc4m.onion)." msgstr "" +"عÙ٠سبÙ٠اÙÙ Ø«Ø§Ù Ø onion DuckDuckGo ÙÙ [https://3g2upl4pq6kufc4m.onion] " +"(https://3g2upl4pq6kufc4m.onion)." #: https//support.torproject.org/onionservices/onionservices-1/ #: (content/onionservices/onionservices-1/contents+en.lrquestion.description) msgid "You can access these websites by using Tor Browser." -msgstr "" +msgstr "ÙÙ ÙÙ٠اÙÙصÙ٠إÙÙ Ùذ٠اÙÙ ÙاÙع باستخدا٠٠تصÙØ Tor." #: https//support.torproject.org/onionservices/onionservices-1/ #: (content/onionservices/onionservices-1/contents+en.lrquestion.description) @@ -8869,6 +8873,8 @@ msgid "" "The addresses must be shared with you by the website host, as onions are not" " indexed in search engines in the typical way that vanilla websites are." msgstr "" +"Ùجب ٠شارÙØ© اÙعÙاÙÙ٠٠ع٠٠٠Ùب٠٠ضÙ٠اÙÙ ÙÙع Ø ØÙØ« Ùا تت٠ÙÙرسة onion ÙÙ " +"Ù ØرÙات اÙبØØ« باÙطرÙÙØ© اÙ٠عتادة Ù Ø«Ù Ù ÙاÙع اÙÙاÙÙÙÙا." #: https//support.torproject.org/onionservices/onionservices-2/ #: (content/onionservices/onionservices-2/contents+en.lrquestion.title) @@ -8881,6 +8887,8 @@ msgid "" "Onion services allow people to browse but also to publish anonymously, " "including publishing anonymous websites." msgstr "" +"ØªØ³Ù Ø Ø®Ø¯Ù Ø§Øª Onion ÙÙأشخاص باÙتصÙØ ÙÙÙ٠أÙضÙا ÙÙÙشر دÙ٠اÙÙش٠ع٠ÙÙÙتÙÙ Ø Ø¨Ù Ø§ " +"ÙÙ Ø°ÙÙ Ùشر Ù ÙاÙع اÙÙÙب اÙ٠جÙÙÙØ©." #: https//support.torproject.org/onionservices/onionservices-2/ #: (content/onionservices/onionservices-2/contents+en.lrquestion.description) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Add test for torrc %include functionality and seccomp sandbox
commit 49dab42782e06e51c218cf40b8cbb98b80d9803f Author: Daniel Pinto Date: Wed Jun 3 22:09:51 2020 +0100 Add test for torrc %include functionality and seccomp sandbox Tests %include with files and folders, modifying and reloading the config file with sandbox enabled and reponse of SAVECONF and getinfo config-can-saveconf control commmands. --- .travis.yml | 4 +- src/test/include.am | 3 + src/test/test_include.py | 196 +++ src/test/test_include.sh | 111 +++ 4 files changed, 312 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 01343e65d9..aaca3a7368 100644 --- a/.travis.yml +++ b/.travis.yml @@ -237,8 +237,8 @@ install: - dd ibs=1 count=1024 if=/dev/urandom > ~/.torrc script: - # Skip test_rebind on macOS - - if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then export TOR_SKIP_TEST_REBIND=true; fi + # Skip test_rebind and test_include on macOS + - if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then export TOR_SKIP_TEST_REBIND=true; export TOR_SKIP_TEST_INCLUDE=true; fi - ./autogen.sh - CONFIGURE_FLAGS="$ASCIIDOC_OPTIONS $COVERAGE_OPTIONS $HARDENING_OPTIONS $MODULES_OPTIONS $NSS_OPTIONS $OPENSSL_OPTIONS $RUST_OPTIONS --enable-fatal-warnings --disable-silent-rules" - echo "Configure flags are $CONFIGURE_FLAGS CC=\"$CC $C_DIALECT_OPTIONS\"" diff --git a/src/test/include.am b/src/test/include.am index c049053438..478547d655 100644 --- a/src/test/include.am +++ b/src/test/include.am @@ -46,6 +46,7 @@ else # Only do this when coverage is not on, since it invokes lots of code # in a kind of unpredictable way. TESTSCRIPTS += src/test/test_rebind.sh +TESTSCRIPTS += src/test/test_include.sh endif endif @@ -431,6 +432,8 @@ EXTRA_DIST += \ src/test/slownacl_curve25519.py \ src/test/test_rebind.sh \ src/test/test_rebind.py \ + src/test/test_include.sh \ + src/test/test_include.py \ src/test/zero_length_keys.sh \ scripts/maint/run_check_subsystem_order.sh \ src/test/rust_supp.txt \ diff --git a/src/test/test_include.py b/src/test/test_include.py new file mode 100644 index 00..0ac87ee8fb --- /dev/null +++ b/src/test/test_include.py @@ -0,0 +1,196 @@ +# Future imports for Python 2.7, mandatory in 3.0 +from __future__ import division +from __future__ import print_function +from __future__ import unicode_literals + +import errno +import logging +import os +import random +import socket +import subprocess +import sys +import time +import re + +CONTROL_SOCK_TIMEOUT = 10.0 +LOG_TIMEOUT = 60.0 +LOG_WAIT = 0.1 + +def fail(msg): +logging.error('FAIL') +sys.exit(msg) + +def skip(msg): +logging.warning('SKIP: {}'.format(msg)) +sys.exit(77) + +def wait_for_log(s): +cutoff = time.time() + LOG_TIMEOUT +while time.time() < cutoff: +l = tor_process.stdout.readline() +l = l.decode('utf8', 'backslashreplace') +if s in l: +logging.info('Tor logged: "{}"'.format(l.strip())) +return +# readline() returns a blank string when there is no output +# avoid busy-waiting +if len(l) == 0: +logging.debug('Tor has not logged anything, waiting for "{}"'.format(s)) +time.sleep(LOG_WAIT) +else: +logging.info('Tor logged: "{}", waiting for "{}"'.format(l.strip(), s)) +fail('Could not find "{}" in logs after {} seconds'.format(s, LOG_TIMEOUT)) + +def pick_random_port(): +port = 0 +random.seed() + +for i in range(8): +port = random.randint(1, 6) +s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) +if s.connect_ex(('127.0.0.1', port)) == 0: +s.close() +else: +break + +if port == 0: +fail('Could not find a random free port between 1 and 6') + +return port + +def check_control_list(control_out_file, expected, value_name): +received_count = 0 +for e in expected: +received = control_out_file.readline().strip() +received_count += 1 +parts = re.split('[ =-]', received.strip()) +if len(parts) != 3 or parts[0] != '250' or parts[1] != value_name or parts[2] != e: +fail('Unexpected value in response line "{}". Expected {} for value {}'.format(received, e, value_name)) +if received.startswith('250 '): +break + +if received_count != len(expected): +fail('Expected response with {} lines but received {} lines'.format(len(expected), received_count)) + + +logging.basicConfig(level=logging.DEBUG, +format='%(asctime)s.%(msecs)03d %(message)s', +datefmt='%Y-%m-%d %H:%M:%S') + +if sys.hexversion < 0x0207: +fail("ERROR: unsupported Python version (should be >= 2.7)") + +if sys.hexversion > 0x0300 and sys.hexversion < 0x0301: +fail("ERROR: unsupported Python3 version
[tor-commits] [tor/master] Fix small memory leak in config include tests
commit 3c70f26dc07cea98e009dc8b15009ca8e72ba226 Author: Daniel Pinto Date: Wed Jun 3 22:09:54 2020 +0100 Fix small memory leak in config include tests Tests that used options_init_from_string were not freeing calling config_free_all(). --- src/test/test_config.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/test/test_config.c b/src/test/test_config.c index 617a52b8ac..cb88b95761 100644 --- a/src/test/test_config.c +++ b/src/test/test_config.c @@ -5875,6 +5875,7 @@ test_config_include_flag_both_without(void *data) done: tor_free(errmsg); + config_free_all(); } static void @@ -5915,6 +5916,7 @@ test_config_include_flag_torrc_only(void *data) tor_free(errmsg); tor_free(path); tor_free(dir); + config_free_all(); } static void @@ -5955,6 +5957,7 @@ test_config_include_flag_defaults_only(void *data) tor_free(errmsg); tor_free(path); tor_free(dir); + config_free_all(); } static void ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Add support for patterns on %include #25140
commit 34fa2c4d0d8117b75c5c52a7c825486eb0284ae0 Author: Daniel Pinto Date: Wed Jun 3 22:09:42 2020 +0100 Add support for patterns on %include #25140 Also adds generic tor_glob function to expand globs. --- configure.ac | 5 +- doc/man/tor.1.txt | 12 +- scripts/codegen/fuzzing_include_am.py | 2 +- src/app/include.am| 4 +- src/config/torrc.sample.in| 13 +- src/lib/fs/conffile.c | 141 +++ src/lib/fs/files.c| 16 ++ src/lib/fs/files.h| 2 + src/lib/fs/path.c | 296 +++ src/lib/fs/path.h | 7 + src/lib/sandbox/sandbox.c | 2 + src/lib/string/util_string.c | 9 + src/lib/string/util_string.h | 1 + src/test/fuzz/include.am | 2 +- src/test/include.am | 21 +- src/test/test_config.c| 346 ++- src/test/test_helpers.c | 83 +++ src/test/test_helpers.h | 2 + src/test/test_util.c | 436 ++ src/tools/include.am | 8 +- 20 files changed, 1330 insertions(+), 78 deletions(-) diff --git a/configure.ac b/configure.ac index 3076f2f1ff..b53df59148 100644 --- a/configure.ac +++ b/configure.ac @@ -866,6 +866,7 @@ dnl Where do you live, libevent? And how do we call you? if test "$bwin32" = "true"; then TOR_LIB_WS32=-lws2_32 TOR_LIB_IPHLPAPI=-liphlpapi + TOR_LIB_SHLWAPI=-lshlwapi # Some of the cargo-cults recommend -lwsock32 as well, but I don't # think it's actually necessary. TOR_LIB_GDI=-lgdi32 @@ -878,6 +879,7 @@ fi AC_SUBST(TOR_LIB_WS32) AC_SUBST(TOR_LIB_GDI) AC_SUBST(TOR_LIB_IPHLPAPI) +AC_SUBST(TOR_LIB_SHLWAPI) AC_SUBST(TOR_LIB_USERENV) tor_libevent_pkg_redhat="libevent" @@ -1646,7 +1648,8 @@ AC_CHECK_HEADERS([errno.h \ sys/utime.h \ sys/wait.h \ syslog.h \ - utime.h]) + utime.h \ + glob.h]) AC_CHECK_HEADERS(sys/param.h) diff --git a/doc/man/tor.1.txt b/doc/man/tor.1.txt index ca54fa125b..bb01315d46 100644 --- a/doc/man/tor.1.txt +++ b/doc/man/tor.1.txt @@ -205,14 +205,22 @@ backslash character (\) before the end of the line. Comments can be used in such multiline entries, but they must start at the beginning of a line. Configuration options can be imported from files or folders using the %include -option with the value being a path. If the path is a file, the options from the +option with the value being a path. This path can have wildcards. Wildcards are +expanded first, using lexical order. Then, for each matching file or folder, the +following rules are followed: if the path is a file, the options from the file will be parsed as if they were written where the %include option is. If the path is a folder, all files on that folder will be parsed following lexical -order. Files starting with a dot are ignored. Files on subfolders are ignored. +order. Files starting with a dot are ignored. Files in subfolders are ignored. The %include option can be used recursively. New configuration files or directories cannot be added to already running Tor instance if **Sandbox** is enabled. +The supported wildcards are * meaning any number of characters including none +and ? meaning exactly one character. These characters can be escaped by preceding +them with a backslash, except on Windows. Files starting with a dot are not matched +when expanding wildcards unless the starting dot is explicitly in the pattern, except +on Windows. + By default, an option on the command line overrides an option found in the configuration file, and an option in a configuration file overrides one in the defaults file. diff --git a/scripts/codegen/fuzzing_include_am.py b/scripts/codegen/fuzzing_include_am.py index ae50563074..b3892b6fd3 100755 --- a/scripts/codegen/fuzzing_include_am.py +++ b/scripts/codegen/fuzzing_include_am.py @@ -35,7 +35,7 @@ FUZZING_LIBS = \ $(rust_ldadd) \ @TOR_ZLIB_LIBS@ @TOR_LIB_MATH@ \ @TOR_LIBEVENT_LIBS@ $(TOR_LIBS_CRYPTLIB) \ - @TOR_LIB_WS32@ @TOR_LIB_IPHLPAPI@ @TOR_LIB_GDI@ @TOR_LIB_USERENV@ @CURVE25519_LIBS@ \ + @TOR_LIB_WS32@ @TOR_LIB_IPHLPAPI@ @TOR_LIB_SHLWAPI@ @TOR_LIB_GDI@ @TOR_LIB_USERENV@ @CURVE25519_LIBS@ \ @TOR_SYSTEMD_LIBS@ \ @TOR_LZMA_LIBS@ \ @TOR_ZSTD_LIBS@ diff --git a/src/app/include.am b/src/app/include.am index 3caa0bab1c..8488a1bf19 100644 --- a/src/app/include.am +++ b/src/app/include.am @@ -18,7 +18,7 @@ src_app_tor_LDFLAGS = @TOR_LDFLAGS_zlib@ $(TOR_LDFLAGS_CRYPTLIB) @TOR_LDFLAGS_li src_app_tor_LDADD = $(TOR_INTERNAL_LIBS) \ $(rust_ldadd) \ @TOR_ZLIB_LIBS@ @TOR_LIB_MATH@ @TOR_LIBEVENT_LIBS@ $(TOR_LIBS_CRYPTLIB) \ -
[tor-commits] [tor/master] Merge remote-tracking branch 'tor-gitlab/mr/62'
commit e1514420377b2290dfa52fa65374276a66973131 Merge: 11f1fe3aca 60c436d14d Author: Nick Mathewson Date: Wed Aug 12 14:37:21 2020 -0400 Merge remote-tracking branch 'tor-gitlab/mr/62' .travis.yml | 4 +- changes/feature25140 | 3 + configure.ac | 5 +- doc/man/tor.1.txt | 14 +- scripts/codegen/fuzzing_include_am.py | 2 +- src/app/include.am| 4 +- src/config/torrc.sample.in| 13 +- src/lib/fs/conffile.c | 141 +++ src/lib/fs/files.c| 16 ++ src/lib/fs/files.h| 2 + src/lib/fs/path.c | 362 src/lib/fs/path.h | 7 + src/lib/sandbox/sandbox.c | 2 + src/lib/string/util_string.c | 9 + src/lib/string/util_string.h | 1 + src/test/fuzz/include.am | 2 +- src/test/include.am | 24 +- src/test/test_config.c| 349 ++- src/test/test_helpers.c | 83 +++ src/test/test_helpers.h | 2 + src/test/test_include.py | 196 +++ src/test/test_include.sh | 111 + src/test/test_util.c | 436 ++ src/tools/include.am | 8 +- 24 files changed, 1715 insertions(+), 81 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Improved documentation and comments #25140
commit 0b633b1f6d022a359806a705bd4a6082d92bd3f3 Author: Daniel Pinto Date: Tue Aug 11 18:26:41 2020 +0100 Improved documentation and comments #25140 --- doc/man/tor.1.txt | 6 ++-- src/lib/fs/path.c | 104 -- 2 files changed, 88 insertions(+), 22 deletions(-) diff --git a/doc/man/tor.1.txt b/doc/man/tor.1.txt index bb01315d46..3febf43513 100644 --- a/doc/man/tor.1.txt +++ b/doc/man/tor.1.txt @@ -206,9 +206,9 @@ such multiline entries, but they must start at the beginning of a line. Configuration options can be imported from files or folders using the %include option with the value being a path. This path can have wildcards. Wildcards are -expanded first, using lexical order. Then, for each matching file or folder, the -following rules are followed: if the path is a file, the options from the -file will be parsed as if they were written where the %include option is. If +expanded first, then sorted using lexical order. Then, for each matching file or +folder, the following rules are followed: if the path is a file, the options from +the file will be parsed as if they were written where the %include option is. If the path is a folder, all files on that folder will be parsed following lexical order. Files starting with a dot are ignored. Files in subfolders are ignored. The %include option can be used recursively. diff --git a/src/lib/fs/path.c b/src/lib/fs/path.c index 4532bfb7d1..1a60ca70d4 100644 --- a/src/lib/fs/path.c +++ b/src/lib/fs/path.c @@ -44,13 +44,6 @@ #include #include -#ifdef _WIN32 -#define IS_GLOB_CHAR(s,i) (((s)[(i)]) == '*' || ((s)[(i)]) == '?') -#else -#define IS_GLOB_CHAR(s,i) s)[(i)]) == '*' || ((s)[(i)]) == '?') &&\ - ((i) == 0 || (s)[(i)-1] != '\\')) /* check escape */ -#endif - /** Removes enclosing quotes from path and unescapes quotes between the * enclosing quotes. Backslashes are not unescaped. Return the unquoted * path on success or 0 if path is not quoted correctly. */ @@ -321,8 +314,78 @@ make_path_absolute(const char *fname) #endif /* defined(_WIN32) */ } -/** Expands globs in pattern for the path fragment between - * prev_sep and next_sep. Returns NULL on failure. */ +/* The code below implements tor_glob and get_glob_opened_files. Because it is + * not easy to understand it by looking at individual functions, the big + * picture explanation here should be read first. + * + * Purpose of the functions: + * - tor_glob - recevies a pattern and returns all the paths that result from + * its glob expansion, globs can be present on all path components. + * - get_glob_opened_files - receives a pattern and returns all the paths that + * are opened during its expansion (the paths before any path fragment that + * contains a glob as they have to be opened to check for glob matches). This + * is used to get the paths that have to be added to the seccomp sandbox + * allowed list. + * + * Due to OS API differences explained below, the implementation of tor_glob is + * completly different for Windows and POSIX systems, so we ended up with three + * different implementations: + * - tor_glob for POSIX - as POSIX glob does everything we need, we simply call + * it and process the results. This is completly implemented in tor_glob. + * - tor_glob for WIN32 - because the WIN32 API only supports expanding globs + * in the last path fragment, we need to expand the globs in each path + * fragment manually and call recursively to get the same behaviour as POSIX + * glob. When there are no globs in pattern, we know we are on the last path + * fragment and collect the full path. + * - get_glob_opened_files - because the paths before any path fragment with a + * glob will be opened to check for matches, we need to collect them and we + * need to expand the globs in each path fragments and call recursively until + * we find no more globs. + * + * As seen from the description above, both tor_glob for WIN32 and + * get_glob_opened_files receive a pattern and return a list of paths and have + * to expand all path fragments that contain globs and call themselves + * recursively. The differences are: + * - get_glob_opened_files collects paths before path fragments with globs + * while tor_glob for WIN32 collects full paths resulting from the expansion + * of all globs. + * - get_glob_opened_files can call tor_glob to expand path fragments with + * globs while tor_glob for WIN32 cannot because it IS tor_glob. For tor_glob + * for WIN32, an auxiliary function has to be used for this purpose. + * + * To avoid code duplication, the logic of tor_glob for WIN32 and + * get_glob_opened_files is implemented in get_glob_paths. The differences are + * configured by the extra function parameters: + * - final - if true, returns a list of paths obtained from expanding pattern + * (implements tor_glob). Otherwise, returns the paths before path fragments + *
[tor-commits] [tor/master] Use correct type for gl_closedir to avoid a compiler warning
commit 9ef991dbe37b18c1d3af4a5a4d0b674d3f18cc5f Author: Nick Mathewson Date: Wed Aug 12 14:42:30 2020 -0400 Use correct type for gl_closedir to avoid a compiler warning --- src/lib/fs/path.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/lib/fs/path.c b/src/lib/fs/path.c index 1a60ca70d4..f03cecf52d 100644 --- a/src/lib/fs/path.c +++ b/src/lib/fs/path.c @@ -553,6 +553,12 @@ prot_lstat(const char *pathname, struct stat *buf) { return lstat(sandbox_intern_string(pathname), buf); } +/** As closedir, but has the right type for gl_closedir */ +static void +wrap_closedir(void *arg) +{ + closedir(arg); +} #endif /* defined(_WIN32) */ /** Return a new list containing the paths that match the pattern @@ -580,7 +586,7 @@ tor_glob(const char *pattern) typedef void (*gl_closedir)(void *); matches.gl_opendir = (gl_opendir) _opendir; matches.gl_readdir = (gl_readdir) - matches.gl_closedir = (gl_closedir) + matches.gl_closedir = (gl_closedir) _closedir; matches.gl_stat = _stat; matches.gl_lstat = _lstat; #endif /* defined(GLOB_ALTDIRFUNC) */ ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Changes file for #25140
commit ac2c4b1e4ae297b33be6c971fd0e4aca1198b282 Author: Daniel Pinto Date: Wed Jun 3 22:09:58 2020 +0100 Changes file for #25140 --- changes/feature25140 | 3 +++ 1 file changed, 3 insertions(+) diff --git a/changes/feature25140 b/changes/feature25140 new file mode 100644 index 00..5202fa11ce --- /dev/null +++ b/changes/feature25140 @@ -0,0 +1,3 @@ + o Minor feature (configuration): +- Allow the using wildcards (* and ?) with the %include option on + configuration files. Closes ticket 25140. Patch by Daniel Pinto. ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Attempt to fix test_include.py on TravisCI #25140
commit 60c436d14d05dcc5e03058a16373c53ff94469c5 Author: Daniel Pinto Date: Wed Aug 12 14:35:51 2020 +0100 Attempt to fix test_include.py on TravisCI #25140 --- src/test/test_include.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/test_include.py b/src/test/test_include.py index 0ac87ee8fb..ec261da86c 100644 --- a/src/test/test_include.py +++ b/src/test/test_include.py @@ -127,7 +127,7 @@ tor_process = subprocess.Popen([tor_path, if tor_process == None: fail('ERROR: running tor failed') -wait_for_log('Opened Control listener on') +wait_for_log('Opened Control listener') control_socket = socket.socket(socket.AF_INET, socket.SOCK_STREAM) if control_socket.connect_ex(('127.0.0.1', control_port)): ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [translation/support-portal] https://gitweb.torproject.org/translation.git/commit/?h=support-portal
commit 7de0296a7e027e933e3d24c7d2d456b23cf56fbd Author: Translation commit bot Date: Wed Aug 12 18:18:11 2020 + https://gitweb.torproject.org/translation.git/commit/?h=support-portal --- contents+ar.po | 41 + 1 file changed, 37 insertions(+), 4 deletions(-) diff --git a/contents+ar.po b/contents+ar.po index d943438f02..d829b7fa39 100644 --- a/contents+ar.po +++ b/contents+ar.po @@ -8673,6 +8673,8 @@ msgid "" "Relays and bridges report some of the data in 24-hour intervals which may " "end at any time of the day." msgstr "" +"تÙÙ٠اÙ٠رØÙات ÙاÙجسÙر باÙإبÙاغ ع٠بعض اÙبÙاÙات ÙÙ Ùترات 24 ساعة ÙاÙت٠Ùد " +"تÙتÙÙ Ù٠أ٠ÙÙت ٠٠اÙÙÙÙ ." #: https//support.torproject.org/metrics/graphs/ #: (content/metrics/graphs/contents+en.lrquestion.description) @@ -8680,6 +8682,8 @@ msgid "" "And after such an interval is over relays and bridges might take another 18 " "hours to report the data." msgstr "" +"Ùبعد اÙتÙاء Ùذا اÙÙاص٠اÙز٠ÙÙ Ø Ùد تستغر٠اÙجسÙر 18 ساعة أخر٠ÙÙإبÙاغ ع٠" +"اÙبÙاÙات." #: https//support.torproject.org/metrics/graphs/ #: (content/metrics/graphs/contents+en.lrquestion.description) @@ -8688,13 +8692,16 @@ msgid "" "the last data point in a graph indicates a recent trend change which is in " "fact just an artifact of the algorithm." msgstr "" +"ÙÙد ÙطعÙا اÙرسÙ٠اÙبÙاÙÙØ© Ù٠اÙÙÙÙ Ù٠اÙأخÙرÙÙ Ø ÙØ£ÙÙا ÙرÙد تجÙب أ٠تشÙر آخر " +"ÙÙطة بÙاÙات Ù٠اÙرس٠اÙبÙاÙ٠إÙ٠تغÙÙر ØدÙØ« Ù٠اÙاتجا٠ÙÙÙ Ù٠اÙÙاÙع ٠جرد " +"Ùطعة أثرÙØ© ÙÙØ®Ùارز٠ÙØ©." #: https//support.torproject.org/metrics/methodology/ #: (content/metrics/methodology/contents+en.lrquestion.title) msgid "" "Why do you believe the current approach to estimate user numbers is more " "accurate?" -msgstr "" +msgstr "Ù٠اذا تعتÙد أ٠اÙÙÙج اÙØاÙÙ ÙتÙدÙر أعداد اÙ٠ستخد٠Ù٠أÙثر دÙØ©Ø" #: https//support.torproject.org/metrics/methodology/ #: (content/metrics/methodology/contents+en.lrquestion.description) @@ -8702,6 +8709,8 @@ msgid "" "For direct users, we include all directories which we didn't do in the old " "approach." msgstr "" +"باÙÙسبة ÙÙ٠ستخد٠Ù٠اÙ٠باشرÙÙ Ø ÙÙÙ٠بتض٠Ù٠ج٠Ùع اÙأدÙØ© اÙت٠ÙÙ ÙÙ٠بÙا ÙÙ " +"اÙطرÙÙØ© اÙÙدÙÙ Ø©." #: https//support.torproject.org/metrics/methodology/ #: (content/metrics/methodology/contents+en.lrquestion.description) @@ -8709,6 +8718,8 @@ msgid "" "We also use histories that only contain bytes written to answer directory " "requests, which is more precise than using general byte histories." msgstr "" +"Ùستخد٠أÙضÙا سجÙات تØتÙ٠عÙÙ ÙØدات باÙت ÙÙØ· Ù ÙتÙبة ÙÙإجابة عÙÙ Ø·Ùبات اÙدÙÙÙ " +"Ø ÙÙ٠أÙثر دÙØ© ٠٠استخدا٠سجÙات اÙباÙت اÙعا٠ة." #: https//support.torproject.org/metrics/stats-report/ #: (content/metrics/stats-report/contents+en.lrquestion.title) @@ -8716,6 +8727,8 @@ msgid "" "Are there more fine-grained numbers available, for example, on the number of" " users per hour?" msgstr "" +"ÙÙ ÙÙا٠اÙ٠زÙد ٠٠اÙأرÙا٠اÙدÙÙÙØ© اÙ٠تاØØ© Ø Ø¹Ù٠سبÙ٠اÙÙ Ø«Ø§Ù Ø ØÙ٠عدد " +"اÙ٠ستخد٠ÙÙ Ù٠اÙساعةØ" #: https//support.torproject.org/metrics/stats-report/ #: (content/metrics/stats-report/contents+en.lrquestion.description) @@ -8723,6 +8736,8 @@ msgid "" "No, the relays that report these statistics aggregate requests by country of" " origin and over a period of 24 hours." msgstr "" +"Ùا Ø Ø§Ù٠رØÙات اÙت٠تÙبÙغ ع٠Ùذ٠اÙØ¥ØصائÙات تج٠ع اÙØ·Ùبات Øسب بÙد اÙÙ Ùشأ ÙعÙÙ " +"٠دار 24 ساعة." #: https//support.torproject.org/metrics/stats-report/ #: (content/metrics/stats-report/contents+en.lrquestion.description) @@ -8730,11 +8745,13 @@ msgid "" "The statistics we would need to gather for the number of users per hour " "would be too detailed and might put users at risk." msgstr "" +"ستÙÙ٠اÙØ¥Øصاءات اÙت٠سÙØتاج Ø¥Ù٠ج٠عÙا Ùعدد اÙ٠ستخد٠ÙÙ Ù٠اÙساعة Ù ÙصÙØ© ÙÙغاÙØ©" +" ÙÙد تعرض اÙ٠ستخد٠ÙÙ ÙÙخطر." #: https//support.torproject.org/metrics/user-estimation/ #: (content/metrics/user-estimation/contents+en.lrquestion.title) msgid "How is it even possible to count users in an anonymity network?" -msgstr "" +msgstr "ÙÙÙ ÙÙ ÙÙ Øت٠عد اÙ٠ستخد٠ÙÙ Ù٠شبÙØ© اÙ٠جÙÙÙÙØ©Ø" #: https//support.torproject.org/metrics/user-estimation/ #: (content/metrics/user-estimation/contents+en.lrquestion.description) @@ -8743,11
[tor-commits] [translation/support-portal] https://gitweb.torproject.org/translation.git/commit/?h=support-portal
commit 5f5206cdae7eb31490131f42133c03cfe10f68b5 Author: Translation commit bot Date: Wed Aug 12 17:48:00 2020 + https://gitweb.torproject.org/translation.git/commit/?h=support-portal --- contents+ar.po | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/contents+ar.po b/contents+ar.po index 173c2f2605..d943438f02 100644 --- a/contents+ar.po +++ b/contents+ar.po @@ -8603,11 +8603,13 @@ msgid "" "Average number of concurrent users, estimated from data collected over a " "day. We can't say how many distinct users there are." msgstr "" +"٠تÙسط عدد اÙ٠ستخد٠Ù٠اÙ٠تزا٠ÙÙÙ Ø Ù ÙدرÙا ٠٠اÙبÙاÙات اÙت٠ت٠ج٠عÙا عÙ٠٠دار " +"ÙÙÙ . Ùا ÙÙ ÙÙÙا تØدÙد عدد اÙ٠ستخد٠Ù٠اÙ٠ت٠ÙزÙÙ." #: https//support.torproject.org/metrics/country-origin/ #: (content/metrics/country-origin/contents+en.lrquestion.title) msgid "How do you know which countries users come from?" -msgstr "" +msgstr "ÙÙ٠تعر٠اÙبÙدا٠اÙت٠Ùأت٠٠ÙÙا اÙ٠ستخد٠ÙÙØ" #: https//support.torproject.org/metrics/country-origin/ #: (content/metrics/country-origin/contents+en.lrquestion.description) @@ -8616,11 +8618,13 @@ msgid "" "numbers in aggregate form. This is one of the reasons why tor ships with a " "GeoIP database." msgstr "" +"تÙÙ٠اÙدÙائ٠بتØÙÙ٠عÙاÙÙÙ IP Ø¥Ù٠ر٠Ùز اÙبÙدا٠ÙاÙإبÙاغ ع٠Ùذ٠اÙأرÙا٠ÙÙ " +"Ø´Ù٠إج٠اÙÙ. Ùذا Ù٠أØد أسباب Ø´ØÙ tor ٠ع Ùاعدة بÙاÙات GeoIP." #: https//support.torproject.org/metrics/directories-reports/ #: (content/metrics/directories-reports/contents+en.lrquestion.title) msgid "Do all directories report these directory request numbers?" -msgstr "" +msgstr "Ù٠تÙÙ٠ج٠Ùع اÙأدÙØ© باÙإبÙاغ ع٠أرÙا٠طÙبات اÙدÙÙÙ ÙØ°ÙØ" #: https//support.torproject.org/metrics/directories-reports/ #: (content/metrics/directories-reports/contents+en.lrquestion.description) @@ -8628,6 +8632,8 @@ msgid "" "No, but we can see what fraction of directories reported them, and then we " "can extrapolate the total number in the network." msgstr "" +"Ùا Ø ÙÙÙÙ ÙÙ ÙÙÙا Ø£Ù Ùر٠أ٠جزء ٠٠اÙأدÙØ© أبÙغت عÙÙا Ø ÙÙ Ù Ø«Ù ÙÙ ÙÙÙا استÙراء " +"اÙعدد اÙإج٠اÙÙ Ù٠اÙشبÙØ©." #: https//support.torproject.org/metrics/graph-data-point/ #: (content/metrics/graph-data-point/contents+en.lrquestion.title) @@ -8635,6 +8641,8 @@ msgid "" "But I noticed that the last data point went up/down a bit since I last " "looked a few hours ago. Why is that?" msgstr "" +"ÙÙÙÙÙ ÙاØظت أ٠آخر ÙÙطة بÙاÙات ارتÙعت / اÙØ®Ùضت ÙÙÙÙا٠٠ÙØ° آخر ٠رة بØثت ÙÙÙا " +"Ùب٠ساعات ÙÙÙÙØ©. Ù٠اذا ÙذاØ" #: https//support.torproject.org/metrics/graph-data-point/ #: (content/metrics/graph-data-point/contents+en.lrquestion.description) @@ -8642,6 +8650,8 @@ msgid "" "The reason is that we publish user numbers once we're confident enough that " "they won't change significantly anymore." msgstr "" +"ÙاÙسبب Ù٠أÙÙا ÙÙشر أرÙا٠اÙ٠ستخد٠Ù٠ب٠جرد Ø£Ù ÙÙÙÙ ÙاثÙÙ٠بدرجة ÙاÙÙØ© Ù Ù " +"Ø£ÙÙا Ù٠تتغÙر بشÙÙ ÙبÙر بعد اÙØ¢Ù." #: https//support.torproject.org/metrics/graph-data-point/ #: (content/metrics/graph-data-point/contents+en.lrquestion.description) @@ -8649,11 +8659,13 @@ msgid "" "But it's always possible that a directory reports data a few hours after we " "were confident enough, but which then slightly changed the graph." msgstr "" +"ÙÙÙ٠٠٠اÙÙ Ù Ù٠دائ٠Ùا Ø£Ù ÙÙÙ٠اÙدÙÙ٠باÙإبÙاغ ع٠اÙبÙاÙات بعد بضع ساعات Ù Ù " +"Ø«ÙتÙا ب٠ا ÙÙ٠اÙÙÙاÙØ© Ø ÙÙÙ٠بعد Ø°Ù٠غÙÙر اÙرس٠اÙبÙاÙÙ ÙÙÙÙاÙ." #: https//support.torproject.org/metrics/graphs/ #: (content/metrics/graphs/contents+en.lrquestion.title) msgid "Why do the graphs end 2 days in the past and not today?" -msgstr "" +msgstr "Ù٠اذا تÙتÙ٠اÙرسÙ٠اÙبÙاÙÙØ© بÙÙÙ ÙÙ Ù٠اÙ٠اض٠ÙÙÙس اÙÙÙÙ Ø" #: https//support.torproject.org/metrics/graphs/ #: (content/metrics/graphs/contents+en.lrquestion.description) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [translation/support-portal] https://gitweb.torproject.org/translation.git/commit/?h=support-portal
commit 75e494b803003472a3577f4db4cfcd1bf4b4a776 Author: Translation commit bot Date: Wed Aug 12 17:18:08 2020 + https://gitweb.torproject.org/translation.git/commit/?h=support-portal --- contents+ar.po | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/contents+ar.po b/contents+ar.po index 0dba68d1e7..173c2f2605 100644 --- a/contents+ar.po +++ b/contents+ar.po @@ -8525,17 +8525,18 @@ msgstr "" msgid "" "tl;dr: in the old approach we measured the wrong thing, and now we measure " "the right thing." -msgstr "" +msgstr "Ù٠اÙÙÙج اÙÙدÙÙ ÙÙ Ùا بÙÙاس اÙØ´ÙØ¡ اÙخطأ Ø ÙاÙØ¢Ù ÙÙÙس اÙØ´ÙØ¡ اÙصØÙØ." #: https//support.torproject.org/metrics/bridges-reports/ #: (content/metrics/bridges-reports/contents+en.lrquestion.title) msgid "Why are there so few bridge users that are not using the default OR" msgstr "" +"Ù٠اذا ÙÙجد عدد ÙÙÙ٠جدÙا ٠٠٠ستخد٠٠اÙجسر اÙØ°ÙÙ Ùا Ùستخد٠ÙÙ OR اÙاÙتراضÙ" #: https//support.torproject.org/metrics/bridges-reports/ #: (content/metrics/bridges-reports/contents+en.lrquestion.title) msgid "protocol or that are using IPv6?" -msgstr "" +msgstr "اÙبرÙتÙÙÙ٠أ٠اÙت٠تستخد٠IPv6Ø" #: https//support.torproject.org/metrics/bridges-reports/ #: (content/metrics/bridges-reports/contents+en.lrquestion.description) @@ -8543,18 +8544,21 @@ msgid "" "Very few bridges report data on transports or IP versions yet, and by " "default we consider requests to use the default OR protocol and IPv4." msgstr "" +"عدد ÙÙÙ٠جدÙا ٠٠اÙجسÙر ÙبÙغ ع٠بÙاÙات ØÙ٠ع٠ÙÙات اÙÙÙ٠أ٠إصدارات IP Øت٠" +"اÙØ¢Ù Ø ÙبشÙ٠اÙتراض٠ÙØÙ Ùأخذ Ù٠اÙاعتبار Ø·Ùبات استخدا٠برÙتÙÙÙÙ OR " +"اÙاÙتراض٠٠IPv4." #: https//support.torproject.org/metrics/bridges-reports/ #: (content/metrics/bridges-reports/contents+en.lrquestion.description) msgid "" "Once more bridges report these data, the numbers will become more accurate." -msgstr "" +msgstr "ب٠جرد أ٠تبÙغ اÙجسÙر ع٠Ùذ٠اÙبÙاÙات Ø Ø³ØªØµØ¨Ø Ø§ÙأرÙا٠أÙثر دÙØ©." #: https//support.torproject.org/metrics/censorship-events/ #: (content/metrics/censorship-events/contents+en.lrquestion.title) msgid "" "What are these red and blue dots indicating possible censorship events?" -msgstr "" +msgstr "٠ا ÙÙ Ùذ٠اÙÙÙاط اÙØ٠راء ÙاÙزرÙاء اÙت٠تشÙر Ø¥Ù٠أØداث رÙابة Ù Øت٠ÙØ©Ø" #: https//support.torproject.org/metrics/censorship-events/ #: (content/metrics/censorship-events/contents+en.lrquestion.description) @@ -8563,6 +8567,8 @@ msgid "" "user numbers over a series of days and predicts the user number in the next " "days." msgstr "" +"ÙØÙ ÙدÙر Ùظا٠Ùا ÙاÙتشا٠اÙرÙابة Ùائ٠Ùا عÙ٠اÙشذÙØ° ÙبØØ« Ù٠أعداد اÙ٠ستخد٠ÙÙ " +"اÙÙ Ùدرة عÙ٠٠دار سÙسÙØ© ٠٠اÙØ£Ùا٠ÙÙتÙÙع عدد اÙ٠ستخد٠ÙÙ Ù٠اÙØ£Ùا٠اÙÙاد٠ة." #: https//support.torproject.org/metrics/censorship-events/ #: (content/metrics/censorship-events/contents+en.lrquestion.description) @@ -8570,6 +8576,8 @@ msgid "" "If the actual number is higher or lower, this might indicate a possible " "censorship event or release of censorship." msgstr "" +"إذا Ùا٠اÙرÙ٠اÙÙعÙ٠أعÙ٠أ٠أÙÙ Ø ÙÙد ÙØ´Ùر Ø°Ù٠إÙ٠اØت٠ا٠ÙجÙد Øدث رÙابة Ø£Ù" +" تØرÙر ÙÙرÙابة." #: https//support.torproject.org/metrics/censorship-events/ #: (content/metrics/censorship-events/contents+en.lrquestion.description) @@ -8577,6 +8585,8 @@ msgid "" "For more details, see our [technical " "report](https://research.torproject.org/techreports/detector-2011-09-09.pdf)." msgstr "" +"Ù٠زÙد ٠٠اÙتÙاصÙÙ Ø Ø±Ø§Ø¬Ø¹ [تÙرÙرÙا اÙÙÙÙ] " +"(https://research.torproject.org/techreports/detector-2011-09-09.pdf)." #: https//support.torproject.org/metrics/concurrent-users/ #: (content/metrics/concurrent-users/contents+en.lrquestion.title) @@ -8584,6 +8594,8 @@ msgid "" "So, are these distinct users per day, average number of users connected over" " the day, or what?" msgstr "" +"Ø¥Ø°Ù Ø ÙÙ ÙؤÙاء اÙ٠ستخد٠Ù٠اÙ٠ت٠ÙزÙÙ ÙÙÙ ÙÙا Ø Ù ØªÙسط عدد اÙ٠ستخد٠Ù٠اÙ٠تصÙÙÙ " +"عÙ٠٠دار اÙÙÙÙ Ø Ø£Ù Ù Ø§Ø°Ø§Ø" #: https//support.torproject.org/metrics/concurrent-users/ #: (content/metrics/concurrent-users/contents+en.lrquestion.description) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.4.3' into release-0.4.3
commit f94ad3d32a92f3709df37ae06b57ca47616a552f Merge: cff2c3d5fa 309eafbc3c Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.3' into release-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.3' into maint-0.4.4
commit f98b1988ec5ca1eb3d2923faf43c1df7541efafb Merge: 670f5632a5 309eafbc3c Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.4' into release-0.4.4
commit 1b3c206cf3f88792d6a78fc50b263fc145f910ea Merge: 16d5201af1 f98b1988ec Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.4' into release-0.4.4 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 309eafbc3c0064a670f5f4b11c3639f7ade6229b Merge: 489d33e190 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) diff --cc Makefile.am index 3b30c01f00,f6346fe0fc..1041494da7 --- a/Makefile.am +++ b/Makefile.am @@@ -162,53 -157,40 +162,55 @@@ include src/include.a include doc/include.am include contrib/include.am + manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ - ChangeLog \ - CONTRIBUTING\ - CODE_OF_CONDUCT \ - INSTALL \ - LICENSE \ - Makefile.nmake \ - README \ - ReleaseNotes\ - scripts/maint/checkIncludes.py \ - scripts/maint/checkSpace.pl \ - scripts/maint/checkShellScripts.sh \ - scripts/maint/practracker/README\ - scripts/maint/practracker/exceptions.txt\ - scripts/maint/practracker/includes.py \ - scripts/maint/practracker/metrics.py\ - scripts/maint/practracker/practracker.py\ - scripts/maint/practracker/practracker_tests.py \ - scripts/maint/practracker/problem.py\ - scripts/maint/practracker/testdata/.may_include \ - scripts/maint/practracker/testdata/a.c \ - scripts/maint/practracker/testdata/b.c \ - scripts/maint/practracker/testdata/ex0-expected.txt \ - scripts/maint/practracker/testdata/ex0.txt \ - scripts/maint/practracker/testdata/ex1-expected.txt \ - scripts/maint/practracker/testdata/ex1.txt \ - scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ - scripts/maint/practracker/testdata/ex.txt \ - scripts/maint/practracker/testdata/header.h \ - scripts/maint/practracker/testdata/not_c_file \ - scripts/maint/practracker/test_practracker.sh \ - scripts/maint/practracker/util.py + ChangeLog \ + CONTRIBUTING\ + CODE_OF_CONDUCT \ + INSTALL \ + LICENSE \ + Makefile.nmake \ + README \ + ReleaseNotes\ + scripts/maint/checkIncludes.py \ + scripts/maint/checkSpace.pl \ + scripts/maint/checkSpaceTest.sh \ + scripts/maint/checkspace_tests/dubious.c\ + scripts/maint/checkspace_tests/dubious.h\ + scripts/maint/checkspace_tests/expected.txt \ + scripts/maint/checkspace_tests/good_guard.h \ + scripts/maint/checkspace_tests/same_guard.h \ + scripts/maint/checkspace_tests/subdir/dubious.c \ + scripts/maint/checkShellScripts.sh \ + scripts/maint/practracker/README\ + scripts/maint/practracker/exceptions.txt\ + scripts/maint/practracker/includes.py \ + scripts/maint/practracker/metrics.py\ + scripts/maint/practracker/practracker.py\ + scripts/maint/practracker/practracker_tests.py \ + scripts/maint/practracker/problem.py\ + scripts/maint/practracker/testdata/.may_include \ + scripts/maint/practracker/testdata/a.c \ + scripts/maint/practracker/testdata/b.c \ + scripts/maint/practracker/testdata/ex0-expected.txt \ + scripts/maint/practracker/testdata/ex0.txt \ + scripts/maint/practracker/testdata/ex1-expected.txt \ + scripts/maint/practracker/testdata/ex1.txt \ + scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ + scripts/maint/practracker/testdata/ex1-regen-expected.txt
[tor-commits] [tor/release-0.4.2] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.4.3' into maint-0.4.4
commit f98b1988ec5ca1eb3d2923faf43c1df7541efafb Merge: 670f5632a5 309eafbc3c Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.3] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 309eafbc3c0064a670f5f4b11c3639f7ade6229b Merge: 489d33e190 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) diff --cc Makefile.am index 3b30c01f00,f6346fe0fc..1041494da7 --- a/Makefile.am +++ b/Makefile.am @@@ -162,53 -157,40 +162,55 @@@ include src/include.a include doc/include.am include contrib/include.am + manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ - ChangeLog \ - CONTRIBUTING\ - CODE_OF_CONDUCT \ - INSTALL \ - LICENSE \ - Makefile.nmake \ - README \ - ReleaseNotes\ - scripts/maint/checkIncludes.py \ - scripts/maint/checkSpace.pl \ - scripts/maint/checkShellScripts.sh \ - scripts/maint/practracker/README\ - scripts/maint/practracker/exceptions.txt\ - scripts/maint/practracker/includes.py \ - scripts/maint/practracker/metrics.py\ - scripts/maint/practracker/practracker.py\ - scripts/maint/practracker/practracker_tests.py \ - scripts/maint/practracker/problem.py\ - scripts/maint/practracker/testdata/.may_include \ - scripts/maint/practracker/testdata/a.c \ - scripts/maint/practracker/testdata/b.c \ - scripts/maint/practracker/testdata/ex0-expected.txt \ - scripts/maint/practracker/testdata/ex0.txt \ - scripts/maint/practracker/testdata/ex1-expected.txt \ - scripts/maint/practracker/testdata/ex1.txt \ - scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ - scripts/maint/practracker/testdata/ex.txt \ - scripts/maint/practracker/testdata/header.h \ - scripts/maint/practracker/testdata/not_c_file \ - scripts/maint/practracker/test_practracker.sh \ - scripts/maint/practracker/util.py + ChangeLog \ + CONTRIBUTING\ + CODE_OF_CONDUCT \ + INSTALL \ + LICENSE \ + Makefile.nmake \ + README \ + ReleaseNotes\ + scripts/maint/checkIncludes.py \ + scripts/maint/checkSpace.pl \ + scripts/maint/checkSpaceTest.sh \ + scripts/maint/checkspace_tests/dubious.c\ + scripts/maint/checkspace_tests/dubious.h\ + scripts/maint/checkspace_tests/expected.txt \ + scripts/maint/checkspace_tests/good_guard.h \ + scripts/maint/checkspace_tests/same_guard.h \ + scripts/maint/checkspace_tests/subdir/dubious.c \ + scripts/maint/checkShellScripts.sh \ + scripts/maint/practracker/README\ + scripts/maint/practracker/exceptions.txt\ + scripts/maint/practracker/includes.py \ + scripts/maint/practracker/metrics.py\ + scripts/maint/practracker/practracker.py\ + scripts/maint/practracker/practracker_tests.py \ + scripts/maint/practracker/problem.py\ + scripts/maint/practracker/testdata/.may_include \ + scripts/maint/practracker/testdata/a.c \ + scripts/maint/practracker/testdata/b.c \ + scripts/maint/practracker/testdata/ex0-expected.txt \ + scripts/maint/practracker/testdata/ex0.txt \ + scripts/maint/practracker/testdata/ex1-expected.txt \ + scripts/maint/practracker/testdata/ex1.txt \ + scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ + scripts/maint/practracker/testdata/ex1-regen-expected.txt
[tor-commits] [tor/release-0.4.2] Merge branch 'maint-0.4.2' into release-0.4.2
commit b6fb925e99945b40573a96e469117ed2bf9e884f Merge: 6753b804e4 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into release-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.4] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 309eafbc3c0064a670f5f4b11c3639f7ade6229b Merge: 489d33e190 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) diff --cc Makefile.am index 3b30c01f00,f6346fe0fc..1041494da7 --- a/Makefile.am +++ b/Makefile.am @@@ -162,53 -157,40 +162,55 @@@ include src/include.a include doc/include.am include contrib/include.am + manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ - ChangeLog \ - CONTRIBUTING\ - CODE_OF_CONDUCT \ - INSTALL \ - LICENSE \ - Makefile.nmake \ - README \ - ReleaseNotes\ - scripts/maint/checkIncludes.py \ - scripts/maint/checkSpace.pl \ - scripts/maint/checkShellScripts.sh \ - scripts/maint/practracker/README\ - scripts/maint/practracker/exceptions.txt\ - scripts/maint/practracker/includes.py \ - scripts/maint/practracker/metrics.py\ - scripts/maint/practracker/practracker.py\ - scripts/maint/practracker/practracker_tests.py \ - scripts/maint/practracker/problem.py\ - scripts/maint/practracker/testdata/.may_include \ - scripts/maint/practracker/testdata/a.c \ - scripts/maint/practracker/testdata/b.c \ - scripts/maint/practracker/testdata/ex0-expected.txt \ - scripts/maint/practracker/testdata/ex0.txt \ - scripts/maint/practracker/testdata/ex1-expected.txt \ - scripts/maint/practracker/testdata/ex1.txt \ - scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ - scripts/maint/practracker/testdata/ex.txt \ - scripts/maint/practracker/testdata/header.h \ - scripts/maint/practracker/testdata/not_c_file \ - scripts/maint/practracker/test_practracker.sh \ - scripts/maint/practracker/util.py + ChangeLog \ + CONTRIBUTING\ + CODE_OF_CONDUCT \ + INSTALL \ + LICENSE \ + Makefile.nmake \ + README \ + ReleaseNotes\ + scripts/maint/checkIncludes.py \ + scripts/maint/checkSpace.pl \ + scripts/maint/checkSpaceTest.sh \ + scripts/maint/checkspace_tests/dubious.c\ + scripts/maint/checkspace_tests/dubious.h\ + scripts/maint/checkspace_tests/expected.txt \ + scripts/maint/checkspace_tests/good_guard.h \ + scripts/maint/checkspace_tests/same_guard.h \ + scripts/maint/checkspace_tests/subdir/dubious.c \ + scripts/maint/checkShellScripts.sh \ + scripts/maint/practracker/README\ + scripts/maint/practracker/exceptions.txt\ + scripts/maint/practracker/includes.py \ + scripts/maint/practracker/metrics.py\ + scripts/maint/practracker/practracker.py\ + scripts/maint/practracker/practracker_tests.py \ + scripts/maint/practracker/problem.py\ + scripts/maint/practracker/testdata/.may_include \ + scripts/maint/practracker/testdata/a.c \ + scripts/maint/practracker/testdata/b.c \ + scripts/maint/practracker/testdata/ex0-expected.txt \ + scripts/maint/practracker/testdata/ex0.txt \ + scripts/maint/practracker/testdata/ex1-expected.txt \ + scripts/maint/practracker/testdata/ex1.txt \ + scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ + scripts/maint/practracker/testdata/ex1-regen-expected.txt
[tor-commits] [tor/master] Merge branch 'maint-0.4.4'
commit 11f1fe3aca08c19ab07d9380c1bd47678c99abc4 Merge: 06c31b1b79 f98b1988ec Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.4' .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] Merge branch 'maint-0.3.5' into release-0.3.5
commit ac5a06d42c0eed5686d7d63efc2610a8b2420a14 Merge: 607325122f f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into release-0.3.5 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.4.2] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 309eafbc3c0064a670f5f4b11c3639f7ade6229b Merge: 489d33e190 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) diff --cc Makefile.am index 3b30c01f00,f6346fe0fc..1041494da7 --- a/Makefile.am +++ b/Makefile.am @@@ -162,53 -157,40 +162,55 @@@ include src/include.a include doc/include.am include contrib/include.am + manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ - ChangeLog \ - CONTRIBUTING\ - CODE_OF_CONDUCT \ - INSTALL \ - LICENSE \ - Makefile.nmake \ - README \ - ReleaseNotes\ - scripts/maint/checkIncludes.py \ - scripts/maint/checkSpace.pl \ - scripts/maint/checkShellScripts.sh \ - scripts/maint/practracker/README\ - scripts/maint/practracker/exceptions.txt\ - scripts/maint/practracker/includes.py \ - scripts/maint/practracker/metrics.py\ - scripts/maint/practracker/practracker.py\ - scripts/maint/practracker/practracker_tests.py \ - scripts/maint/practracker/problem.py\ - scripts/maint/practracker/testdata/.may_include \ - scripts/maint/practracker/testdata/a.c \ - scripts/maint/practracker/testdata/b.c \ - scripts/maint/practracker/testdata/ex0-expected.txt \ - scripts/maint/practracker/testdata/ex0.txt \ - scripts/maint/practracker/testdata/ex1-expected.txt \ - scripts/maint/practracker/testdata/ex1.txt \ - scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ - scripts/maint/practracker/testdata/ex.txt \ - scripts/maint/practracker/testdata/header.h \ - scripts/maint/practracker/testdata/not_c_file \ - scripts/maint/practracker/test_practracker.sh \ - scripts/maint/practracker/util.py + ChangeLog \ + CONTRIBUTING\ + CODE_OF_CONDUCT \ + INSTALL \ + LICENSE \ + Makefile.nmake \ + README \ + ReleaseNotes\ + scripts/maint/checkIncludes.py \ + scripts/maint/checkSpace.pl \ + scripts/maint/checkSpaceTest.sh \ + scripts/maint/checkspace_tests/dubious.c\ + scripts/maint/checkspace_tests/dubious.h\ + scripts/maint/checkspace_tests/expected.txt \ + scripts/maint/checkspace_tests/good_guard.h \ + scripts/maint/checkspace_tests/same_guard.h \ + scripts/maint/checkspace_tests/subdir/dubious.c \ + scripts/maint/checkShellScripts.sh \ + scripts/maint/practracker/README\ + scripts/maint/practracker/exceptions.txt\ + scripts/maint/practracker/includes.py \ + scripts/maint/practracker/metrics.py\ + scripts/maint/practracker/practracker.py\ + scripts/maint/practracker/practracker_tests.py \ + scripts/maint/practracker/problem.py\ + scripts/maint/practracker/testdata/.may_include \ + scripts/maint/practracker/testdata/a.c \ + scripts/maint/practracker/testdata/b.c \ + scripts/maint/practracker/testdata/ex0-expected.txt \ + scripts/maint/practracker/testdata/ex0.txt \ + scripts/maint/practracker/testdata/ex1-expected.txt \ + scripts/maint/practracker/testdata/ex1.txt \ + scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ + scripts/maint/practracker/testdata/ex1-regen-expected.txt
[tor-commits] [tor/master] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.4.3' into maint-0.4.4
commit f98b1988ec5ca1eb3d2923faf43c1df7541efafb Merge: 670f5632a5 309eafbc3c Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.3' into maint-0.4.4 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/release-0.3.5] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.3.5] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Try disabling "make all" when checking docs.
commit 50b7bd243fdea1ae1ab3ee0c3304d9d39246581c Author: Nick Mathewson Date: Wed Aug 12 11:35:15 2020 -0400 Try disabling "make all" when checking docs. --- .gitlab-ci.yml | 2 +- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 54 ++--- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 3959bba024..8ed615687b 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -111,7 +111,6 @@ debian-distcheck: script: - ./scripts/ci/ci-driver.sh -# TODO: it would be neat to disable the "make all" part of this one. debian-docs: image: debian:stable <<: *debian-template @@ -119,6 +118,7 @@ debian-docs: DOXYGEN: "yes" ASCIIDOC: "yes" CHECK: "no" +RUN_STAGE_BUILD: "no" script: - ./scripts/ci/ci-driver.sh diff --git a/Makefile.am b/Makefile.am index 5d18666edc..a868be7362 100644 --- a/Makefile.am +++ b/Makefile.am @@ -149,6 +149,8 @@ include src/include.am include doc/include.am include contrib/include.am +manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ ChangeLog \ CONTRIBUTING\ diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index 05c1415da0..ba4b800c04 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -21,6 +21,11 @@ COLOR_CI="${COLOR_CI:-yes}" # Options for which CI system this is. ON_GITLAB="${ON_GITLAB:-yes}" +# Options for things we usually won't want to skip. +RUN_STAGE_CONFIGURE="${RUN_STAGE_CONFIGURE:-yes}" +RUN_STAGE_BUILD="${RUN_STAGE_BUILD:-yes}" +RUN_STAGE_TEST="${RUN_STAGE_TEST:-yes}" + # Options for how to build Tor. All should be yes/no. FATAL_WARNINGS="${FATAL_WARNINGS:-yes}" HARDENING="${HARDENING:-no}" @@ -165,33 +170,6 @@ else } fi -if [[ "$*" == "" ]]; then -RUN_STAGE_CONFIGURE="yes" -RUN_STAGE_BUILD="yes" -RUN_STAGE_TEST="yes" -else -RUN_STAGE_CONFIGURE="no" -RUN_STAGE_BUILD="no" -RUN_STAGE_TEST="no" - -for stage in "$@"; do - case "$stage" in - configure) - RUN_STAGE_CONFIGURE="yes" - ;; - build) - RUN_STAGE_BUILD="yes" - ;; - test) - RUN_STAGE_TEST="yes" - ;; - *) - error "Unknown stage $stage" - ;; - esac -done -fi - # # Validate inputs. @@ -206,6 +184,10 @@ yes_or_no RUST yes_or_no DOXYGEN yes_or_no ASCIIDOC +yes_or_no RUN_STAGE_CONFIGURE +yes_or_no RUN_STAGE_BUILD +yes_or_no RUN_STAGE_TEST + yes_or_no CHECK yes_or_no STEM yes_or_no DISTCHECK @@ -370,6 +352,7 @@ if [[ "$RUN_STAGE_BUILD" = "yes" ]] ; then end_section Distcheck fi fi + ## # Run tests. @@ -378,13 +361,6 @@ if [[ "$RUN_STAGE_TEST" == "no" ]]; then exit 0 fi -if [[ "$RUN_STAGE_BUILD" = "no" ]] ; then -debug "Skipped build stage. Making sure that ./src/app/tor exists." -if [[ ! -f "./src/app/tor" ]]; then - die "$(pwd)/src/app/tor does not exist" -fi -fi - FAILED_TESTS="" if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then @@ -397,6 +373,16 @@ if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then end_section Doxygen fi +if [[ "${ASCIIDOC}" = 'yes' ]]; then +start_section Asciidoc +if runcmd make manpages; then +hooray "make manpages has succeeded." +else +FAILED_TESTS="${FAILED_TESTS} asciidoc" +fi +end_section Asciidoc +fi + if [[ "${CHECK}" = "yes" ]]; then start_section "Check" if runcmd make "${make_options[@]}" check; then ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/master] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.4] Merge branch 'maint-0.4.2' into maint-0.4.3
commit 309eafbc3c0064a670f5f4b11c3639f7ade6229b Merge: 489d33e190 02bb719a94 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.4.2' into maint-0.4.3 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) diff --cc Makefile.am index 3b30c01f00,f6346fe0fc..1041494da7 --- a/Makefile.am +++ b/Makefile.am @@@ -162,53 -157,40 +162,55 @@@ include src/include.a include doc/include.am include contrib/include.am + manpages: $(nodist_man1_MANS) + EXTRA_DIST+= \ - ChangeLog \ - CONTRIBUTING\ - CODE_OF_CONDUCT \ - INSTALL \ - LICENSE \ - Makefile.nmake \ - README \ - ReleaseNotes\ - scripts/maint/checkIncludes.py \ - scripts/maint/checkSpace.pl \ - scripts/maint/checkShellScripts.sh \ - scripts/maint/practracker/README\ - scripts/maint/practracker/exceptions.txt\ - scripts/maint/practracker/includes.py \ - scripts/maint/practracker/metrics.py\ - scripts/maint/practracker/practracker.py\ - scripts/maint/practracker/practracker_tests.py \ - scripts/maint/practracker/problem.py\ - scripts/maint/practracker/testdata/.may_include \ - scripts/maint/practracker/testdata/a.c \ - scripts/maint/practracker/testdata/b.c \ - scripts/maint/practracker/testdata/ex0-expected.txt \ - scripts/maint/practracker/testdata/ex0.txt \ - scripts/maint/practracker/testdata/ex1-expected.txt \ - scripts/maint/practracker/testdata/ex1.txt \ - scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ - scripts/maint/practracker/testdata/ex.txt \ - scripts/maint/practracker/testdata/header.h \ - scripts/maint/practracker/testdata/not_c_file \ - scripts/maint/practracker/test_practracker.sh \ - scripts/maint/practracker/util.py + ChangeLog \ + CONTRIBUTING\ + CODE_OF_CONDUCT \ + INSTALL \ + LICENSE \ + Makefile.nmake \ + README \ + ReleaseNotes\ + scripts/maint/checkIncludes.py \ + scripts/maint/checkSpace.pl \ + scripts/maint/checkSpaceTest.sh \ + scripts/maint/checkspace_tests/dubious.c\ + scripts/maint/checkspace_tests/dubious.h\ + scripts/maint/checkspace_tests/expected.txt \ + scripts/maint/checkspace_tests/good_guard.h \ + scripts/maint/checkspace_tests/same_guard.h \ + scripts/maint/checkspace_tests/subdir/dubious.c \ + scripts/maint/checkShellScripts.sh \ + scripts/maint/practracker/README\ + scripts/maint/practracker/exceptions.txt\ + scripts/maint/practracker/includes.py \ + scripts/maint/practracker/metrics.py\ + scripts/maint/practracker/practracker.py\ + scripts/maint/practracker/practracker_tests.py \ + scripts/maint/practracker/problem.py\ + scripts/maint/practracker/testdata/.may_include \ + scripts/maint/practracker/testdata/a.c \ + scripts/maint/practracker/testdata/b.c \ + scripts/maint/practracker/testdata/ex0-expected.txt \ + scripts/maint/practracker/testdata/ex0.txt \ + scripts/maint/practracker/testdata/ex1-expected.txt \ + scripts/maint/practracker/testdata/ex1.txt \ + scripts/maint/practracker/testdata/ex1-overbroad-expected.txt \ + scripts/maint/practracker/testdata/ex1-regen-expected.txt
[tor-commits] [tor/maint-0.4.3] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.2] CI: Try to enable integration tests, hardening, and clang.
commit fe0e62ddc40a59d4f138d53840a2919e9aac7b4d Author: Nick Mathewson Date: Wed Aug 12 09:38:19 2020 -0400 CI: Try to enable integration tests, hardening, and clang. --- .gitlab-ci.yml | 34 +- scripts/ci/ci-driver.sh | 3 +++ 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index abc5003678..3959bba024 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -66,6 +66,7 @@ variables: - apt-get install automake build-essential +ca-certificates git libevent-dev liblzma-dev @@ -77,6 +78,10 @@ variables: zlib1g-dev - if [ "$ASCIIDOC" = yes ]; then apt-get install asciidoc xmlto; fi - if [ "$DOXYGEN" = yes ]; then apt-get install doxygen; fi +- if [ "$STEM" = yes ]; then apt-get install timelimit; fi +- if [ "$CC" = clang ]; then apt-get install clang; fi +- if [ "$STEM" = yes ]; then git clone --depth 1 https://git.torproject.org/stem.git ; export STEM_PATH="$(pwd)/stem"; fi +- if [ "$CHUTNEY" = yes ]; then git clone --depth 1 https://git.torproject.org/chutney.git ; export CHUTNEY_PATH="$(pwd)/chutney"; fi debian-minimal: image: debian:stable @@ -84,17 +89,18 @@ debian-minimal: script: - ./scripts/ci/ci-driver.sh -### -# Temporarily diabled. This one just takes too long to finish right now! + +# TODO: This one just takes too long to finish right now! # Maybe we need to divide the call to ./src/test/test into a few segments, # that all end in similar amount of time? -#debian-hardened: -# image: debian:testing -# <<: *debian-template -# variables: -#HARDENING: "yes" -# script: -#- ./scripts/ci/ci-driver.sh +debian-hardened: + image: debian:testing + <<: *debian-template + variables: +HARDENING: "yes" +CC: "clang" + script: +- ./scripts/ci/ci-driver.sh debian-distcheck: image: debian:stable @@ -115,3 +121,13 @@ debian-docs: CHECK: "no" script: - ./scripts/ci/ci-driver.sh + +debian-integration: + image: debian:stable + <<: *debian-template + variables: +CHECK: "no" +CHUTNEY: "yes" +# STEM: "yes" -- currently failing on <044. + script: +- ./scripts/ci/ci-driver.sh diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ad08ee5af0..05c1415da0 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -411,10 +411,13 @@ fi if [[ "${CHUTNEY}" = "yes" ]]; then start_section "Chutney" +export CHUTNEY_TOR_SANDBOX=0 +export CHUTNEY_ALLOW_FAILURES=2 if runcmd make "${CHUTNEY_MAKE_TARGET}"; then hooray "Chutney tests have succeeded" else error "Chutney says:" +export CHUTNEY_DATA_DIR="${CHUTNEY_PATH}/net" runcmd "${CHUTNEY_PATH}"/tools/diagnostics.sh || true # These next two should be part of a make target. runcmd ls test_network_log || true ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.4.3] Merge branch 'maint-0.3.5' into maint-0.4.2
commit 02bb719a9494a010b3cad4cf81942ec7619f5806 Merge: 6ef40144ae f5b9471547 Author: Nick Mathewson Date: Wed Aug 12 12:29:17 2020 -0400 Merge branch 'maint-0.3.5' into maint-0.4.2 .gitlab-ci.yml | 36 --- Makefile.am | 2 ++ scripts/ci/ci-driver.sh | 77 + 3 files changed, 67 insertions(+), 48 deletions(-) ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
[tor-commits] [tor/maint-0.3.5] CI: improve output when skipping doxygen
commit f5b94715479e9b7bef5675ab72c8b2e260bcb830 Author: Nick Mathewson Date: Wed Aug 12 12:13:56 2020 -0400 CI: improve output when skipping doxygen --- scripts/ci/ci-driver.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/scripts/ci/ci-driver.sh b/scripts/ci/ci-driver.sh index ba4b800c04..6faffd7b40 100755 --- a/scripts/ci/ci-driver.sh +++ b/scripts/ci/ci-driver.sh @@ -65,12 +65,14 @@ STEM_PATH="${STEM_PATH:-}" if [[ "${COLOR_CI}" == "yes" ]]; then T_RED=$(tput setaf 1 || true) T_GREEN=$(tput setaf 2 || true) +T_YELLOW=$(tput setaf 3 || true) T_DIM=$(tput dim || true) T_BOLD=$(tput bold || true) T_RESET=$(tput sgr0 || true) else T_RED= T_GREEN= +T_YELLOW= T_DIM= T_BOLD= T_RESET= @@ -85,6 +87,12 @@ function die() echo "${T_BOLD}${T_RED}FATAL ERROR:${T_RESET} $*" 1>&2 exit 1 } + +function skipping() +{ +echo "${T_BOLD}${T_YELLOW}Skipping $*${T_RESET}" +} + function hooray() { echo "${T_BOLD}${T_GREEN}$*${T_RESET}" @@ -363,12 +371,16 @@ fi FAILED_TESTS="" -if [[ "${DOXYGEN}" = 'yes' && "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if [[ "${DOXYGEN}" = 'yes' ]]; then start_section Doxygen -if runcmd make doxygen; then - hooray "make doxygen has succeeded." +if [[ "${TOR_VER_AT_LEAST_043}" = 'yes' ]]; then +if runcmd make doxygen; then + hooray "make doxygen has succeeded." +else + FAILED_TESTS="${FAILED_TESTS} doxygen" +fi else - FAILED_TESTS="${FAILED_TESTS} doxygen" +skipping "make doxygen: doxygen is broken for Tor < 0.4.3" fi end_section Doxygen fi ___ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits