[X2Go-Commits] [x2goserver] 60/99: {X2Go/Server/Agent/NX/Options.pm, x2goserver/lib/x2goupdateoptionsstring}: update Perl critic overrides to allow the constant pragma and document it and other overrides (unless they are self-explanatory).
git-admin at x2go.org
git-admin at x2go.org
Mon Dec 28 06:10:48 CET 2020
This is an automated email from the git hooks/post-receive script.
x2go pushed a commit to branch master
in repository x2goserver.
commit c5e3ee2a64251a49196589cf5b60e4b0154d8b7e
Author: Mihai Moldovan <ionic at ionic.de>
Date: Sun Nov 22 22:13:48 2020 +0100
{X2Go/Server/Agent/NX/Options.pm,x2goserver/lib/x2goupdateoptionsstring}: update Perl critic overrides to allow the constant pragma and document it and other overrides (unless they are self-explanatory).
---
X2Go/Server/Agent/NX/Options.pm | 6 ++++++
debian/changelog | 4 ++++
x2goserver/bin/x2goupdateoptionsstring | 6 ++++++
3 files changed, 16 insertions(+)
diff --git a/X2Go/Server/Agent/NX/Options.pm b/X2Go/Server/Agent/NX/Options.pm
index 79bb4d44..83bfb70b 100644
--- a/X2Go/Server/Agent/NX/Options.pm
+++ b/X2Go/Server/Agent/NX/Options.pm
@@ -18,7 +18,13 @@
package X2Go::Server::Agent::NX::Options;
# Disable some Perl::Critic violations.
+#
+# I like parentheses.
## no critic (ProhibitParensWithBuiltins)
+#
+# It's recommended not to use the constant pragma, but we don't want to use
+# non-core modules like Readonly.
+## no critic (ValuesAndExpressions::ProhibitConstantPragma)
use strict;
use warnings;
diff --git a/debian/changelog b/debian/changelog
index b04c120c..0cb1421a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -305,6 +305,10 @@ x2goserver (4.1.0.4-0x2go1.2) UNRELEASED; urgency=medium
functionality.
- X2Go/Server/Agent/NX/Options.pm: port modes "enum" subs to the constant
pragma to make the code more consistent with the program wrapper.
+ - {X2Go/Server/Agent/NX/Options.pm,
+ x2goserver/lib/x2goupdateoptionsstring}: update Perl critic overrides
+ to allow the constant pragma and document it and other overrides
+ (unless they are self-explanatory).
* debian/control:
+ Build-depend upon lsb-release for distro version detection.
* debian/x2goserver.manpages:
diff --git a/x2goserver/bin/x2goupdateoptionsstring b/x2goserver/bin/x2goupdateoptionsstring
index 127bdc7d..890b541a 100755
--- a/x2goserver/bin/x2goupdateoptionsstring
+++ b/x2goserver/bin/x2goupdateoptionsstring
@@ -18,7 +18,13 @@
# 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA.
# Disable some Perl::Critic violations.
+#
+# I like parentheses.
## no critic (ProhibitParensWithBuiltins)
+#
+# It's recommended not to use the constant pragma, but we don't want to use
+# non-core modules like Readonly.
+## no critic (ValuesAndExpressions::ProhibitConstantPragma)
use strict;
use warnings;
--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/x2goserver.git
More information about the x2go-commits
mailing list