From Mathieu Bridon <bochecha(a)fedoraproject.org>, 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" <lars(a)pixar.com>
>
> 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