This is an automated email from the git hooks/post-receive script. x2go pushed a commit to branch master in repository x2goserver. commit 9aec86288f75aa8f83d661eca2156bd6c29301bf Author: Mihai Moldovan <ionic@ionic.de> Date: Sat Nov 7 02:11:53 2020 +0100 X2Go/Server/Agent/NX/Options.pm: document that duplicate keys in the input options string are possible and accepted, but actually discouraged and semantically redundant. --- X2Go/Server/Agent/NX/Options.pm | 19 +++++++++++++++++-- debian/changelog | 3 +++ 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/X2Go/Server/Agent/NX/Options.pm b/X2Go/Server/Agent/NX/Options.pm index 8c669b8d..e0062cbb 100644 --- a/X2Go/Server/Agent/NX/Options.pm +++ b/X2Go/Server/Agent/NX/Options.pm @@ -777,8 +777,8 @@ If this is not the case, please report a bug. B<X2Go/NX Agent> options strings are fully documented in L<nxagent> and additional, linked places. -This module is not concerned with the actual content of an options string, -but merely its format. +This module is not really concerned with the actual content of an options +string, but mostly its format. An options string follows the form [[I<key>[C<=>I<value>,]]C<:>I<displaynumber>. @@ -853,6 +853,21 @@ Quoting with any character is unsupported. Quotes and other grouping characters (like B<curly braces> [C<{}>]) are seen verbatim without any special meaning. +=item * + +There are no provisions (other than the mentioned invalid characters) on the +content of keys and values. + +Importantly, this also means that the same key can show up multiple times in +the options string. Semantically, this is redundant, since only the last +occurrence of a key (assuming the options string is parsed from left to right) +will take any effect. Syntactically, it's completely legal, though. + +It is recommended to avoid duplicate keys in the input options string. + +Note that, due to the nature of the supported transformations, keys can not be +duplicated with this module. + =back =head1 TRANSFORMATIONS diff --git a/debian/changelog b/debian/changelog index b2a242dd..65507baf 100644 --- a/debian/changelog +++ b/debian/changelog @@ -208,6 +208,9 @@ x2goserver (4.1.0.4-0x2go1.2) UNRELEASED; urgency=medium string" with "options string". - X2Go/Server/Agent/NX/Options.pm: replace occurrences of "option string" with "options string". + - X2Go/Server/Agent/NX/Options.pm: document that duplicate keys in the + input options string are possible and accepted, but actually discouraged + and semantically redundant. * debian/control: + Build-depend upon lsb-release for distro version detection. * debian/x2goserver.manpages: -- Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/x2goserver.git