On Sat, Mar 10, 2007 at 12:57:54PM +0200, Ville Skyttä wrote:
> Hello,
>
> Whether the perl/perl-devel split in devel (#226276) is here to stay or not,
> I'd like to make the attached change to the rpmdevtools perl spec template.
>
> Using perl(ExtUtils::MakeMaker) as the "base" build dependency is correct in
> the vast majority of cases, no matter in which package ExtUtils::MakeMaker is
> included. Packages using Module::Build should change the dep to
> perl(Module::Build) - that'll pull in ExtUtils::MakeMaker too in case it's
> needed for Module::Build's non-compat mode.
>
> Doing the build dep this way should work for all distro versions as-is without
> the need to put ugly conditionals on whether to pull in perl-devel or not or
> whatever other perl package reorganizations we might see in the
> future.
Good idea!
> I see some packagers have already started to make those unnecessary
> specfile complications/per-distro forks so it'd be good to have this
> in the spec template as well as applicable Wiki pages as soon as
> possible.
>
> Comments, objections?
Even if it seems (technically) redunant, I would recommend to leave
the explicit perl dependency in it, as a documentation/educational
entity.
> Index: spectemplate-perl.spec
> ===================================================================
> RCS file: /cvs/fedora/fedora-rpmdevtools/spectemplate-perl.spec,v
> retrieving revision 1.14
> diff -u -r1.14 spectemplate-perl.spec
> --- spectemplate-perl.spec 20 Jul 2006 17:46:34 -0000 1.14
> +++ spectemplate-perl.spec 10 Mar 2007 10:52:17 -0000
> @@ -10,7 +10,8 @@
> BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
>
> BuildArch:
> -BuildRequires: perl
> +# Correct for lots of packages, other common choices include eg. Module::Build
> +BuildRequires: perl(ExtUtils::MakeMaker)
> Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
>
> %description
--
Axel.Thimm at ATrpms.net
Attachment:
pgpiml6VaerHz.pgp
Description: PGP signature
-- Fedora-packaging mailing list Fedora-packaging@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-packaging
