[pve-devel] applied-series: [PATCH 00/12] installer: add crate for common code
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Nov 2 20:05:41 CET 2023
On 25/10/2023 17:59, Aaron Lauterer wrote:
> since work on the auto installer is happenning in parallel, now would be
> a good point to move commonly used code into its own crate. Otherwise
> the auto-installer will always have to play catch up with the ongoing
> development of the tui installer.
>
> I tried to split up the commits as much as possible, but there are two
> larger ones, copying most the code over to the new repo and making the
> switch. The former because it is difficult to pull apart the parts that
> belong together. The latter needed to be this big as most local
> occurences needed to be removed at the same time to avoid dependency
> conflicts.
>
> One last things that is missing, is the "InstallConfig" struct.
> This should also be part of the common crate, but I need to look further
> into how to make it possible that it can be created from structs of each
> crate (tui, auto) as implementing a ::From can only be done within the
> crate where the struct lives IIUC.
>
> This series depends on the patches by Christoph to remove the global
> unsafe setup info, version 2 [0]. Without those patches applied first,
> this series will not apply.
>
> [0] https://lists.proxmox.com/pipermail/pve-devel/2023-October/059628.html
>
> Aaron Lauterer (12):
> add proxmox-installer-common crate
> common: copy common code from tui-installer
> common: utils: add dependency for doc test
> common: make InstallZfsOption public
> common: disk_checks: make functions public
> tui-installer: add dependency for new common crate
> tui: switch to common crate
> tui: remove now unused utils.rs
> common: add installer_setup method
> common: document installer_setup method
> tui: use installer_setup from common cate
> tui: remove unused read_json function
applied series, with Christoph's build-sys fix squashed into your first
patch, and his R-b added, thanks!
More information about the pve-devel
mailing list