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@xxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxx\" --------------------------------- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review