[Distutils] Patch for new install options

Harry Henry Gebel hgebel@inet.net
Thu, 27 Apr 2000 00:25:25 -0400


On Wed, Apr 26, 2000 at 09:27:14PM -0400, Greg Ward wrote:
> > ./setup.py install --prefix=$RPM_BUILD_ROOT'$sys_prefix' \
> >   --exec-prefix=$RPM_BUILD_ROOT'$sys_exec_prefix'
> > 
> > Compare this with:
> > 
> > ./setup.py install --package-prefix=$RPM_BUILD_ROOT
> 
> Hmmm, you have a point.  When generating built distributions, you really 
> do want to sneak a new root directory into the installation paths.  So
> how about this minor spelling change:
> 
>     ./setup.py install --root=$RPM_BUILD_ROOT

That's better, I never was good at thinking up names.

> PS. using "--prefix='/tmp$sys_prefix'" won't require knowledge of
> Distutils internals, since the configuration variables will be a
> documented part of the interface.  I admit that it's still a tricky and
> specialized part of the interface though, and it would be best to
> restrict config vars to the config file if at all possible.  It just
> seems bogus to *disallow* them from the command line when they might be
> useful, hence last night's hack.  Err, patch.

I agree, disallowing them is wrong too, I just didn't think in was a good
idea in that location where people would, in a sense, be using them
without realizing it.

-- 
Harry Henry Gebel, Senior Developer, Landon House SBS
West Dover Hundred, Delaware
PyNcurses ncurses binding for Python: http://pyncurses.sourceforge.ne