Re: C++17 skeleton with move semantics and std::variant

[Hans Åberg]

> On 10 Apr 2018, at 17:24, Frank Heckenbach <address@hidden> wrote:
> 
> Hans Åberg wrote:
> 
>> Rename, if possible, all these patched files to location-c++17.cc,
>> stack-c++17.hh, etc, as they are not in the original distribution.
> 
> I called it stack-c++11.hh because it only uses move (C++11), not
> variants (C++17). And I've explained why I didn't rename
> location.cc. If you disagree, please reply to those posts of mine.

Before it gets integrated into the Bison distribution, you might want to put it 
in the package source directory. I was able to do it, with the following 
changes:

In the .yy file, I had to put in "./", to:
  %skeleton "./lalr1-c++17.cc"
There seems to be a bug in Bison 3.0.4, looking only in the installation 
directory if it is not there.

In lalr1-c++17.cc, I took away the prefix 'b4_pkgdatadir/' for the files 
c++17.m4, location-c++11.cc, stack-c++11.hh, variant-c++17.hh. The renaming of 
your location.cc then makes sure there is no confusion with the original Bison 
one.

Then Makefile compiles with 'make', and 'make check'.