[Buildroot] [pkg-perl infra 07/12] host-libmodule-build-perl: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Nov 20 17:21:37 UTC 2013
Dear Francois Perrad,
On Wed, 20 Nov 2013 18:01:55 +0100, Francois Perrad wrote:
> the version coming with host-perl is too old
>
> Signed-off-by: Francois Perrad <francois.perrad at gadz.org>
> ---
> package/cpan/Config.in | 1 +
> package/cpan/libmodule-build-perl/Config.in | 6 ++++++
> .../libmodule-build-perl/libmodule-build-perl.mk | 16 ++++++++++++++++
> scancpan | 3 +++
> 4 files changed, 26 insertions(+)
> create mode 100644 package/cpan/libmodule-build-perl/Config.in
> create mode 100644 package/cpan/libmodule-build-perl/libmodule-build-perl.mk
>
> diff --git a/package/cpan/Config.in b/package/cpan/Config.in
> index bb98dc3..c821064 100644
> --- a/package/cpan/Config.in
> +++ b/package/cpan/Config.in
> @@ -1 +1,2 @@
> +source "package/cpan/libmodule-build-perl/Config.in"
> source "package/cpan/libxml-parser-perl/Config.in"
> diff --git a/package/cpan/libmodule-build-perl/Config.in b/package/cpan/libmodule-build-perl/Config.in
> new file mode 100644
> index 0000000..dc63312
> --- /dev/null
> +++ b/package/cpan/libmodule-build-perl/Config.in
> @@ -0,0 +1,6 @@
> +config BR2_PACKAGE_LIBMODULE_BUILD_PERL
> + bool "libmodule-build-perl"
> + depends on BR2_HOST_ONLY
I think this BR2_HOST_ONLY thing is stupid. I know we have it in
Buildroot, but we should just kill it and remove the useless Config.in
files.
> + help
> + Build and install Perl modules
> +
> diff --git a/package/cpan/libmodule-build-perl/libmodule-build-perl.mk b/package/cpan/libmodule-build-perl/libmodule-build-perl.mk
> new file mode 100644
> index 0000000..a08385b
> --- /dev/null
> +++ b/package/cpan/libmodule-build-perl/libmodule-build-perl.mk
> @@ -0,0 +1,16 @@
> +################################################################################
> +#
> +# libmodule-build-perl
> +#
> +################################################################################
> +
> +LIBMODULE_BUILD_PERL_VERSION = 0.4200
> +LIBMODULE_BUILD_PERL_AUTHOR = LEONT
> +LIBMODULE_BUILD_PERL_SOURCE = Module-Build-$(LIBMODULE_BUILD_PERL_VERSION).tar.gz
> +LIBMODULE_BUILD_PERL_SITE = $(BR2_CPAN_MIRROR)/authors/id/L/LE/LEONT/
> +#LIBMODULE_BUILD_PERL_DEPENDENCIES = perl
> +HOST_LIBMODULE_BUILD_PERL_DEPENDENCIES =
Is this needed? Maybe to remove host-perl as an automatic dependency?
In any case, having a comment on top of this line would be good.
> +LIBMODULE_BUILD_PERL_LICENSE = perl_5
> +
> +#$(eval $(perl-package))
Remove commented lines.
> +$(eval $(host-perl-package))
> diff --git a/scancpan b/scancpan
> index d8a68d6..e0ae7ba 100755
> --- a/scancpan
> +++ b/scancpan
> @@ -509,8 +509,10 @@ sub fetch {
> $dist{$name} = $result;
> my @deps_build = ();
> my @deps_runtime = ();
> + my $mb;
> foreach my $dep (@{$result->{dependency}}) {
> my $modname = ${$dep}{module};
> + $mb = 1 if $modname eq q{Module::Build};
> next if $modname eq q{perl};
> next if $modname =~ m|^Alien|;
> next if $modname =~ m|^Win32|;
> @@ -527,6 +529,7 @@ sub fetch {
> }
> fetch( $distname );
> }
> + unshift @deps_build, q{Module-Build} if $mb;
> $deps_build{$name} = \@deps_build;
> $deps_runtime{$name} = \@deps_runtime;
> }
I believe this chunk should be part of the patch adding scancpan.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list