What would you like to be added
For the chroot parameter to the constructor to be an optional keyword only parameter, rather than packed into the host list, and for set_hosts to not have the ability to even try to set the chroot.
Why is this needed
There is a lot of complicated code to deal with this. And is you can't use set_hosts to actually set the chroot, it's mostly wasted anyway. Additionally, once you've set the chroot, if you try to call set_hosts you have to match the chroot you passed last time, which is toilsome.
I realise this'd be an incompatible change and plenty of warning/deprecation messages would be required.
What would you like to be added
For the
chrootparameter to the constructor to be an optional keyword only parameter, rather than packed into the host list, and forset_hoststo not have the ability to even try to set the chroot.Why is this needed
There is a lot of complicated code to deal with this. And is you can't use
set_hoststo actually set the chroot, it's mostly wasted anyway. Additionally, once you've set the chroot, if you try to callset_hostsyou have to match the chroot you passed last time, which is toilsome.I realise this'd be an incompatible change and plenty of warning/deprecation messages would be required.