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: yum https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226666 pertusus@xxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pertusus@xxxxxxx ------- Additional Comments From pertusus@xxxxxxx 2007-02-14 04:19 EST ------- Issues: * Why is it a BuildRequires: python and not Buildrequires: python-devel? * it seems to me that the python-devel BuildRequires should be versionned then Requires: python >= 2.4 would be autodetected * Prereq should be replaced with Requires(preun) and so on... * Is the conflict really needed? pirut >= 1.1.4 should requires a recent enough yum version. suggestions: * use %defattr(-, root, root, -) instead of %defattr(-, root, root) * prefix plugin.conf source file by a disambiguating prefix like yum-plugin.conf * remove the -f to be notified when the file doesn't exist/change name... rm -f $RPM_BUILD_ROOT/%{_sysconfdir}/yum/yum.conf * there is an occurence of /etc hardcoded in the specfile, it could be %_sysconfdir. However the upstream Makefiles and programs have many paths hardcoded so this is not really an issue. (There is also a /var and some /usr, but I guess they are also hardcoded in the package). * It seems to me that BuildArch is more used that BuildArchitectures * There is no need of / after $RPM_BUILD_ROOT -- 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