Re: Proposed F12 perl cleanups
On 08/15/2009 09:00 PM, Tom "spot" Callaway wrote: Out of the thread on p5p, I'd like to propose the following changes for F-12: * Rename "perl-core" to "perl" * Rename "perl" to "perl-minimal" The biggest change here is that there are still packages which "Require: perl", usually to specify a specific minimal version. Here is a list of rawhide packages which do this: With this change, these packages will have a larger installation footprint, unless they're cleaned up. Instead of: Requires: perl or Requires: perl> 5.6.0 They should either have: 1. If they're version dependent, they should have Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version)) 2. If they're not, they could either accept the larger install footprint, or switch to: Requires: perl-minimal Thoughts on this? -1 Rationale: a) As others already pointed out, many of these "requires:" are automatically added => Part of the issue is inside of perl.req etc. and not inside of the perl package b) "Requires: perl > " is intuitively understandable ("A natural solution" ), perl(:MODULE_COMPAT_..) isn't. c) R: perl(:MODULE_COMPAT_..) is not a natural replacement for "R: perl > XXX". They have different semantics. R: perl(:MODULE_COMPAT_..) basically refers to module search paths, while "R: perl > XXX", can refer to many to subjects and may originiate from other issues, such as changes of the perl language, bugs a perl module author might have encountered, etc. Example: # rpm -q -requires help2man | grep perl /usr/bin/perl perl >= 0:5.005 perl(Getopt::Long) perl(POSIX) perl(Text::Tabs) perl(locale) perl(strict) * There is no explicit "requires: perl >= ..." inside of this package's spec, * The package actually doen't require a "perl-package >= 5.005", it requires "The Perl Language > 5.005" * The package doen't install any module, so "perl(:MODULE_COMPAT_..)" isn't right either. That said, I think, we need another rpm tag, besides "perl" and "perl(:MODULE_COMPAT..)" to denote "The Perl Language" version and let rpm add this in perl.req instead of "perl > XXX". Ralf -- Fedora Extras Perl SIG http://www.fedoraproject.org/wiki/Extras/SIGs/Perl Fedora-perl-devel-list mailing list Fedora-perl-devel-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list
Re: Proposed F12 perl cleanups
> * Rename "perl" to "perl-minimal" +1 > They should either have: > > 1. If they're version dependent, they should have > > Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo > $version)) +1 Marcela -- Fedora Extras Perl SIG http://www.fedoraproject.org/wiki/Extras/SIGs/Perl Fedora-perl-devel-list mailing list Fedora-perl-devel-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list
Re: Proposed F12 perl cleanups
On Sat, Aug 15, 2009 at 12:00 PM, Tom "spot" Callaway wrote: > Out of the thread on p5p, I'd like to propose the following changes for > F-12: > > * Rename "perl-core" to "perl" > * Rename "perl" to "perl-minimal" > > The biggest change here is that there are still packages which "Require: > perl", usually to specify a specific minimal version. Here is a list of > rawhide packages which do this: [...snip...] > > They should either have: > > 1. If they're version dependent, they should have > > Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo > $version)) > They should all require this; this also indicates what version of Perl this package was built/compiled against, rather than a minimum version of Perl that this module requires to function. (that is, the package requires version x.y.z as it's installed in /usr/lib/perl5/x.y.z/). > 2. If they're not, they could either accept the larger install > footprint, or switch to: > > Requires: perl-minimal > Iain is right It looks like this is being auto-added.[1] We'd have to fix that in the rpm macros -- and maybe get the filtering macros in there, as I don't seem to be getting much response to https://bugzilla.redhat.com/show_bug.cgi?id=516240 -- then rebuild the impacted packages. In either case, I'm +1 to the renaming (and am somewhat to blame for the -core name to begin with... sigh). I think we should probably take a look at the package descriptions and summaries as part of this as well. -Chris [1] And, can I just say again how erratic our Perl rpm autoreq/prov bits are? In practice Perl metadata like this isn't kept in scripts/modules themselves, and some bits (like Moose traits/roles) are completely masked from it. If we want accurate rpm metadata, we ought to be generating our Perl req/prov out of META.yml and its kind. -- Chris Weyl Ex astris, scientia -- Fedora Extras Perl SIG http://www.fedoraproject.org/wiki/Extras/SIGs/Perl Fedora-perl-devel-list mailing list Fedora-perl-devel-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list
Re: Proposed F12 perl cleanups
* Tom spot Callaway [15/08/2009 22:24] : > > Out of the thread on p5p, I'd like to propose the following changes for > F-12: > > * Rename "perl-core" to "perl" > * Rename "perl" to "perl-minimal" > Thoughts on this? +1 for your proposal. Emmanuel -- Fedora Extras Perl SIG http://www.fedoraproject.org/wiki/Extras/SIGs/Perl Fedora-perl-devel-list mailing list Fedora-perl-devel-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list
Re: Proposed F12 perl cleanups
On Sat, Aug 15, 2009 at 9:00 PM, Tom "spot" Callaway wrote: > The biggest change here is that there are still packages which "Require: > perl", usually to specify a specific minimal version. Here is a list of > rawhide packages which do this: [big snip] Many of those packages don't actually have an explicit Requires: perl. It's being added automatically when perl.req finds things like "use 5.006". -- Iain. -- Fedora Extras Perl SIG http://www.fedoraproject.org/wiki/Extras/SIGs/Perl Fedora-perl-devel-list mailing list Fedora-perl-devel-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-perl-devel-list
Proposed F12 perl cleanups
Out of the thread on p5p, I'd like to propose the following changes for F-12: * Rename "perl-core" to "perl" * Rename "perl" to "perl-minimal" The biggest change here is that there are still packages which "Require: perl", usually to specify a specific minimal version. Here is a list of rawhide packages which do this: abicheck-1.2-23.src.rpm acheck-0.5.1-3.fc12.src.rpm akmods-0.3.6-3.fc12.src.rpm amanda-2.6.0p2-12.fc12.src.rpm amavisd-new-2.6.2-4.fc12.src.rpm anjuta-2.27.3.0-2.fc12.src.rpm autoconf213-2.13-20.fc12.src.rpm autoconf-2.63-3.fc12.src.rpm automake-1.11-3.fc12.src.rpm automake15-1.5-27.src.rpm automake16-1.6.3-18.src.rpm automake17-1.7.9-13.src.rpm awstats-6.9-3.fc12.src.rpm bogofilter-1.2.0-2.fc12.src.rpm bontmia-0.14-5.fc12.src.rpm bugzilla-3.4.1-1.fc12.src.rpm calamaris-2.59-3.fc12.src.rpm cdrkit-1.1.9-9.fc12.src.rpm checkgmail-1.13-5.svn20080730.fc12.src.rpm clc-intercal-0-0.2.1._94._2.fc2.src.rpm clearsilver-0.10.5-7.fc12.src.rpm clusterssh-3.26-2.fc12.src.rpm condor-7.2.1-2.fc12.src.rpm conmux-0.0-9.493svn.fc12.src.rpm convmv-1.14-2.fc12.src.rpm cook-2.32-3.fc12.src.rpm crypto-utils-2.4.1-22.src.rpm cvsutils-0.2.5-3.fc12.src.rpm cvsweb-3.0.6-9.fc12.src.rpm cyrus-imapd-2.3.14-3.fc12.src.rpm dap-server-3.9.3-2.fc12.src.rpm dayplanner-0.10-2.fc12.src.rpm dbxml-perl-2.0040016-2.fc12.src.rpm ddclient-3.7.3-3.fc12.src.rpm dnssec-tools-1.5-3.fc12.src.rpm dvdrip-0.98.10-2.fc11.src.rpm eb-4.3.2-3.fc12.src.rpm eclipse-epic-0.6.35-2.fc12.src.rpm EekBoek-1.04.04-2.fc12.src.rpm emacs-23.1-1.fc12.src.rpm foomatic-4.0.2-4.fc12.src.rpm freenx-server-0.7.3-15.fc12.src.rpm fvwm-2.5.26-4.fc12.src.rpm git-1.6.4-1.fc12.src.rpm GraphicsMagick-1.3.6-2.fc12.src.rpm grepmail-5.3033-6.fc12.src.rpm gresolver-0.0.5-6.fc12.src.rpm GtkAda-2.10.2-2.fc11.src.rpm gtkpod-0.99.14-3.fc12.src.rpm hdrprep-0.1.2-6.fc12.src.rpm help2man-1.36.4-5.fc12.src.rpm httpd-2.2.11-10.src.rpm i2c-tools-3.0.2-4.fc12.src.rpm ifm-5.1-10.fc12.src.rpm ikiwiki-3.1415-2.fc12.src.rpm ImageMagick-6.5.4.7-3.fc12.src.rpm inn-2.5.0-4.fc12.src.rpm java-1.5.0-gcj-1.5.0.0-29.fc12.src.rpm JSDoc-1.10.2-8.fc12.src.rpm kcbench-data-0.1-5.src.rpm kdelibs3-3.5.10-13.fc12.src.rpm kdesdk-4.3.0-1.fc12.src.rpm kdevelop-3.5.4-5.fc12.src.rpm ksplice-0.9.7-4.fc12.src.rpm lagan-2.0-6.fc12.src.rpm latex2html-2008-3.fc12.src.rpm lcdproc-0.5.3-3.fc12.src.rpm linuxdoc-tools-0.9.65-2.fc12.src.rpm llvm-2.5-3.fc12.src.rpm llvm-2.5-3.fc12.src.rpm lm_sensors-3.1.1-3.fc12.src.rpm mgetty-1.1.36-4.fc12.src.rpm mhonarc-2.6.16-7.fc12.src.rpm migrationtools-47-4.fc12.src.rpm mimedefang-2.67-2.fc12.src.rpm mirrormanager-1.2.11-2.fc12.src.rpm mod_perl-2.0.4-9.src.rpm mon-1.2.0-5.fc12.src.rpm mrtg-2.16.2-4.fc12.src.rpm mtd-utils-1.2.0-3.fc12.src.rpm munin-1.2.6-10.fc12.src.rpm mysqlreport-3.5-4.fc12.src.rpm mytop-1.6-4.fc12.src.rpm nagios-plugins-1.4.13-16.fc12.src.rpm namazu-2.0.19-3.fc12.src.rpm NaturalDocs-1.4-4.fc12.src.rpm netatalk-2.0.4-2.fc12.src.rpm netpbm-10.35.64-1.fc12.src.rpm net-snmp-5.4.2.1-14.fc12.src.rpm nginx-0.7.61-1.fc12.src.rpm openoffice.org-3.1.1-16.1.fc12.src.rpm openssl-0.9.8k-7.fc12.src.rpm parrot-1.4.0-8.fc12.src.rpm pdsh-2.18-3.fc12.src.rpm pem-0.7.7-2.fc12.src.rpm perl-accessors-1.01-2.fc12.src.rpm perl-Acme-Damn-0.04-2.fc12.src.rpm perl-Algorithm-CheckDigits-0.50-3.fc12.src.rpm perl-Algorithm-CurveFit-1.03-4.fc12.src.rpm perl-Algorithm-Dependency-1.110-2.fc12.src.rpm perl-Algorithm-FastPermute-0.999-4.fc12.src.rpm perl-Alien-SeleniumRC-1.00-2.fc12.src.rpm perl-Apache-DBI-Cache-0.08-3.fc12.src.rpm perl-App-Cmd-0.203-3.fc12.src.rpm perl-Array-Compare-1.17-2.fc12.src.rpm perl-asa-0.02-4.fc12.src.rpm perl-Authen-Captcha-1.023-4.fc12.src.rpm perl-Authen-DigestMD5-0.04-7.fc12.src.rpm perl-Authen-Krb5-1.7-7.fc12.src.rpm perl-Authen-Krb5-Admin-0.11-4.fc12.src.rpm perl-autobox-2.55-3.fc12.src.rpm perl-AutoXS-Header-1.02-2.fc12.src.rpm perl-Best-0.12-2.fc12.src.rpm perl-B-Hooks-EndOfScope-0.08-2.fc12.src.rpm perl-B-Hooks-OP-Check-0.17-2.fc12.src.rpm perl-bioperl-run-1.6.1-2.fc12.src.rpm perl-boolean-0.20-4.fc12.src.rpm perl-Boulder-1.30-7.fc12.src.rpm perl-BSD-Resource-1.29.03-1.fc12.src.rpm perl-Business-CreditCard-0.30-3.fc12.src.rpm perl-Business-Hours-0.09-3.fc12.src.rpm perl-B-Utils-0.07-4.fc12.src.rpm perl-Cache-2.04-5.fc12.src.rpm perl-Cache-FastMmap-1.34-2.fc12.src.rpm perl-Calendar-Simple-1.20-3.fc12.src.rpm perl-Capture-Tiny-0.05-2.fc12.src.rpm perl-Carp-Assert-0.20-4.fc12.src.rpm perl-Catalyst-Plugin-CGI-Untaint-0.05-6.fc12.src.rpm perl-Catalyst-Plugin-StackTrace-0.10-2.fc12.src.rpm perl-CGI-Application-Plugin-Stream-2.10-2.fc12.src.rpm perl-CGI-FormBuilder-3.0501-7.fc12.src.rpm perl-CGI-Prototype-0.9053-7.fc12.src.rpm perl-CGI-Simple-1.108-2.fc12.src.rpm perl-Chatbot-Eliza-1.04-6.fc12.src.rpm perl-Class-Accessor-0.31-6.fc12.src.rpm perl-Class-Adapter-1.05-3.fc12.src.rpm perl-Class-Autouse-1.29-6.fc12.src.rpm perl-Class-C3-XS-0.11-2.fc12.src.rpm perl-Class-Date-1.1.9-6.fc12.src.rpm perl-Class-DBI-3.0.17-5.fc12.src.rpm perl-Class-DBI-AsForm-2.42-9.fc12.s