[pve-devel] [PATCH installer 0/3] tui: remove global, unsafe setup info

Christoph Heiss c.heiss at proxmox.com
Wed Oct 4 18:00:54 CEST 2023


Removes the `static mut` for holding a `SetupInfo` instance.

This is done by either passing the needed info as parameter, or in some
cases, the needed information is already available through other means.

Not only does it get rid of some ugly, unsafe code, it is needed anyway
as a prerequisite by Aaron for pulling out non-TUI-related code into a
separate, shared crate.

No functional changes overall.

Christoph Heiss (3):
  tui: refactor `NetworkOptions` to have a `defaults_from()` function
  tui: bootdisk: pass down product info to advanced dialog
  tui: remove obsolete, global `SetupInfo` state

 proxmox-tui-installer/src/main.rs           | 34 +++-------
 proxmox-tui-installer/src/options.rs        | 36 ++++------
 proxmox-tui-installer/src/setup.rs          | 10 ++-
 proxmox-tui-installer/src/views/bootdisk.rs | 74 ++++++++++++++-------
 4 files changed, 74 insertions(+), 80 deletions(-)

--
2.41.0






More information about the pve-devel mailing list