[PATCH] iutil: Introduce getSysroot()/getTargetPhysicalRoot(), use instead of ROOT_PATH
Vratislav Podzimek
vpodzime at redhat.com
Fri Apr 25 08:09:18 UTC 2014
On Thu, 2014-04-24 at 13:37 -0700, Brian C. Lane wrote:
> On Thu, Apr 24, 2014 at 07:59:16AM +0200, Vratislav Podzimek wrote:
> > On Wed, 2014-04-23 at 21:10 +0000, Colin Walters wrote:
> > > On Wed, Apr 23, 2014 at 2:47 AM, Vratislav Podzimek
> > > <vpodzime at redhat.com> wrote:
> > > > I'm sorry, but I don't quite get this. The docstring says, this can
> > > > return something different from the sysroot and the only thing it
> > > > does is returning the value of getSysroot().
> > >
> > > Right. This attached patch fixes it to return a constant ROOT_PATH -
> > > we never want to mount the storage anywhere else really.
> > >
> > >
> > > However I think it's worth having a new function because it allows us
> > > to "git grep ROOT_PATH" and find any locations that need to be
> > > changed.
> > >
> > >
> > > Also, I took this opportunity to introduce setSysroot() which was in a
> > > later patch.
> > >
> > >
> > > New patch attached, thanks for the review!
> > This looks good to me. Unless somebody finds something problematic in
> > it, I'm gonna push this tomorrow.
>
> This looks ok to me as well. I also think I can finally hook up
> --dirinstall to pass it an actual path instead of the ENV hack.
Same for the Initial Setup. Pushing.
--
Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
More information about the anaconda-patches
mailing list