[Bug 478749] Review Request: dinotrace - X11 waveform viewer for electronics

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=478749





--- Comment #25 from wsnyder at wsnyder.org  2009-04-08 19:56:53 EDT ---
>[wolfy at wolfy tmp]$ dinotrace /tmp/*vcd
>%E, Strange bit value U on line 388 of /tmp/wave1.vcd
>wave1,vcd is in fact part of stock IUS82, that is
>/tools.lnx86/inca/examples/vhpi/waveform/test/golden/wave1.vcd 

U's aren't legal in VCDs.  This is probably a EVCD file, which verilator
doesn't support. EVCDs are used primarily by chip testers, it's a bit odd that
an example would use that.

>2. Is the emacs subpackage really supposed to be used without having dinotrace
>installed? I see no reason to not allow that, but I want your confirmation that
>this is intentional.  

No, the emacs part won't be usable without the verilog-mode.el package, but
the rest of the tool will be.

If you're distributing with emacs-21.4 then you really should be providing a
verilog-mode.el RPM.  It's far more in demand than Dinotrace/Verilator/Icarus
all which you're including.  Emacs-22 includes it as part of the Emacs core
library, so if you release a more recent emacs that gets it too.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.