Hans Åberg wrote: > > On 10 Apr 2018, at 17:24, Frank Heckenbach <f.heckenb...@fh-soft.de> 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 won't for my code (but you, or anyone else, may want to). FWIW, I need patches to some Debian packages, some of them for years as many maintainers don't seem very interested in bug fixes (that don't affect them personally), unfortunately. So I keep a directory of all such patches, so I can easily reapply them after upgrades, and my Bison patch will just join them there, until it's integrated, or otherwise forever (probably not the only one): ... > 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. Bug or feature, I don't know. Maybe it's supposed to work this way. I will now submit my changes to savannah. If the maintainers react to them, great, and we can discuss the details; otherwise, I think doing anything else now would just be a waste of time. Regards, Frank
