| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
I tried to use petgraph, but it was optimized for performance over
usability, and the usability was beyond me.
|
| | |
|
| |
|
|
|
|
|
|
|
| |
The problem with our old rules is that "reachable from __nonadditive"
and "reachable from experimental" were not themselves sensible
definitions of nonadditive and experimental.
See
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1068#note_2887939
|
| | |
|
| |
|
|
|
|
|
|
| |
The generational-arena crate is distributed under MPL-2.0,
so we need to allow it.
I believe that this license is fine and does not interfere with
our code or our users; the reviewer should double-check.
|
| | |
|
| |
|
|
| |
Now that we require 1.65, Rust 1.63 is no longer sufficient.
|
| | |
|
| |
|
|
| |
(We don't include Tor employees in our "thanks" section.)
|
| | |
|
| | |
|
| |
|
|
| |
In each case, use a circumlocution.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
| |
See top-level main.rs comment for an explanation.
Closes #771
|
| |\
| |
| |
| |
| | |
maint: add a full dependency check
See merge request tpo/core/arti!1030
|
| | |
| |
| |
| |
| | |
This commit removes the dependency check introduced by b4b8d040, because
it's task is now done by the recently added `check_env` script.
|
| | |
| |
| |
| |
| |
| |
| | |
This commit adds a script that checks the environment for it's suitable
as an Arti development environment.
It follows an idea pitched by Ian Jackson and me in !1025.
|
| |\ \
| | |
| | |
| | |
| | | |
Approve zlib licence
See merge request tpo/core/arti!1035
|
| | | |
| | |
| | |
| | |
| | | |
There's nothing wrong with the previous idiom since it's a trusted
variable, but whatever.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is a very permissive BSD/MIT-like licence. I have reviewed the
licence text and it is inoffensive.
I want this for slotmap, in !1034
|
| | | |
| | |
| | |
| | | |
Even if it contains spaces.
|
| | | |
| | |
| | |
| | | |
This confused me while I was debugging the interaction with nailing-cargo.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
maint: fix `adhoc-add-lint-blocks` script
See merge request tpo/core/arti!1031
|
| | |/
| |
| |
| |
| |
| | |
This commit fixes a bug in maint/adhoc-add-lint-blocks, which made it
impossible to execute, because `/usr/bin/env` looked at Perl's command
line arguments to be a part of the binary name.
|
| |/
|
|
|
| |
The maint/ directory contains several python scripts, which will create
a __pycache__ directory.
|
| |\
| |
| |
| |
| |
| |
| | |
maint: add a dependency check to coverage
Closes #776
See merge request tpo/core/arti!1025
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
| |
| |
| |
| |
| | |
In Python, str.find returns -1 if it doesn't find the target
character, and -1 is a true value. So `if pos := foo.find('c')`
won't do what we want.
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
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.
|
| | |
|
| |
|
|
|
|
|
|
| |
This commit adds a test called `shebang`, which checks if all shebangs
in scripts use relative paths through `#/usr/bin/env`, rather than
absolute paths, such as `#/bin/bash`.
See 833b10575bbb18200ae9e6ba4f0a1dd858417020.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
| |
This panics on error, and we're fine with a panic on misbehavior in
tests.
|
| |
|
|
|
|
|
|
| |
This warning kind of snuck up on us! (See #748) For now, let's
disable it. (I've cleaned it up in a couple of examples, since
those are meant to be more idiomatic and user-facing.)
Closes #748.
|
| |
|
|
|
| |
This commit updates the shellcheck_all script, in order to look for
/usr/bin/env, instead of /bin.
|
| |
|
|
|
|
|
|
|
|
| |
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
|
| | |
|
| |\
| |
| |
| |
| | |
Test vanilla bridge in shadow
See merge request tpo/core/arti!915
|
| | | |
|
| |/ |
|