mirrors/arti.git/crates/tor-circmgr/src, branch arti-v0.4.0mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v0.4.02022-05-23T16:55:37ZResolve the new `derive_partial_eq_without_eq` lint.2022-05-23T16:55:37ZNick Mathewsonnickm@torproject.org2022-05-23T16:32:31Zurn:sha1:546ae3000eb179aedc97de30b0ccdb1d45b93395
It's a little overzealous sometimes, but it's mostly to the good.
impl_standard_builder: Have it generate FooConfig::builder2022-05-12T17:50:26ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-12T17:15:39Zurn:sha1:be5bc04c02d5023c46a117a048bf0a659094f4eb
This deletes many handcoded impls. It also generates lots of impls
that we previously didn't have.
Merge branch 'builder-default-bis' into 'main'2022-05-12T15:57:56Zetator@eta.st2022-05-12T15:57:56Zurn:sha1:2255778afa664ff059785d4d1f6790227a02d3b7
impl_standard_builder followup
See merge request tpo/core/arti!505Rename impl_standard_builder from impl_default_via_builder2022-05-12T14:59:13ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-12T14:58:58Zurn:sha1:c1c6f2b376827c4ebb13bc08456f8355717d15e5
I have Plans for this macro. In particular:
* I have a wip branch which tests that the Builder can be
deserialised from an empty config (ie, that config reading
of a config with a blank section for this item works).
* I think we should autogenerate $Config::builder(),
and promote that, rather than $ConfigBuilder::default().
This macro could do that.
config: Replace more handwritten impl Default2022-05-12T14:59:10ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-12T14:35:44Zurn:sha1:888d6e0511d01379d42ae154ac99ce2d9e84fd44Merge branch 'ticket_412_467' into 'main'2022-05-12T14:42:51ZNick Mathewsonnickm@torproject.org2022-05-12T14:42:51Zurn:sha1:2a5ee2c8c9ceb0141db842d944f619337e366bf1
Teach DirMgr to use slightly untimely directories
Closes #467 and #412
See merge request tpo/core/arti!500Add correct serde(default) attrs for humantime_serde::option2022-05-12T10:34:53ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-12T09:09:46Zurn:sha1:04b8729d6baf411847ec89d29b97848781a96dbb
Discovered by a test case in my local tree. The test case was
macro-generated by an extension of impl_standard_builder (which
macro istself currently awaiting review, arti!499)
Have also sent an MR to update the upstream docs
https://github.com/jean-airoldie/humantime-serde/pull/8
Note a TODO in exitpath construction.2022-05-11T16:42:38ZNick Mathewsonnickm@torproject.org2022-05-11T16:42:38Zurn:sha1:dd20ac45abc2b231c85914fd115728f64233515cconfig derive attrs: Make builders serde, and validated structs not2022-05-05T09:35:52ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-04T16:45:25Zurn:sha1:d47e94b45993162fece0d1bdfa4d3da0407a4861
* Builders additionally derive: Debug, Serialize, Deserialize.
* Validated structs no longer derive: Serialize, Deserialize
and all related attributes deleted.
* As a consequence, all the `#[serde(deny_unknown_fields)]`
are gone. That means that right now unknown fields are totally
ignored. This is good for compatibility but poor for useability.
Doing something better here is arti#417, in progress.
* As a consequence, delete tor_dirmgr::retry::default_parallelism.
(The default value was already duplicated into a builder attr.)
Merge branch 'socket-addr-list-builder' into 'main'2022-05-04T18:13:45ZIan Jacksoniwj@torproject.org2022-05-04T18:13:45Zurn:sha1:a334f17262e80620cfffa3c2ec5f50d4ea49a22f
FallbackDir: orports: Introduce and use VecBuilder
See merge request tpo/core/arti!474