* [pve-devel] [PATCH ha-manager] ha-simulator: add xauth dependency
@ 2021-02-01 10:18 Aaron Lauterer
2021-02-01 10:46 ` Thomas Lamprecht
0 siblings, 1 reply; 2+ messages in thread
From: Aaron Lauterer @ 2021-02-01 10:18 UTC (permalink / raw)
To: pve-devel
The xauth package is needed for the example in the docs to access the
ha-simulator via ssh with x11 forwarding.
On systems with a desktop environment the xauth package is usually
already installed, but on a clean PVE installation it is missing.
Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
---
I am not sure if there is more needed from my side for this patch. If
so, let me know.
debian/control | 1 +
1 file changed, 1 insertion(+)
diff --git a/debian/control b/debian/control
index 7b4ffb0..62eb4e9 100644
--- a/debian/control
+++ b/debian/control
@@ -46,6 +46,7 @@ Depends: libclone-perl,
libnet-ip-perl,
libstring-shellquote-perl,
liburi-perl,
+ xauth,
${misc:Depends},
${perl:Depends},
Description: Proxmox VE HA Simulator
--
2.20.1
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [pve-devel] [PATCH ha-manager] ha-simulator: add xauth dependency
2021-02-01 10:18 [pve-devel] [PATCH ha-manager] ha-simulator: add xauth dependency Aaron Lauterer
@ 2021-02-01 10:46 ` Thomas Lamprecht
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2021-02-01 10:46 UTC (permalink / raw)
To: Proxmox VE development discussion, Aaron Lauterer
On 01.02.21 11:18, Aaron Lauterer wrote:
> The xauth package is needed for the example in the docs to access the
> ha-simulator via ssh with x11 forwarding.
then the client needs it, and the client does not installs the ha-sim
if it's used over ssh anyway?
Even if required for the server side in that case it'd be only a Suggests
or at max a Recommends, but no hard-dependency (could especially become
a problem with some pure wayland setups in the future)
>
> On systems with a desktop environment the xauth package is usually
> already installed, but on a clean PVE installation it is missing.
>
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
>
> I am not sure if there is more needed from my side for this patch. If
> so, let me know.
>
> debian/control | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/debian/control b/debian/control
> index 7b4ffb0..62eb4e9 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -46,6 +46,7 @@ Depends: libclone-perl,
> libnet-ip-perl,
> libstring-shellquote-perl,
> liburi-perl,
> + xauth,
> ${misc:Depends},
> ${perl:Depends},
> Description: Proxmox VE HA Simulator
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-02-01 10:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-01 10:18 [pve-devel] [PATCH ha-manager] ha-simulator: add xauth dependency Aaron Lauterer
2021-02-01 10:46 ` Thomas Lamprecht
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox