Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-07 Thread Wolfgang Bumiller
I'm sure there are other ways but the simulator this code was already there in Firewall.pm and I there are packages with more annoying build issues I wanted to move on to ;-) Personally I don't really see a problem with conditional imports. (They're particularly useful in dealing with package boot

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Stefan Priebe - Profihost AG
To avoid cycling deps an alternative might be forward decleration. http://www.perlmonks.org/?node_id=1057957 Stefan Excuse my typo sent from my mobile phone. Am 06.02.2017 um 18:38 schrieb Dietmar Maurer : >> An alternative might be >> http://perldoc.perl.org/autouse.html > > Thank for that i

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Dietmar Maurer
> An alternative might be > http://perldoc.perl.org/autouse.html Thank for that interesting link. But It would prefer a way to avoid that cyclic package dependency at all. On the other way, the current solution is not that bad ;-) ___ pve-devel mailing

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Stefan Priebe - Profihost AG
An alternative might be http://perldoc.perl.org/autouse.html Stefan Excuse my typo sent from my mobile phone. > Am 06.02.2017 um 18:20 schrieb Dietmar Maurer : > > is there really no other way to solve this issue? > > ___ > pve-devel mailing list > p

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Dietmar Maurer
is there really no other way to solve this issue? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Wolfgang Bumiller
applied and added another missing hunk for qemu-server On Tue, Jan 31, 2017 at 11:15:22AM +0100, Wolfgang Bumiller wrote: > --- > in order to be able to bootstrap the package (eg. for stretch) > (copypasted as-is from Firewall.pm) > > src/PVE/FirewallSimulator.pm | 19 +-- > 1 fi