On Fri, 25 Jan 2002, Dan Sugalski wrote:
> At 10:14 AM -0500 1/25/02, Andy Dougherty wrote:
> >For parrot, we'd ideally like to make it a lot safer to
> >
> > #include <parrot/parrot.h>
> Nope--we'd ideally like to smack anyone writing non-core code that
> does that. :)
> Embedders will include parrot/parrot_embed.h, while extension folks
> will include parrot/parrot_extend.h. Neither will include
> parrot/parrot.h--not safe.
Sounds like a good plan. Perhaps something like the following patch is in
order then, more as a reminder for the future than anything actually
useful for now? (Note the changed file names: parrot/parrot_e*.h is
apparently redundant and definitely isn't 8.3-friendly, but perhaps you
were guarding against the VMS #include problem I vaguely recall where the
directory name "parrot" in "parrot/foo.h" could sometimes get ignored?)
diff -r -u parrot-orig/include/parrot/parrot.h parrot-andy/include/parrot/parrot.h
--- parrot-orig/include/parrot/parrot.h Mon Jan 14 15:04:29 2002
+++ parrot-andy/include/parrot/parrot.h Fri Jan 25 16:52:14 2002
@@ -10,6 +10,11 @@
* References:
*/
+/* Only parrot core files should include this file.
+ Extensions should include <parrot/extend.h>.
+ Programs embedding parrot should include <parrot/embed.h>.
+*/
+
#if !defined(PARROT_PARROT_H_GUARD)
#define PARROT_PARROT_H_GUARD
--
Andy Dougherty [EMAIL PROTECTED]
Dept. of Physics
Lafayette College, Easton PA 18042
