[Bug 226715] Review Request: irsim - Switch-level simulator

bugzilla at redhat.com bugzilla at redhat.com
Thu Feb 1 16:07:43 UTC 2007


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

Summary: Review Request: irsim - Switch-level simulator


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





------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2007-02-01 11:07 EST -------
Well, for -2.1:

* License:
  I doubt that this is licensed under GPL.
  All texts which include explicit license terms
  refer to the same sentences, however, I just
  don't know how this license is called...
  Please check:
  ./analyzer/anXhelper.c for example.
  (Note: license for files created by autotools
   should be ignored).

* Some documentation
  I wonder if the 3 documentation
-----------------------------------------
/usr/lib/irsim/doc/irsim-analyzer.doc
/usr/lib/irsim/doc/irsim.doc
/usr/lib/irsim/doc/netchange.doc
-----------------------------------------
  are really needed because:
  * they are the same as man pages installed.
  * it seems that they are not used at runtime.

? Would you give me some examples so that
  I can check if this program works well?

? I just wonder if the following compilation flag is
  proper:
---------------------------------
DPACKAGE_BUGREPORT=\"magic-hackers at csl.cornell.edu\"
---------------------------------
  Should this be your mail address?

? By the way, what are the files under other/
  directory?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list