[Bug 468516] Review Request: verilator - A fast simulator of synthesizable Verilog HDL

[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=468516





--- Comment #20 from Chitlesh GOORAH <cgoorah at yahoo.com.au>  2009-01-03 19:26:35 EDT ---
#001: These should not be shipped

/usr/share/verilator/include/verilated.mk.in --> duplicate with
/usr/share/verilator/include/verilated.mk

#002: 

do we need to ship these with this package ?
usr/share/verilator/perl-systemc/src
/usr/share/verilator/perl-systemc/src/SpCommon.h
/usr/share/verilator/perl-systemc/src/SpTraceVcdC.cpp
/usr/share/verilator/perl-systemc/src/SpTraceVcdC.h
/usr/share/verilator/perl-systemc/src/systemperl.h

Wouldn't it be wise to package perl-SystemPerl ? I have already started
packaging perl-SystemPerl. Soon I'll post a package review for perl-SystemPerl

-- 
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.