https://bugzilla.redhat.com/show_bug.cgi?id=1011793



--- Comment #2 from Mathieu Bridon <[email protected]> ---
(In reply to Christopher Meng from comment #1)
> 2. Missing %config(noreplace) with %{_sysconfdir}/%{name}.config

Oops, good catch!

> 3. BuildRequires: /usr/bin/perl
> 
> just
> 
> BuildRequires: perl

I hesitated on that one, not sure which one is better, but ok.

> 4.
> http://downloads.sourceforge.net/project/docx2txt/docx2txt/v1.2/docx2txt-1.2.
> tgz
> 
> is the real URL.

Nope: https://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net

-----

Spec URL: http://bochecha.fedorapeople.org/packages/docx2txt.spec
SRPM URL: http://bochecha.fedorapeople.org/packages/docx2txt-1.2-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=wIcCAbnnav&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to