[libvirt] [PATCH]show the autostart status when displaya'virsh dominfo'.
Daniel Veillard
veillard at redhat.com
Mon Jun 2 11:03:15 UTC 2008
On Mon, Jun 02, 2008 at 10:39:36AM +0100, Richard W.M. Jones wrote:
> On Wed, May 28, 2008 at 02:18:50PM +0100, Daniel P. Berrange wrote:
> > On Wed, May 28, 2008 at 11:21:05AM +0900, Atsushi SAKAI wrote:
> > > Hi, Rich and Dan
> > >
> > > Thank you for fixing this.
> > > I think it should be noted about submitting patch on HACKING file or others.
> > > How do you think?
> > > (like make check; make syntax-check; make tests)
> >
> > Yes, basically the three things you should run before committing patches
> > are
> >
> > make check
> > make syntax-check
> > make valgrind (in the tests/ sub-dir)
> >
> > The latter checks for memory leaks.
> >
> > Of course I myself forget this often - which is why we have the automated
> > nightly builds, so we never miss the problem for longer than a day
>
> Suggested patch to HACKING file.
Nice writeup ! +1
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard at redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
More information about the libvir-list
mailing list