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

Jeremy Katz katzj at redhat.com
Tue Aug 7 20:35:01 UTC 2007


On Tue, 2007-08-07 at 03:43 -0500, Douglas McClendon wrote:
> Jeroen van Meeuwen wrote:
> > 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
> 
> 
> Are you actually certain that you don't do any removal?

Indeed there is -- I thought there was, but then my quick look yesterday
didn't see it.  That really makes this a bit (okay, a lot) scary as it's
basically a quick path to nuking your system which is something that we
try pretty hard to be careful to not do.

And since you can just make use of the build dir after instantiating, I
think the risk is more than the gain

Jeremy




More information about the Fedora-livecd-list mailing list