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=469811 Jussi Lehtola <jussi.lehtola@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jussi.lehtola@xxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |jussi.lehtola@xxxxxx Flag| |fedora-review? --- Comment #4 from Jussi Lehtola <jussi.lehtola@xxxxxx> 2008-11-08 08:30:18 EDT --- MUST: Change patch1 to patch0. MUST: License is GPLv2+, not GPLv2. SHOULD: Please break BuildRequires one per line in alphabetized order. MUST: Add libjpeg-devel to BR. MUST: Modify install bit to install -p -m 644 -D {doc,%{buildroot}%{_mandir}/man1}/pdf2djvu.1 install -p -m 755 -D {,%{buildroot}%{_bindir}/}pdf2djvu to preserve time stamps. SHOULD: Capitalize first letters of sentence in %changelog. After you've corrected these I'll approve the package. -- 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