all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal