On Thu, 21 Jul 2011 10:03:22 +0200 Romain Geissler <romain.geiss...@gmail.com> wrote:
> Hi, > > I added a few includes i need to define some new primitives. > > Basile, i included two "c-family/*.h" files, but be aware that currently > GCC does not install this headers in the c-family directory but in the > directly in the plugin/include root directory with all other headers. I posted > a patch some times ago to solve that (because it breaks includes of > C++ plugin headers) for the trunk. Unfortunately, the patch hasn't been > reviewed yet. See > http://old.nabble.com/-PLUGIN--c-family-files-installation-td32038394.html > Once again, Debian maintainers applied a similar patch for a long time > and thus theses files are in the c-family directory (but only for Debian > users). > > So i don't know if you should apply the patch as is or remove > the "c-family/" prefix in the includes. > > Romain Geissler > > gcc/ > > 2011-07-21 Romain Geissler <romain.geiss...@gmail.com> > > * melt-run.proto.h: Include "c-family/c-pragma.h", > "c-family/c-pretty-print.h", "cpplib.h" and "langhooks.h". > Actually, we have the same issue in melt-runtime.c, which has #include "c-pragma.h" And I want badly MELT to be compilable as a plugin to an umodified gcc-4.6-plugin-dev package on Debian. I don't understand if it is an issue related to building GCC & its plugin with C++, or it is an issue with 4.7 ? I am tempted to put in melt-runtime.c something like /* Headers from c-family/ should be included directly with GCC4.6, but not with GCC 4.7 or when compiling with a C++ compiler. */ #if defined(GCCPLUGIN_VERSION) || MELT_GCC_VERSION>4006 || defined(__cplusplus) #include "c-family/c-pragma.h" #else #include "c-pragma.h" #endif And likewise in melt-run.proto.h What do you think? (I really want the MELT plugin to be compilable on an installed gcc-4.6!) Cheers -- Basile STARYNKEVITCH http://starynkevitch.net/Basile/ email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359 8, rue de la Faiencerie, 92340 Bourg La Reine, France *** opinions {are only mine, sont seulement les miennes} ***
