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=486390 --- Comment #3 from Hedayat Vatankhah <hedayat@xxxxxxxx> 2009-03-16 09:14:49 EDT --- Thank you for reviewing the package. * BR: - Latex related cmake files look for "convert" utility of ImageMagic and will fail if it does not exist. I'm not sure if it is actually used during documentation generation. - OK, will be removed * Requires: - Yes, each distro (F9, F10 and F11) use a different name! I'll either add conditional statements for each distro or a file level dependency. I think package dependency is preferred so I'll add that. (any suggestions?) - Yes. Ruby is highly integrated into simspark and is used to glue different plugins and subsystems together to create a functional part. So, I think there is no need to create subpackages for it. (to be more specific, zeitgeist library requires ruby, and other main libraries(oxygen and kerosin) require zeitgeist). * Build: - Fixed. * Cflags: - sorry for that :( these issues will be fixed. * ldconfig: - Yes, it is the default behavior of the upstream package. But, while most of the libraries are plugins, some of them are libraries which executable files will be linked to. Should I create a config file in /etc/ld.so.conf.d for /usr/lib{64}/simspark ?! * OK, I'll add the conflicts statement. -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review