[Bug 469811] Review Request: pdf2djvu - PDF to DjVu converter

[bugzilla at redhat.com]

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 at iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jussi.lehtola at iki.fi
         AssignedTo|nobody at fedoraproject.org    |jussi.lehtola at iki.fi
               Flag|                            |fedora-review?




--- Comment #4 from Jussi Lehtola <jussi.lehtola at iki.fi>  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.