Re: [PATCH v2 5/6] tracetool: add test suite for tracetool with reference output

["Daniel P. Berrangé" <berrange@redhat.com>]

On Wed, Aug 06, 2025 at 04:58:26PM +0200, Paolo Bonzini wrote:
> On 8/6/25 12:24, Daniel P. Berrangé wrote:
> > +#line 48 "../../../../../../../../dev/stdout"
> 
> These line directives depend on the name of the QEMU build directory...
> 
> > +    args += [events,
> > +             "/dev/stdout"]
> 
> ... because you're using an absolute path here.
> 
> Maybe you can use a temporary directory, in which you copy the source file
> as well?

The other issue is that this will be unlikely to work on
Windows which lacks /dev/stdout. The easy solution to both
these problems is to make tracetool accept "-" as a shorthand
for stdout, in keeping with traditional UNIX tools.


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|