summaryrefslogtreecommitdiff
path: root/maint
Commit message (Collapse)AuthorAgeFilesLines
...
* fixup-features: Add an option to not annotate.Nick Mathewson2023-05-153-6/+25
|
* fixup-features: ability to add annotations for everything.Nick Mathewson2023-05-151-7/+13
|
* fixup-features: fix off-by-one in argument reading.Nick Mathewson2023-05-151-1/+1
|
* fixup-features: Implement remaining rules.Nick Mathewson2023-05-151-34/+64
|
* fixup-features: Enforce __is_experimental tagging rule.Nick Mathewson2023-05-151-0/+19
|
* fixup-features: Revise our rule 2 enforcement to use newer APIs.Nick Mathewson2023-05-151-14/+14
|
* fixup-features: Refactor "apply a list of changes" code into a new module.Nick Mathewson2023-05-152-25/+88
|
* fixup-features: Make a feature graph type in a submoduleNick Mathewson2023-05-153-60/+149
| | | | | I tried to use petgraph, but it was optimized for performance over usability, and the usability was beyond me.
* fixup-features: minor spelling and comment fixes.Nick Mathewson2023-05-151-1/+1
|
* fixup-features: Describe the semantics we actually wantNick Mathewson2023-05-151-6/+26
| | | | | | | | | 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
* allow rustls-webpki licensetrinity-1686a2023-05-081-0/+1
|
* check-licenses: Permit MPL-2.0Nick Mathewson2023-05-041-0/+1
| | | | | | | | 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.
* fix macos reproducible buildstrinity-1686a2023-04-161-1/+1
|
* Bump Rust version in reproducible build scriptsNick Mathewson2023-04-131-1/+1
| | | | Now that we require 1.65, Rust 1.63 is no longer sufficient.
* Increment MSRV to 1.65 in every crate.Nick Mathewson2023-04-111-1/+1
|
* Add Gabi to exclude_contributors.Nick Mathewson2023-03-311-0/+3
| | | | (We don't include Tor employees in our "thanks" section.)
* Fix typosDimitris Apostolou2023-03-241-1/+1
|
* maint/check_todos: Offer advice on workarounds, if trippedIan Jackson2023-03-211-0/+22
|
* Obscure various literal uses of the todo markerIan Jackson2023-03-212-2/+2
| | | | In each case, use a circumlocution.
* maint/check_todos: New scriptIan Jackson2023-03-211-0/+31
|
* fixup-features: Use write-and-rename pattern.Nick Mathewson2023-03-201-1/+3
|
* fixup-features: Improve a comment.Nick Mathewson2023-03-201-1/+1
|
* fixup-features: Simplify "optionality" detection.Nick Mathewson2023-03-201-17/+13
|
* New "fixup-features" tool to enforce properties in Cargo.tomlsNick Mathewson2023-03-203-0/+346
| | | | | | See top-level main.rs comment for an explanation. Closes #771
* Merge branch 'check_env' into 'main'Ian Jackson2023-03-012-23/+122
|\ | | | | | | | | maint: add a full dependency check See merge request tpo/core/arti!1030
| * maint: remove the dependency check from coverageEmil Engler2023-02-241-23/+0
| | | | | | | | | | This commit removes the dependency check introduced by b4b8d040, because it's task is now done by the recently added `check_env` script.
| * maint: add a full dependency checkEmil Engler2023-02-241-0/+122
| | | | | | | | | | | | | | 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.
* | Merge branch 'zlib' into 'main'Ian Jackson2023-02-271-2/+9
|\ \ | | | | | | | | | | | | Approve zlib licence See merge request tpo/core/arti!1035
| * | maint/check_licensse: Placate shellcheckIan Jackson2023-02-241-1/+1
| | | | | | | | | | | | | | | There's nothing wrong with the previous idiom since it's a trusted variable, but whatever.
| * | maint/check_licensse: Approve Zlib licenceIan Jackson2023-02-241-0/+1
| | | | | | | | | | | | | | | | | | | | | 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
| * | maint/check_licensse: Honour CARGO variableIan Jackson2023-02-241-2/+8
| | | | | | | | | | | | Even if it contains spaces.
| * | maint/check_licensse: Do not discard actual errorIan Jackson2023-02-241-1/+1
| | | | | | | | | | | | This confused me while I was debugging the interaction with nailing-cargo.
* | | Merge branch 'fix-adhoc' into 'main'Ian Jackson2023-02-241-1/+1
|\ \ \ | |/ / |/| | | | | | | | maint: fix `adhoc-add-lint-blocks` script See merge request tpo/core/arti!1031
| * | maint: fix `adhoc-add-lint-blocks` scriptEmil Engler2023-02-221-1/+1
| |/ | | | | | | | | | | 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.
* / git: add maint/.gitignore for __pycache__Emil Engler2023-02-221-0/+1
|/ | | | | The maint/ directory contains several python scripts, which will create a __pycache__ directory.
* Merge branch 'coverage-dep-check' into 'main'Ian Jackson2023-02-221-0/+23
|\ | | | | | | | | | | | | maint: add a dependency check to coverage Closes #776 See merge request tpo/core/arti!1025
| * maint: add a dependency check to coverageEmil Engler2023-02-211-0/+23
| | | | | | | | | | | | | | | | | | 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
* | Fix a bug in check_doc_featuresNick Mathewson2023-02-211-7/+3
| | | | | | | | | | | | 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.
* | maint: forcefully remove files in coverageEmil Engler2023-02-161-1/+1
|/ | | | | | | | | | | | 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.
* tor-cell: documentation cleanups related to restricted_msgNick Mathewson2023-02-071-0/+5
|
* maint: add shebangEmil Engler2023-01-261-0/+10
| | | | | | | | 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.
* maint: replace llvm-tools-preview with llvm-toolsEmil Engler2023-01-272-5/+5
| | | | | | | | | | | | 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.
* Allow clippy::unchecked_duration_subtraction in testsNick Mathewson2023-01-271-0/+1
| | | | | This panics on error, and we're fine with a panic on misbehavior in tests.
* Disable clippy::unlinlined-format-argsNick Mathewson2023-01-271-0/+1
| | | | | | | | 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.
* maint: update shellcheck_all with /usr/bin/envEmil Engler2023-01-251-1/+1
| | | | | This commit updates the shellcheck_all script, in order to look for /usr/bin/env, instead of /bin.
* shell: use /usr/bin/env instead of absolute pathsEmil Engler2023-01-2533-33/+33
| | | | | | | | | | 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 lint blocks: Add ad hoc script for adding themIan Jackson2022-12-121-0/+26
|
* Merge branch 'shadow-bridge' into 'main'Ian Jackson2022-12-121-0/+13
|\ | | | | | | | | Test vanilla bridge in shadow See merge request tpo/core/arti!915
| * put ci log span magic strings in a dedicated scripttrinity-1686a2022-12-051-0/+13
| |
* | Fix typosDimitris Apostolou2022-12-091-2/+2
|/