Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: ruby https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226381 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-04-25 21:46 EST ------- One another note: * /usr/lib/ruby/1.8/i386-linux/config.h - Including config.h as header files are generally considered as "very bad" because * the name is very generic and may cause name space conflicts * the definition #HAVE_???? macro also may cause conflicts when other config.h is included. Actually, the content of this file (/usr/lib/ruby/1.8/i386-linux/config.h) seems very dangerous. * Also /usr/lib/ruby/1.8/i386-linux/defines.h seems dangerous. -- 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