[PATCH perl-rs/proxmox v2 0/3] perl: enable strict and warnings in modules that lack them

[Lukas Sichert <l.sichert@proxmox.com>]

A scan of the perl modules in these repositories found a few that
enable neither 'use v5.36' nor 'use strict' and 'use warnings'. Enable
them to adhere to the style guide and so that mistakes such as typos
in variable names or uses of undefined values do not go unnoticed.

The Fixup.pm files in pve-rs and pmg-rs are intentionally left alone.
They are not modules of their own, but snippets that perlmod's
genpackage.pl inserts into the generated Proxmox::Lib::PVE and
Proxmox::Lib::PMG right after the 'use strict; use warnings;' of its
template, so they already run under both pragmas.

Testing:
- proxmox: running 'make refresh' in pve-api-types before and after the
  change generates identical output and identical stderr.
- perl-rs: syntax check and including the common file with perl -e
  returned no warnings

Changes v1 -> v2:
- removed the pve-network patch as it is already applied
- deleted the CalendarEvent files instead of enabling strict and warnings
- enable v5.36 instead of strict and warnings in proxmox


perl-rs:

Lukas Sichert (2):
  common: remove deprecated PVE::RS::CalendarEvent compat module
  common: enable v5.36 in Proxmox::Lib::Common

 common/pkg/PVE/RS/CalendarEvent.pm | 3 ---
 common/pkg/Proxmox/Lib/Common.pm   | 2 ++
 2 files changed, 2 insertions(+), 3 deletions(-)
 delete mode 100644 common/pkg/PVE/RS/CalendarEvent.pm


proxmox:

Lukas Sichert (1):
  pve-api-types: enable v5.36 in Bool

 pve-api-types/generator-lib/Bool.pm | 2 ++
 1 file changed, 2 insertions(+)


Summary over all repositories:
  3 files changed, 4 insertions(+), 3 deletions(-)

-- 
Generated by murpp 0.12.0