Re: [PATCH] rm daemon/remote_dispatch_table.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 05, 2011 at 09:24:39AM +0100, Daniel P. Berrange wrote:
> On Thu, May 05, 2011 at 02:59:39PM +0800, Daniel Veillard wrote:
> > On Thu, May 05, 2011 at 02:31:57PM +0800, Hu Tao wrote:
> > > On Thu, May 05, 2011 at 08:07:49AM +0200, Matthias Bolte wrote:
> > > > 2011/5/5 Hu Tao <hutao@xxxxxxxxxxxxxx>:
> > > > > 1. this file is automatically generated at compile-time, so rm it
> > > > > Â to avoid further commit to this file.
> > > > > 2. any update should made to file src/remote/remote_protocol.x.
> > > > > ---
> > > > > Âdaemon/remote_dispatch_table.h | 1054 ----------------------------------------
> > > > > Â1 files changed, 0 insertions(+), 1054 deletions(-)
> > > > > Âdelete mode 100644 daemon/remote_dispatch_table.h
> > > > >
> > > > 
> > > > There is probably a reason why this generated file is under version
> > > > control, maybe Dan knows.
> > > > 
> > > > If we decide to remove the generated protocol dispatch header files
> > > > then we should remove all of them (daemon/remote_dispatch_*.h and
> > > > daemon/qemu_dispatch_*.h) and not just this single one.
> > > 
> > > :( doesn't noticed other daemon/remote_dispatch_*.h are also generated
> > > ones. Thanks for pointing it out.
> > > 
> > > Dan, what's your opinion?
> > 
> >   my recollection is that they are kept because when compiling on
> > windows we don't have ways to generate them. So while generated,
> > they are kept in git and in the dist tarballs.
> 
> We kept them because the rpcgen binary was kind of flakey on some
> platforms. This may no longer be required, but will need to check

But daemon/remote_dispatch_*.h are generated by
daemon/remote_generate_stubs.pl which doesn't depend on rpcgen. I
deleted the *.h files and /usr/bin/rpcgen, and `make' still generated
the *.h files.

Maybe you mean src/remote/rpcgen_fix.pl but it is not related to the
daemon/remote_dispatch_*.h files.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]