RE: Re: [PATCH] install.sh: install as root withreasonable permissions

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

RE: Re: [PATCH] install.sh: install as root withreasonable permissions

Ian Pratt
 
> To be quite honest, I'm not entirely happy with the solution
> myself.  I think install.sh should really manage what is
> installed more intellegently. And allow for uninstall too.
> However that is really more the role of a package manager,
> this fix does seem to be reasonably clean and solve the
> problem at hand farily unintrousively, so I think it is a
> good choice, at least for now.

I think this patch at least helps improve install.sh.

I would like to see a --uninstall option to install.sh added though,
preferably one that removed all traces of all possible previous xen
versions (and renamed /etc/xen as per make uninstall)

Best,
Ian

_______________________________________________
Xen-devel mailing list
[hidden email]
http://lists.xensource.com/xen-devel
Reply | Threaded
Open this post in threaded view
|

Re: Re: [PATCH] install.sh: install as root withreasonable permissions

Simon Horman
On Sat, Dec 17, 2005 at 03:28:19AM -0000, Ian Pratt wrote:

>  
> > To be quite honest, I'm not entirely happy with the solution
> > myself.  I think install.sh should really manage what is
> > installed more intellegently. And allow for uninstall too.
> > However that is really more the role of a package manager,
> > this fix does seem to be reasonably clean and solve the
> > problem at hand farily unintrousively, so I think it is a
> > good choice, at least for now.
>
> I think this patch at least helps improve install.sh.
>
> I would like to see a --uninstall option to install.sh added though,
> preferably one that removed all traces of all possible previous xen
> versions (and renamed /etc/xen as per make uninstall)

Agreed

--
Horms

_______________________________________________
Xen-devel mailing list
[hidden email]
http://lists.xensource.com/xen-devel