[pve-devel] [PATCH manager] CephTools: cleanup module usage

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Nov 20 09:36:12 CET 2018


as done with the api ceph modules:
most of this was imported by just copying without veryfing if all is
actually required. Some lost it's purpose as we re-used more from our
existing module code base (e.g., pve-common) but wasn't actually
removed.

As this file includes two perl modules you need to take a bit caution
when looking at this, as some things are used in one module but not
the other - simple grep'ing at this may give false positives.

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---

Actually, dir_glob_foreach could be removed too, but I plan to use it soon
here, so it may not make much sense doing a remove/add.

 PVE/CephTools.pm | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/PVE/CephTools.pm b/PVE/CephTools.pm
index 25160d73..446ff531 100644
--- a/PVE/CephTools.pm
+++ b/PVE/CephTools.pm
@@ -2,13 +2,10 @@ package PVE::CephTools;
 
 use strict;
 use warnings;
-use File::Basename;
+
 use File::Path;
-use POSIX qw (LONG_MAX);
-use Cwd qw(abs_path);
-use IO::Dir;
 
-use PVE::Tools qw(extract_param run_command file_get_contents file_read_firstline dir_glob_regex dir_glob_foreach);
+use PVE::Tools qw(run_command dir_glob_foreach);
 
 my $ccname = 'ceph'; # ceph cluster name
 my $ceph_cfgdir = "/etc/ceph";
-- 
2.19.1





More information about the pve-devel mailing list