mirrors/arti.git/maint/coverage, branch arti-v1.1.13mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.1.132023-08-08T14:32:31Zmaint: remove useless optarg2023-08-08T14:32:31ZEmil Englerme@emilengler.com2023-08-08T14:32:31Zurn:sha1:ab4d0182ab0b775b705c8c2ff7f65b08784c5942
The `maint/coverage` script has a useless option `c`.
This commit removes it.
maint: list the HTML dependencies in coverage2023-08-08T14:29:53ZEmil Englerme@emilengler.com2023-08-08T14:29:53Zurn:sha1:3c1e5e32c2e84006444a84e2cbf30b45f228e90c
Currently, the `maint/coverage` script does not inform about the
dependencies required for generating the HTML output, those are, the
Python packages `bs4` and `lxml`.
This commit fixes that, by updating the help section accordingly.
maint: remove the dependency check from coverage2023-02-24T17:32:06ZEmil Englerme@emilengler.com2023-02-22T17:35:46Zurn:sha1:20e40d79db3467d3af9037bf0d4e8044ede96cb9
This commit removes the dependency check introduced by b4b8d040, because
it's task is now done by the recently added `check_env` script.
Merge branch 'coverage-dep-check' into 'main'2023-02-22T12:00:28ZIan Jacksoniwj@torproject.org2023-02-22T12:00:28Zurn:sha1:ed5857fab1ba0d12bc111d7d67382b8013159509
maint: add a dependency check to coverage
Closes #776
See merge request tpo/core/arti!1025maint: add a dependency check to coverage2023-02-21T19:09:17ZEmil Englerme@emilengler.com2023-02-16T14:38:38Zurn:sha1:b4b8d040a024f92ac862e701ee3d7a5807832832
This commit implements a dependency check to the `maint/coverage`
script, that checks for the rustup, python3, and grcov binaries in
$PATH; the bs4 and lxml python packages; and the llvm-tools Rust
component.
Fixes #776
maint: forcefully remove files in coverage2023-02-16T14:14:08ZEmil Englerme@emilengler.com2023-02-16T14:14:08Zurn:sha1:66fd697a73c99bc117a5ece88274ef03e57cc834
When performing the initial run of `./maint/coverage`, it will report an
error message to stderr, that several files cannot be deleted, due to
their non-existence.
While the error is still ignored, I personally think that there is no
benefit in showing it either, thereby surpressing it with this commit.
Beside this, this syntax might be easier to understand from a semantical
point of view.
maint: replace llvm-tools-preview with llvm-tools2023-01-27T14:56:09ZEmil Englerme@emilengler.com2023-01-27T14:56:09Zurn:sha1:54f549dd53315aa9d25aa122208b5aa58a4ec63a
This commit replaces all occurrences of `llvm-tools-preview` with
`llvm-tools`, because it became a stable feature with Rust 1.60, thereby
making the `-preview` prefix obsolete. Beside this, `llvm-tools-preview`
is a redirection to the latter one anyway.
However, it does not work the other way around. At the moment, it is not
possible to execute the `maint/coverage` script on a NixOS system,
because the rustup component list only includes `llvm-tools`, but not
`llvm-tools-preview`, thereby making this change necessary.
shell: use /usr/bin/env instead of absolute paths2023-01-25T17:35:53ZEmil Englerme@emilengler.com2023-01-25T17:35:53Zurn:sha1:79097484de7401bf247e14c1bdf5982aee4518a1
This commit changes the shebang in all shell scripts from absolute
paths (such as `/bin/bash` or `/usr/bin/python3`) to the `/usr/bin/env`
binary with the accompanying interpreter as it's argument.
The reason for this are as follows:
- NixOS cannot work with absolute paths
- BSD systems install their packages in /usr/local/bin
Fix "coverage -h" command2022-05-26T15:12:36ZSteven Murdochs.murdoch@ucl.ac.uk2022-05-26T15:12:36Zurn:sha1:235fd786020eedca8fcbb3756b8f3cc72a8e2358
$SCRIPT_NAME is not defined so causes an error. Use $(basename "$0") instead.
make almost all scripts use bash and set euo pipefail2022-05-25T20:25:58Ztrinity-1686atrinity@deuxfleurs.fr2022-05-25T11:10:55Zurn:sha1:8eb843f2af349e4dbbd377a4c16fa57094c90414