I agree with this in principle. mock can be configured to put chroots anywhere, so it makes sense to NOT assume /var/lib/mock.
ack On Tue, Feb 4, 2014 at 10:41 PM, Mathieu Bridon <[email protected]> wrote: > Anyone? > > > -- > Mathieu > > On Wed, 2014-01-29 at 17:56 +0800, Mathieu Bridon wrote: >> The builder configuration allows setting this to a different folder. >> --- >> koji/daemon.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/koji/daemon.py b/koji/daemon.py >> index fa22d5a..4f4b575 100644 >> --- a/koji/daemon.py >> +++ b/koji/daemon.py >> @@ -651,7 +651,7 @@ class TaskManager(object): >> if id is None or name is None: >> continue >> # see if there's a dir for the buildroot >> - vardir = "/var/lib/mock/%s" % name >> + vardir = os.path.join(self.options.mockdir, name) >> #XXX >> buildroots[id] = {} >> buildroots[id]['name'] = name > > > > -- > buildsys mailing list > [email protected] > https://admin.fedoraproject.org/mailman/listinfo/buildsys -- -Jon -- buildsys mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/buildsys
