[Cocci] Parse error with genl-const.cocci
julia.lawall at lip6.fr
Thu Dec 14 16:16:42 CET 2017
On Tue, 12 Dec 2017, Remington Furman wrote:
> On 12/01/2017 02:00 AM, Julia Lawall wrote:
> > On Fri, 1 Dec 2017, Johannes Berg wrote:
> > > Hi Julia,
> > >
> > > On Fri, 2017-12-01 at 10:18 +0100, Julia Lawall wrote:
> > > > The handling of attributes has also been extended so that you can
> > > > actually
> > > > match against them in some cases, such as on function declarations. On
> > > > the oher hand, nothing was done to allow attributes in front of types
> > > > that
> > > > are alone.
> > > Very cool!
> > > > I can try to fix this. In the short term, perhaps using 1.0.4 is
> > > > acceptable.
> > > To me that's OK, I'll try to keep it in mind.
> > >
> > > Can we do some "#ifdef" type syntax in spatch on the version of it? :-)
> > There's nothing for that available at the moment.
> > > I guess I could also have two different versions of the patch and write
> > > some code to pick up which one to use, but that's a bit awkward ...
> > >
> > > But let me know if you can't actually fix this (easily), then I can do
> > > that instead.
> > OK, I'll try to look at it soon.
> > julia
> I thought I'd point out the layers of coevolution that result from the
> changing spatch behavior. :) It's a bit fractal when every layer in the
> dependency graph has the possibility to change. I suppose a #ifdef type
> solution might remove the potential need to backport the .cocci patches
> themselves going forward.
> But, I think it might be sufficient to just document what version of the
> tools, in this case spatch, were used for a particular backports
> commit/release. It's easy enough to build any version of spatch from git. Or
> perhaps each .cocci patch could have a comment listing the spatch version it
> was developed with. Future backports releases could then update the patch
> syntax as needed, document the spatch version, and leave it at that.
I updated the github version of Coccinelle such that
will almost parse correctly. The only thing that remains is to change
attribute into attribute name.
More information about the Cocci