Re: [libvirt] [PATCH] rpcgen fixes

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

 



On Wed, Dec 17, 2008 at 02:24:29PM +0100, Jim Meyering wrote:

> This sounds like an argument for not putting derived sources
> in CVS, since derived-for-one-system (linux/gnu) doesn't
> work for e.g., Solaris.

If you're OK with requiring rpcgen, that's most likely fine by me
(though you'd want to tighten up that glibc rpcgen check!)


> > +	rpcgen -h -o rp.h-t @top_srcdir@/qemud/remote_protocol.x
> > +	rpcgen -c -o rp.c-t @top_srcdir@/qemud/remote_protocol.x
> 
> Use $(srcdir), in place of @top_srcdir@/qemud (and others below).
> The latter @var@ syntax is old, if not deprecated,

Coo, been a while - will do.

> > +	perl -w @top_srcdir@/qemud/rpcgen_fix.pl rp.h-t \
> > +	    >@top_srcdir@/qemud/remote_protocol.h
> 
> Please don't redirect directly to something used as a build source.
> Instead, redirect to a temporary file, and rename that into place
> upon successful completion.  That makes it less likely you'll ever
> end up with corrupted sources.

Sorry I'm missing why this helps any?

regards
john

--
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]