Re: [qmtest] --root should not be used to write the config file

[Matthias Klose <doko@xxxxxxxxxxxxxxx>]

Mark Mitchell writes:
> Matthias Klose wrote:
> > seen in qmtest-2.4; the Debian build installs into a temporary
> > installation directory, which results in a wrong extension_path in the
> > config.py file.
> 
> Thank you for the patch.
> 
> > --- qmtest-2.4.orig/qmdist/command/install_lib.py
> > +++ qmtest-2.4/qmdist/command/install_lib.py
> > @@ -49,7 +49,7 @@
> >          config_file = join(self.install_dir, 'qm', 'config.py')
> >          self.announce("adjusting config parameters")
> >          i = self.distribution.get_command_obj('install')
> > -        prefix = i.root or i.prefix
> > +        prefix = i.prefix
> >          extension_path = join('share',
> >                                'qmtest',
> >                                'site-extensions-%d.%d'%sys.version_info[:2])
> 
> Can you explain a little more about what this patch is doing?  In 
> looking at the distutils documentation, I couldn't find anything about 
> "root".  I assume that i.prefix is the --prefix argument provided to 
> setup.py, but what's i.root?

It is the value which gets passed when running install --root <path>
(as setting DESTDIR for most make based installations).  But indeed, I
cannot find this option documented in the distutils docs. The help
output documents this option:

  "install everything relative to this alternate root directory"

Matthias