diff options
| author |   Mike Frysinger <vapier@gentoo.org> | 2016-05-19 15:22:13 -0400 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2016-05-19 15:39:22 -0400 |
| commit | d8c463e167f5be77bedf767646d9485a9a8f4f8f (patch) | |
| tree | 4cf6ff2bdc12f2d7a4235287c6947f55c5a51970 | |
| parent | clear_dir: make a bit more robust/flexible (diff) | |
| download | catalyst-d8c463e167f5be77bedf767646d9485a9a8f4f8f.tar.gz catalyst-d8c463e167f5be77bedf767646d9485a9a8f4f8f.tar.bz2 catalyst-d8c463e167f5be77bedf767646d9485a9a8f4f8f.zip | |
clear_dir: use kwargs everywhere
Don't pass args by position when clear_args uses optional args.
This makes it easy to screw up the calls when changing the API.
| -rw-r--r-- | catalyst/base/clearbase.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/catalyst/base/clearbase.py b/catalyst/base/clearbase.py index 9a4c625b..644a385f 100644 --- a/catalyst/base/clearbase.py +++ b/catalyst/base/clearbase.py @@ -30,13 +30,13 @@ class ClearBase(object): def clear_chroot(self): self.chroot_lock.unlock() log.notice('Clearing the chroot path ...') - clear_dir(self.settings["chroot_path"], 0o755, True) + clear_dir(self.settings["chroot_path"], mode=0o755, chg_flags=True) def remove_chroot(self): self.chroot_lock.unlock() log.notice('Removing the chroot path ...') - clear_dir(self.settings["chroot_path"], 0o755, True, remove=True) + clear_dir(self.settings["chroot_path"], mode=0o755, chg_flags=True, remove=True) def clear_packages(self, remove=False): |