| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
This makes it easier to for me to test this script. (I would like to
do this to all of them but it's easier on a case-by-case basis.)
|
| | |
|
| |
|
|
|
|
|
|
| |
--version was removed in a recent update
we use that to detected whether cargo-license is installed,
now use --help instead
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
The license there is `(MIT OR Apache-2.0) AND Unicode-DFS-2016`,
which fine, but the existing shell script doesn't actually handle
"AND" correctly. This commit adds a workaround for licenses that
are "AND", and some comments about weaknesses in our (lack of)
boolean expression parsing.
This should fix CI.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
- arti#448 and arti!607 highlight an issue with upgrading `rsa`: namely,
the `x25519-dalek` version previously used has a hard dependency on
`zeroize` 1.3, which creates a dependency conflict.
- However, `x25519-dalek` version `2.0.0-pre.1` relaxes this dependency.
Reviewing the changelogs, it doesn't look like that version is
substantially different from the current one at all, so it should be
safe to use despite the "prerelease" tag.
- The new `x25519-dalek` version also bumps `rand_core`, which means we
don't have to use the RNG compat wrapper in `tor-llcrypto` as much.
closes arti#448
|
| |
|
|
| |
Previously this would eat the first filename!
|
| |
|
|
|
|
| |
Found these by disabling the nightly dbg macro special case. Now, we
have a mechanism for globally adding suppressions to tests, we can use
that instead.
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
This will allow us to have add_warning manage test lint blocks.
We have to stop printing all the filenames because there are too
many. Filenames still come out on error of course.
|
| |
|
|
| |
As per the linked discussion.
|
| |
|
|
|
| |
As per
https://gitlab.torproject.org/tpo/core/arti/-/issues/501
|
| | |
|
| | |
|
| |
|
|
| |
See arti#493.
|
| | |
|
| | |
|
| |
|
|
|
| |
Otherwise it doesn't work on sane systems where /usr/bin/python is
never an incompatible version from previously.
|
| |\
| |
| |
| |
| |
| |
| | |
Update check_licenses and add it to CI.
Closes #462
See merge request tpo/core/arti!559
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Closes #462.
Note that the license on `ring` is slightly problematic for some
users, including as it does old the openssl license[^1], with
advertising clause and all. That's not a blocker for us now, since
`ring` is not a required dependency. But we wouldn't want `ring` to
become a mandatory dependency because of this.
[^1]: To make the situation even more complicated, modern openssl
has relicensed under apache-2.0, but that doesn't necessarily
help us, since ring took its code from boringssll, which
forked from an older version of openssl.
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/557#note_2808879
|
| | | |
|
| | |
| |
| |
| | |
"python" sometimes means Python 2.
|
| | |
| |
| |
| |
| | |
"bak" ought to be a backup file, with the old contents, which this
most definitely isn't.
|
| | |
| |
| |
| |
| | |
add_warning can now add allows. This one was agreed in some other MR
for one crate. Allow it everywhere.
|
| | |
| |
| |
| |
| | |
These were in some crates, but not all. Put them in add_warning
where they will propagate everywhere.
|
| | |
| |
| |
| |
| |
| |
| |
| | |
We don't process the in-script lint block, just paste it in, now.
This is less complicated.
This reverts/replaces
"maint/add_warnings: Make it add the markers for its future self"
|
| | |
| |
| |
| | |
We'll run this in a moment. Then we'll revert it.
|
| | |
| |
| |
| |
| | |
The crates we have aren't consistent. But changing this means the
deltas resulting from the script are smaller.
|
| |/
|
|
|
|
| |
This puts the actually-used lint list at the top.
No functional change.
|
| |
|
|
|
|
| |
Utilize cargo-sort: https://github.com/DevinR528/cargo-sort
Signed-off-by: Orhun Parmaksız <[email protected]>
|
| |
|
|
| |
$SCRIPT_NAME is not defined so causes an error. Use $(basename "$0") instead.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
A build script reaching into your .git/hooks/ and modifying them
nonconsensually was a bit of a horrifying concept, and also made it hard
to build arti with the feature disabled. Remove this crate, and replace
it with manual instructions on how to install the hooks in
CONTRIBUTING.md.
|
| |
|
|
| |
see https://gitlab.torproject.org/tpo/core/arti/-/issues/400#note_2800763
|
| | |
|
| |\
| |
| |
| |
| | |
Add a comment explaining what downgrade_dependencies is for.
See merge request tpo/core/arti!480
|
| | | |
|
| | |
| |
| |
| |
| | |
nix:0.4.2 is no longer a dependency
synstructure:0.12.0 is no longer required to build
|