Am 30.10.2012 11:20, schrieb Dietmar Maurer:
Why not? What is the problem to use:

use PVE::Cluster;

my $conf = PVE::Cluster::cfs_read_file('datacenter.cfg');

That would create a circular package dependency!

Note: pve-cluster depends on pve-common

*argh* right. Mhm i think cfs_read_file should be a method in pve-common. Wouldn't that make sense?

To make that change not too hard we can still have:

package PVE::Cluster;

..

# here for backward compatiblity
sub cfs_read_file {
  return PVE::Tools::cfs_read_file(@_);
}

..

Stefan
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to