Bug 1020809 - Please simplify %perl_default_filter
Summary: Please simplify %perl_default_filter
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl
Version: 20
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-18 10:21 UTC by Remi Collet
Modified: 2013-10-22 11:11 UTC (History)
9 users (show)

Fixed In Version: perl-5.18.1-289.fc21
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-22 11:11:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Remi Collet 2013-10-18 10:21:00 UTC
See : https://fedorahosted.org/fpc/ticket/353

So, the %perl_default_filter macro definition could be simplyfy

-%global __provides_exclude_from %{perl_vendorarch}/auto/.*\\\\.so$|%{perl_archlib}/.*\\\\.so$|%{_docdir}
+%global __provides_exclude_from %{_docdir}

This change will be reflected in 
https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering#Perl

Comment 1 Remi Collet 2013-10-18 10:21:50 UTC
To be clear "the %perl_default_filter macro definition could be simplify in Fedora >= 20"

Comment 2 Petr Pisar 2013-10-22 06:41:40 UTC
We will see. I will do some tests and in case of success, I will implement it in F21.

Comment 3 Petr Pisar 2013-10-22 07:40:58 UTC
So it seems to work.

However I would like to sanity the filters too: To anchor the expressions and to append them instead of redefining:

-%global __provides_exclude_from %{_docdir}
-%global __requires_exclude_from %{_docdir}
-%global __provides_exclude perl\\\\(VMS|perl\\\\(Win32|perl\\\\(DB\\\\)|perl\\\\(UNIVERSAL\\\\)
-%global __requires_exclude perl\\\\(VMS|perl\\\\(Win32
+%global __provides_exclude_from %{?__provides_exclude_from:%__provides_exclude_from|}^%{_docdir}
+%global __requires_exclude_from %{?__requires_exclude_from:%__requires_exclude_from|}^%{_docdir}
+%global __provides_exclude {?__provides_exclude:%__provides_exclude|}^perl\\\\(VMS|perl\\\\(Win32|perl\\\\(DB\\\\)|^perl\\\\(UNIVERSAL\\\\)
+%global __requires_exclude {?__requires_exclude:%__requires_exclude|}^perl\\\\(VMS|^perl\\\\(Win32

Comment 4 Petr Pisar 2013-10-22 08:00:35 UTC
(In reply to Petr Pisar from comment #3)
> So it seems to work.
> 
> However I would like to sanity the filters too: To anchor the expressions
> and to append them instead of redefining:
> 
Correct new values would be:

%global __provides_exclude_from %{?__provides_exclude_from:%__provides_exclude_from|}^%{_docdir}
%global __requires_exclude_from %{?__requires_exclude_from:%__requires_exclude_from|}^%{_docdir}
%global __provides_exclude %{?__provides_exclude:%__provides_exclude|}^perl\\\\(VMS|^perl\\\\(Win32|^perl\\\\(DB\\\\)|^perl\\\\(UNIVERSAL\\\\)
%global __requires_exclude %{?__requires_exclude:%__requires_exclude|}^perl\\\\(VMS|^perl\\\\(Win32

Comment 5 Petr Pisar 2013-10-22 09:22:18 UTC
(In reply to Petr Pisar from comment #3)
> However I would like to sanity the filters too: To anchor the expressions
> and to append them instead of redefining:
> 
And this works too. I'm going to push this version. I will create automated tests and package them for Fedora to have a canary.


Note You need to log in before you can comment on or make changes to this bug.