[Patches] [Patch #103681] Remove grammar rule from Makefile

noreply@sourceforge.net noreply@sourceforge.net
Tue, 20 Feb 2001 14:19:36 -0800


Patch #103681 has been updated. 

Project: python
Category: Build
Status: Rejected
Submitted by: donnc
Assigned to : nobody
Summary: Remove grammar rule from Makefile

Follow-Ups:

Date: 2001-Feb-20 14:19
By: donnc

Comment:
Actually this is only a more complicated way to do the
same thing - now you generate the files, but you still
don't use them.  Not graminit.h, anyway.  Here's what
happens if I remove graminit.h from the source directory:

cc -c -g -O2 -Wall -Wstrict-prototypes -I.
  -I/tulsa/src/python-2.1a2b/Include -DHAVE_CONFIG_H 
  -o Python/compile.o /tulsa/src/python-2.1a2b/Python/compile.c
/tulsa/src/python-2.1a2b/Python/compile.c:17: graminit.h: No such file or
directory

You would have to add -IInclude to CFLAGS, along with
-I$(srcdir)/Include;  otherwise --srcdir build with
modified grammar is broken (whether srcdir is read only
or read/write.)
-------------------------------------------------------

Date: 2001-Feb-15 20:14
By: nascheme

Comment:
I've checked in, IMHO, a better solution.  The grammar
source files are now generated in the build directory.  If
this doesn't work for a read-only source dir please let me
know.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=103681&group_id=5470