Re: [pve-devel] [PATCH v1 installer 07/18] auto-installer: add dependencies

From: Christoph Heiss <c.heiss@proxmox.com>
To: Aaron Lauterer <a.lauterer@proxmox.com>
Cc: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH v1 installer 07/18] auto-installer: add dependencies
Date: Wed, 31 Jan 2024 14:52:30 +0100	[thread overview]
Message-ID: <pafce33uknghc6izryczkwo7m34rprjgsv6o5vesqs3vtspeof@kxse3lowyj7e> (raw)
In-Reply-To: <20240123170053.490250-8-a.lauterer@proxmox.com>

On Tue, Jan 23, 2024 at 06:00:42PM +0100, Aaron Lauterer wrote:
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
>  proxmox-auto-installer/Cargo.toml | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/proxmox-auto-installer/Cargo.toml b/proxmox-auto-installer/Cargo.toml
> index 75cfb2c..211c605 100644
> --- a/proxmox-auto-installer/Cargo.toml
> +++ b/proxmox-auto-installer/Cargo.toml
> @@ -8,3 +8,7 @@ exclude = [ "build", "debian" ]
>  homepage = "https://www.proxmox.com"
>
>  [dependencies]
> +proxmox-installer-common = { path = "../proxmox-installer-common" }
> +serde = { version = "1.0", features = ["derive"] }
> +serde_json = "1.0"
> +toml = "0.5.11"
This should get an appropriate `Build-Depends` entry in debian/control
too, IMO. Currently, `librust-toml-0.5-dev` is pulled in transitively
anyway.

Same for the `clap` crate later in the series.

> --
> 2.39.2
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>