qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PULL v2 19/20] trace: introduce a formal group name fo


From: Greg Kurz
Subject: Re: [Qemu-devel] [PULL v2 19/20] trace: introduce a formal group name for trace events
Date: Mon, 17 Oct 2016 12:28:20 +0200

On Thu, 13 Oct 2016 15:58:55 +0200
Paolo Bonzini <address@hidden> wrote:

> On 12/10/2016 10:47, Stefan Hajnoczi wrote:
> > +def make_group_name(filename):
> > +    dirname = os.path.realpath(os.path.dirname(filename))
> > +    basedir = os.path.join(os.path.dirname(__file__), os.pardir)
> > +    basedir = os.path.realpath(os.path.abspath(basedir))
> > +    dirname = dirname[len(basedir) + 1:]
> > +
> > +    if dirname == "":
> > +        return "common"
> > +    return re.sub(r"/|-", "_", dirname)
> >    
> 
> The group is based on the build directory, so it includes the relative
> path from srcdir to builddir.  My build directory is weird ("+build")
> and breaks because of this.  Since the group is not really used for
> anything yet, perhaps this patch could be reverted?
> 
> Paolo
> 

As pointed out in other mails, the real problem isn't even about ending
up with weird characters in the group name, but about including the
build directory itself...

--
Greg



reply via email to

[Prev in Thread] Current Thread [Next in Thread]