[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] trace: fix group name generation
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH] trace: fix group name generation |
Date: |
Thu, 17 Nov 2016 10:10:26 +0100 |
On Thu, 17 Nov 2016 16:07:38 +0800
Fam Zheng <address@hidden> wrote:
> On Thu, 10/20 15:25, Stefan Hajnoczi wrote:
> > >
> > > I have two other patches ready to fix the current situation:
> > > - one using os.getcwd() to guess the build directory
> > > - one implementing --group as mentioned in my other mail
> > >
> > > But the one that filters unwanted characters is a less intrusive
> > > workaround.
> >
> > If Dan's patches will eliminate the issue then we can take a workaround.
> >
> > Any more comments about Greg's patch before I merge it?
>
> Should we include this in -rc1? I still see a build error today.
>
> Fam
>
Hi Fam,
My patch was partly superseded by this commit:
commit 630b210b9abbf362905a2096c22c5eb1d6224e77
Author: Stefan Weil <address@hidden>
Date: Thu Oct 13 20:29:30 2016 +0200
Fix build for less common build directories names
which does:
- return re.sub(r"/|-", "_", dirname)
+ return re.sub(r"[^A-Za-z0-9]", "_", dirname)
What is the build error you're hitting ?
--
Greg