porting problem triggered by gcc include search order about nvc HOT 7 CLOSED

I've no idea why that happens. I don't mind renaming rt/signal.h if that fixes it though. How about rt/sigdef.h?

from nvc.

can you check this?

http://cygwin.com/ml/cygwin/2012-02/msg00521.html

so nvc needs to clean up for #include and -I?

from nvc.

On 17/02/12 03:16, KIMURA Masaru wrote:

can you check this?

http://cygwin.com/ml/cygwin/2012-02/msg00521.html

so nvc needs to clean up for #include and -I?

These are added by automake:

http://lists.gnu.org/archive/html/automake/2009-07/msg00005.html

Unfortunately it looks like all three are required to:
a) Support building out-of-tree (which I do normally)
b) Find header files generated by bison
c) Find the autoconf config.h

So renaming the file might be the easiest solution.

Nick

from nvc.

ok, how about this?

hiyuh@077ecfc

I think this is best and simple fix ATM until automake uses -iquote instead of -I for DEFAULT_INCLUDES.

from nvc.

On 17/02/12 10:36, KIMURA Masaru wrote:

I think this is best and simple fix ATM until automake uses -iquote instead of -I for DEFAULT_INCLUDES.

This seems like a good fix but on Debian wheezy I get warnings from GCC
and an error from a TCL header:

gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../../src -I.. -Wall -Werror -I-
-g -O2 -MT nvc.o -MD -MP -MF .deps/nvc.Tpo -c -o nvc.o ../../src/nvc.c
cc1: note: obsolete option -I- used, please use -iquote instead
In file included from ../../src/nvc.c:31:0:
/usr/include/tcl/tcl.h:2244:22: fatal error: tclDecls.h: No such file or
directory

How about only adding -I- on cygwin? Can you see if this patch works?
It's based on your for-nickg branch.

835dcfe

Nick

from nvc.

2012/2/18 Nick Gasson wrote:

How about only adding -I- on cygwin? Can you see if this patch works?
It's based on your for-nickg branch.

835dcfe

It works fine. Maybe it is now minimal and practical fix. I'm OK to
close this issue.
If you do not hate cygwin, could you mind to merge them to your master?

from nvc.

I've merged it. Thanks for your work on this!

from nvc.