aboutsummaryrefslogtreecommitdiff
path: root/maint
Commit message (Collapse)AuthorAgeFilesLines
...
* lints: Scheme for allowing us to name nightly lintsIan Jackson2022-06-231-1/+3
| | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/issues/501
* Avoid matching crate names by prefix.Nick Mathewson2022-06-171-1/+5
|
* Update check_tree to better detect errors in cargo-tree.Nick Mathewson2022-06-161-6/+26
|
* Add script and CI to make sure `ring` doesn't show up in arti/fullNick Mathewson2022-06-161-0/+24
| | | | See arti#493.
* CI: Break out maint/via-cargo-installIan Jackson2022-06-081-0/+10
|
* maint/cargo_sort: Tolerate toplevel Cargo.toml not being sortedIan Jackson2022-06-081-2/+25
|
* maint/toposort: Make shebang be python3, not pythonIan Jackson2022-06-081-1/+1
| | | | | Otherwise it doesn't work on sane systems where /usr/bin/python is never an incompatible version from previously.
* Merge branch 'check-license-fixes' into 'main'Ian Jackson2022-06-021-1/+17
|\ | | | | | | | | | | | | Update check_licenses and add it to CI. Closes #462 See merge request tpo/core/arti!559
| * check_licenses: list new crates with manual license verificationNick Mathewson2022-05-311-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | maint/add_warning: Convert remaining assertsIan Jackson2022-05-311-4/+8
| |
* | maint/add_warning: Introduce ImproperFile and convert one assertIan Jackson2022-05-311-2/+18
| |
* | maint/add_warning: Use python3 argparseIan Jackson2022-05-311-11/+9
| |
* | maint/add_warning: Add "how to fix" to error messageIan Jackson2022-05-311-0/+1
| | | | | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/557#note_2808879
* | maint/add_warning: Provide --check modeIan Jackson2022-05-311-2/+22
| |
* | maint/add_warning: Change shebang to python3Ian Jackson2022-05-311-1/+1
| | | | | | | | "python" sometimes means Python 2.
* | maint/add_warning: Use .tmp~ for temp filename, not .bakIan Jackson2022-05-311-3/+3
| | | | | | | | | | "bak" ought to be a backup file, with the old contents, which this most definitely isn't.
* | lints: Add let_unit_value allow to add_warningIan Jackson2022-05-311-0/+1
| | | | | | | | | | add_warning can now add allows. This one was agreed in some other MR for one crate. Allow it everywhere.
* | lints: Add two missing ones to add_warningIan Jackson2022-05-311-0/+2
| | | | | | | | | | These were in some crates, but not all. Put them in add_warning where they will propagate everywhere.
* | lints: maint/add_warning: New marker-based modeIan Jackson2022-05-311-35/+21
| | | | | | | | | | | | | | | | 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"
* | maint/add_warnings: Make it add the markers for its future selfIan Jackson2022-05-311-0/+6
| | | | | | | | We'll run this in a moment. Then we'll revert it.
* | lints: maint/add_warning: Improve orderingIan Jackson2022-05-311-1/+1
| | | | | | | | | | The crates we have aren't consistent. But changing this means the deltas resulting from the script are smaller.
* | maint/add_warnings: Tidy up, and add some commentaryIan Jackson2022-05-311-2/+7
|/ | | | | | This puts the actually-used lint list at the top. No functional change.
* Lexically sort Cargo.toml dependenciesOrhun Parmaksız2022-05-281-0/+7
| | | | | | Utilize cargo-sort: https://github.com/DevinR528/cargo-sort Signed-off-by: Orhun Parmaksız <[email protected]>
* Fix "coverage -h" commandSteven Murdoch2022-05-261-1/+1
| | | | $SCRIPT_NAME is not defined so causes an error. Use $(basename "$0") instead.
* fix docker_reproducible_build not passing its parameters properlytrinity-1686a2022-05-261-1/+1
|
* update comments in coverage_fuzz_corporatrinity-1686a2022-05-251-25/+15
|
* make almost all scripts use bash and set euo pipefailtrinity-1686a2022-05-2514-23/+33
|
* Remove cargo-husky, and replace with manual instructionseta2022-05-103-1/+28
| | | | | | | | 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.
* use true instead of /bin/truetrinity-1686a2022-05-051-2/+2
| | | | see https://gitlab.torproject.org/tpo/core/arti/-/issues/400#note_2800763
* generate aggregated coverage reportstrinity-1686a2022-05-051-13/+19
|
* Merge branch 'explain_downgrade' into 'main'eta2022-05-031-0/+6
|\ | | | | | | | | Add a comment explaining what downgrade_dependencies is for. See merge request tpo/core/arti!480
| * Add a comment explaining what downgrade_dependencies is for.Nick Mathewson2022-04-281-0/+6
| |
* | remove some unecessary crates from downgrade_dependenciestrinity-1686a2022-04-281-3/+1
| | | | | | | | | | nix:0.4.2 is no longer a dependency synstructure:0.12.0 is no longer required to build
* | update tls-api to 0.8.0trinity-1686a2022-04-281-4/+4
| | | | | | | | and remove corresponding RUSTSEC from ignored list
* | downgrade_dependencies: Remove "quote" exception.Nick Mathewson2022-04-261-1/+0
|/ | | | This is no longer necessary. :)
* Use git source for derive_builder for now, for sub_builder featureIan Jackson2022-04-211-1/+0
| | | | | | | | | | | | | | | This commitid is the current head of my MR branch https://github.com/colin-kiegel/rust-derive-builder/pull/253 https://github.com/ijackson/rust-derive-builder/tree/field-builder Using the commitid prevents surprises if that branch is updated. We will require this newer version of derive_builder. The version will need to be bumped again later, assuming the upstream MR is merged and upstream do a release containing the needed changes. We will need the new version of not only `derive_builder_core` (the main macro implementation) but also`derive_builder` for a new error type.
* Fix typosDimitris Apostolou2022-04-111-3/+3
|
* Merge branch 'cargo-audit' into 'main'Nick Mathewson2022-04-081-0/+11
|\ | | | | | | | | fix maint/cargo_audit not propagating error and add temporary exceptions See merge request tpo/core/arti!452
| * fix maint/cargo_audit not propagating error and add temporary exceptionstrinity-1686a2022-04-081-0/+11
| |
* | use default toolchain from rustup instead of stable for coveragetrinity-1686a2022-04-081-1/+2
| | | | | | | | using stable doesn't work for CI because a specific version like 1.60 is installed
* | bring back RUST_COVERAGE_TOOLCHAINtrinity-1686a2022-04-081-2/+8
| |
* | use rustc 1.60 (stable) for coveragetrinity-1686a2022-04-072-12/+5
|/
* fix with_coverage script to exclude pub(crate) mod test from resultstrinity-1686a2022-03-241-1/+1
|
* downgrade_dependencies: Upgrade darlingIan Jackson2022-03-161-0/+1
| | | | | | | | | The new version of derive_builder requires a newer darling. Ooops. Upstream MR here: https://github.com/colin-kiegel/rust-derive-builder/pull/242 For now, upgrade darling. When there's a new upstream derive_builder, we can use it and drop this.
* tests/chutney: Merge chutney-launching scriptsNick Mathewson2022-03-101-1/+1
| | | | | | | | | Formerly, the "launch a chutney network" code was duplicated in "setup" and "arti-bench", since "setup" always launched an arti proxy, while "arti-bench" didn't want that functionality. Now the "setup" script launches an arti proxy conditionally, depending on whether the "proxy" argument is given.
* Omit arti-testing from coverage.Nick Mathewson2022-03-041-0/+1
|
* fix reproducible build summary giving invalid branch and commit idtrinity-1686a2022-03-041-0/+1
| | | | | fix #378 also fix unrelated error in nightly rustdoc CI
* Reproducible build: Bump to Rust 1.59 and Alpine 3.15Nick Mathewson2022-03-041-1/+4
| | | | | | | | | | | | | | The Rust upgrade is necessary since our Cargo.lock file now requires `ed25519` 1.4.0, which requires edition2021, which requires Rust 1.56 or later. The Alpine upgrade is opportunistic: we might as well. I've also added comments to remind us to keep the .gitlab-ci.yml and docker_reproducible_build files in sync, since my first version of this commit messed that up. Closes #376.
* add_warning: remove clone_on_ref_ptrNick Mathewson2022-03-011-1/+0
| | | | See !352
* Add nix exception to downgrade_dependencies.Nick Mathewson2022-02-281-1/+2
|
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-26 20:00:47 +0000