Re: [Iverilog-devel] Snapshot today?

[Larry D. <ldo...@re...>]

On Fri, Jan 11, 2008 at 02:20:08PM -0800, Stephen Williams wrote:
> I find myself waiting on simulations today so am looking into making
> a devel snapshot. Are there any outstanding patches that people really
> want to see in this snapshot? I'm in the process of going through the
> Patches tracker to apply those already submitted patches.

Not-so-minor issue I just ran into installing on a fresh computer
(I'm rebuilding my laptop -- Gutsy Gibbon, not that it matters).

The build rules for eval_bool.o don't include a proper include
directory to search for _pli_types.h.

gcc -DHAVE_CVS_IDENT=1 -I. -I../../verilog/tgt-vvp/..  -DHAVE_CONFIG_H -fPIC -Wall -g -O2 -MD -c ../../verilog/tgt-vvp/eval_bool.c -o eval_bool.o
../../verilog/tgt-vvp/eval_bool.c:23:57: error: _pli_types.h: No such file or directory
../../verilog/tgt-vvp/eval_bool.c: In function `draw_number_bool64':
../../verilog/tgt-vvp/eval_bool.c:67: error: expected ')' before 'PLI_UINT64_FMT'
../../verilog/tgt-vvp/eval_bool.c:67: warning: too few arguments for format
make[1]: *** [eval_bool.o] Error 1

I think I never hit this before because I always had a previous valid
install, so it finds _pli_types.h from that .. but I'm not sure.
It looks like adding -I.. will do.  <tappety-typ>  Yup, that works, I checked.

This problem is new (as of commit 21730ab228e5812f8b09b4f9b50843f343b99321),
we just added the include so we could print a uint64_t properly
(standards-compliantly).

   - Larry