From Mathieu Bridon <[email protected]>, Mon, Sep 08, 2014 at 09:57:22AM +0200: > On Sat, 2014-09-06 at 20:25 -0700, Lars R. Damerow wrote: > > From: "Lars R. Damerow" <[email protected]> > > > > This patch stops multiple mockbuild runs for different distributions > > from clobbering each other. > > Isn't the disttag already part of the results dir tree? > > results_%(name)s/%(version)s/%(release)s/ > > And the release includes the dist (well, unless it hasn't been set in > the spec file...)
We have lots of old packages that don't include the dist in the release. > This changes where the results are though (they aren't all in the same > results_%(name)s folder any more), which might break some expectations? > > Maybe this specific change should really be in your rpkg-based tool? That's fine with me. The only complication is that this line is in the middle of a fair bit of code, so I'd have to replicate the whole function in my derived class. How about I factor it into its own function call (Commands._set_result_dir()?) that can be easily overridden? thanks, -lars -- lars r. damerow :: button pusher :: pixar animation studios -- buildsys mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/buildsys
