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=481564 --- Comment #2 from Tim Jackson <rpm@xxxxxxxxxx> 2009-01-27 04:19:24 EDT --- (In reply to comment #1) > Can you follow the packaging source url guidelines ? Sure, I can't imagine why that's not there - must have been having a bad day or something when I originally wrote the spec. Updated. > Can you merge this 2 lines in only one ? > > mkdir -p $RPM_BUILD_ROOT%{_bindir} > install -m 755 bind-to-tinydns $RPM_BUILD_ROOT%{_bindir}/ > > something like this > install -Dp -m 755 bind-to-tinydns $RPM_BUILD_ROOT%{_bindir}/bind-to-tinydns Yes, but it's unnecessary. However I've changed it to avoid any debate. Spec URL: http://fedorapeople.org/~timj/packaging/bind-to-tinydns/bind-to-tinydns.spec SRPM URL: http://fedorapeople.org/~timj/packaging/bind-to-tinydns/bind-to-tinydns-0.4.3-2.el5.src.rpm -- 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