[pbs-devel] [PATCH proxmox v2] sys: pass create options as reference

Fabian Grünbichler f.gruenbichler at proxmox.com
Mon Jun 3 14:21:30 CEST 2024


On May 29, 2024 2:44 pm, Dietmar Maurer wrote:
>> I guess we could make a note for the next proxmox-sys bump we do for other reasons, but bumping half the world just for this seems a lot of work for little gain?
> 
> This is just a cleanup, so it is not really required. But if we delay all cleanups, cleanup will take a long time (or will never happen ...)

we don't need to delay all cleanups (and I hope we don't :)), but for
those that contain breaking changes that cause a lot of churn, we need
to decide whether the churn *now* is worth it. and most of the time,
that decision will incorporate the benefits of the change (if it is
mostly cosmetic, then the scales will tip towards "let's put it on a
list of things to do when we have unavoidable/important breaking
changes").

improving our processes so we don't forget to incorporate such small
cleanups when the next good opportunity arrives would also be good
- because we definitely don't want to end up with them never happening!

> But yes, we want to split this crate anyways, so lets wait for that.

ack. issues like this are definitely an argument for crates with limited
scope and clear boundaries ;)




More information about the pbs-devel mailing list