public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Markus Frank <m.frank@proxmox.com>
To: Markus Frank <m.frank@proxmox.com>,
	Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH qemu-sever/novnc/manager/docs v11 0/6] Feature VNC-Clipboard
Date: Fri, 8 Sep 2023 12:53:03 +0200	[thread overview]
Message-ID: <b7402f92-92d8-8000-2b94-6ce54016177a@proxmox.com> (raw)
In-Reply-To: <20230908104649.123330-1-m.frank@proxmox.com>

forgot to update novnc-pve -> ignore this v11 series

At Fri Sep 08 2023 12:46:43 GMT+0200 (Central European Summer Time), Markus Frank wrote:
> overall changes v11:
> * renamed vnc-clipboard to clipboard and changed it to a string
> 
> overall changes v9:
> * renamed vnc_clipboard to vnc-clipboard
> 
> 
> qemu-sever:
> 
> changes v10:
> * separated "vnc-clipboard return at status/current" to its own patch
> * added missing trailing comma
> * changed capitalization
> 
> changes v8:
> * removed unnecessary conditions
> * removed spicedevices variable and changed back to devices
> * created is_spice variable
> 
> changes v6:
> * added check if vga->type is set to prevent error when using default display
> * renamed clipboard to vnc_clipboard
> 
> changes v5:
> * removed return of regex check in the assertion-function for the clipboard config
> * moved clipboard code into spice-if-block
> 
> changes v4:
> * removed duplicate code and created a spicedevices variable to store
> spice/vdagent devices.
> * clipboard_check_compatibility function
> 
> changes v3:
> * added hint to make clearer that the spice guest tools are required for
>   the noVNC-clipboard
> * Checkbox changes to ComboBox when a spice device is selected to make
>   clear that only one clipboard can be used at a time.
> * added 2 test-cases
> 
> changes v2:
> * added pci address to virtio-serial-pci
> 
> Markus Frank (3):
>    enable VNC clipboard parameter in vga_fmt
>    add vnc-clipboard variable to return at status/current
>    test cases for clipboard spice & std
> 
>   PVE/API2/Qemu.pm                          | 15 ++++++
>   PVE/QemuServer.pm                         | 66 ++++++++++++++++-------
>   test/cfg2cmd/VNC-clipboard-spice.conf     |  1 +
>   test/cfg2cmd/VNC-clipboard-spice.conf.cmd | 27 ++++++++++
>   test/cfg2cmd/VNC-clipboard-std.conf       |  1 +
>   test/cfg2cmd/VNC-clipboard-std.conf.cmd   | 27 ++++++++++
>   6 files changed, 119 insertions(+), 18 deletions(-)
>   create mode 100644 test/cfg2cmd/VNC-clipboard-spice.conf
>   create mode 100644 test/cfg2cmd/VNC-clipboard-spice.conf.cmd
>   create mode 100644 test/cfg2cmd/VNC-clipboard-std.conf
>   create mode 100644 test/cfg2cmd/VNC-clipboard-std.conf.cmd
> 
> 
> novnc:
> 
> Markus Frank (1):
>    add "show clipboard button" patch to series
> 
>   .../patches/0019-show-clipboard-button.patch  | 30 +++++++++++++++++++
>   debian/patches/series                         |  1 +
>   2 files changed, 31 insertions(+)
>   create mode 100644 debian/patches/0019-show-clipboard-button.patch
> 
> 
> manager:
> 
> changes v11:
> * added hint that SPICE & VNC Clipboard won't work at the same time.
> 
> changes v7:
> * fixed the empty Display Bug in VGA options found by Friedrich
>   (it should be "Default" instead of empty)
> 
> changes v6:
> * Changed onGetValues in DisplayEdit to only set the saved vnc_clipboard
> variable, so it's possible to reset type and memory to default values
> 
> changes v4:
> * moved clipboard option from Display settings in Hardware to Options
> 
> Markus Frank (1):
>    add clipboard checkbox to VM Options
> 
>   www/manager6/qemu/DisplayEdit.js |  8 +++++
>   www/manager6/qemu/Options.js     | 52 ++++++++++++++++++++++++++++++++
>   2 files changed, 60 insertions(+)
> 
> 
> docs:
> 
> changes v4:
> * rewrote entire Text
> * added command example to set clipboard to 1
> 
> Markus Frank (1):
>    add VNC clipboard documentation
> 
>   qm.adoc | 18 ++++++++++++++++++
>   1 file changed, 18 insertions(+)
> 




      parent reply	other threads:[~2023-09-08 10:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-08 10:46 Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH qemu-sever v11 1/6] enable VNC clipboard parameter in vga_fmt Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH qemu-sever v11 2/6] add clipboard variable to return at status/current Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH qemu-sever v11 3/6] test cases for clipboard spice & std Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH novnc v11 4/6] add "show clipboard button" patch to series Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH manager v11 5/6] add clipboard checkbox to VM Options Markus Frank
2023-09-08 10:46 ` [pve-devel] [PATCH docs v11 6/6] add VNC clipboard documentation Markus Frank
2023-09-08 10:53 ` Markus Frank [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b7402f92-92d8-8000-2b94-6ce54016177a@proxmox.com \
    --to=m.frank@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal