[Fedora-livecd-list] [PATCH 1/6] Set the build directory

[Jeroen van Meeuwen]

Jeremy Katz wrote:
> On Mon, 2007-08-06 at 23:21 +0200, Jeroen van Meeuwen wrote:
>> Jeremy Katz wrote:
>>> On Sun, 2007-08-05 at 20:24 +0200, Jeroen van Meeuwen wrote:
>>>> A patch to enable setting the build directory.
>>>>
>>>> This adds more flexibility for those who don't have as much space in
>>>> /var/tmp.
>>> If you don't have space in /var/tmp, just use
>>> --tmpdir=/some/other/dir/with/space.  Setting the build directory just
>>> sets people up for either a) deleting things they don't intend to or b)
>>> getting stale data in their build
>> Right, tmpdir does evade the i-do-not-have-enough-space-in-/var/tmp
>> issue. Push comes to shovel, I really don't need to set the build_dir to
>> anything I can think of, rather then just use the
>> InstallationTarget.build_dir for any reference after I import
>> livecd-creator and create the extended InstallationTarget instance.
> 
> This sounds a lot better to me...
> 
>> I'm not sure though what you mean by a) and b) though. AFAICT these
>> would never just happen because build_dir === build_dir, regardless of
>> what you set it to, or how you set it.
> 
> Okay, we don't actually do any removal, so you won't hit that.  But if
> you set a build_dir that already exists, then you're setting up for some
> "interesting" conditions.  By ensuring that it's a new mkdtemp'd dir, we
> can be certain that we own it, that it's unique, and that no one has
> done anything with it first
> 

So again if we remove the opts, livecd-tools is safe, and Revisor gets
to do whatever it wants. Would that be an acceptable patch?

Kind regards,

Jeroen van Meeuwen
-kanarip