summaryrefslogtreecommitdiff
path: root/maint
Commit message (Collapse)AuthorAgeFilesLines
...
* maint/check_license: Make MPL-2.0 into an allow-listNick Mathewson2023-05-231-1/+22
| | | | | | | | | | | | | Previously we allowed this license unconditionally. But because of its non-self-enacting nature, we need the actual notice from its "exhibit A" to appear somewhere that says that it applies to all the relevant code. Therefore, we shouldn't take new MPL-2.0 dependencies without hand-checking them. (I am tentatively allowing option-ext, though, since we already have an indirect dependency on that crate via `directories`.) For more info, see https://gitlab.torproject.org/tpo/core/arti/-/issues/845
* thanks: Also acknowledge Suggested-ByNick Mathewson2023-05-221-0/+2
|
* maint/thanks: Split up some long pipelinesNick Mathewson2023-05-181-2/+9
|
* maint/thanks: Remove email addresses from git trailersNick Mathewson2023-05-181-1/+1
| | | | | Okay, technically we're removing everything between the first `<` and the `>` at the end of the line.
* maint/thanks: Include some git trailers in acknowledgmentsNick Mathewson2023-05-181-3/+18
| | | | | | | | When building our list of acknowledgments, previously we would only include author and committer names. Now we also include anybody listed in the "Reported-by", "Co-authored-by", and "Thanks" trailers.
* fixup-features: minor doc fix.Nick Mathewson2023-05-151-5/+5
|
* fixup-features: Do not annotate non-features.Nick Mathewson2023-05-152-3/+7
|
* fixup-features: Do not add edges from non-features.Nick Mathewson2023-05-152-2/+7
|
* fixup-features: distinguish internal and external edgesNick Mathewson2023-05-152-3/+8
| | | | An external edge does not cause its target to be created as a feature.
* 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.