Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=916677

--- Comment #3 from Emmanuel Seyman <[email protected]> ---
(In reply to comment #1)
>
> - perl_default_filter is unnecessary

As it now stands, yes but I like to have it in all my perl-related spec files.

> - so is line 41

Removed.

> - PERL_INSTALL_ROOT could be changed to DESTDIR

Changed.

> - don't package dist.ini

Removed.

Spec URL:
http://people.parinux.org/~seyman/fedora/perl-ElasticSearch-SearchBuilder/perl-ElasticSearch-SearchBuilder.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-ElasticSearch-SearchBuilder/perl-ElasticSearch-SearchBuilder-0.18-2.fc18.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=k7JF5eoKXs&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to