[pve-devel] [PATCH common] CLIHandler: add missing use PVE::INotify

Wolfgang Bumiller w.bumiller at proxmox.com
Wed Feb 10 09:38:43 CET 2016


It uses PVE::INotify::inotify_init() in run_cli_handler().
---
 Note that it also uses PVE::RPCEnvironment in run_cli_handler() but
 adding a `use` clause for this would make the entire package (not
 just that function) depend on the pve-access-control and thereby
 introduce a circular dependency.
 But INotify is part of the same package (as in .deb, not perl
 package) (and does not depend on CLIHandler), so this part makes
 sense.

 src/PVE/CLIHandler.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/PVE/CLIHandler.pm b/src/PVE/CLIHandler.pm
index 21fb916..c2d3438 100644
--- a/src/PVE/CLIHandler.pm
+++ b/src/PVE/CLIHandler.pm
@@ -8,6 +8,7 @@ use PVE::SafeSyslog;
 use PVE::Exception qw(raise raise_param_exc);
 use PVE::RESTHandler;
 use PVE::PodParser;
+use PVE::INotify;
 
 use base qw(PVE::RESTHandler);
 
-- 
2.1.4





More information about the pve-devel mailing list