From: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] pkg-perl: refactor perl infrastructure
Date: Mon, 7 Sep 2015 13:37:00 +0100 [thread overview]
Message-ID: <55ED84EC.7090701@imgtec.com> (raw)
In-Reply-To: <1441222304-6207-1-git-send-email-francois.perrad@gadz.org>
Dear Francois Perrad
since you already sent a v2 of this series, could you please mark this
one as superseded in patchwork?
http://patchwork.ozlabs.org/patch/513641/
http://patchwork.ozlabs.org/patch/513642/
Thanks,
Vincent.
On 09/02/2015 08:31 PM, Francois Perrad wrote:
> the perl dependency of cpan module is no longer generated by scancpan,
> but added at the infrastructure level
>
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
> package/pkg-perl.mk | 8 ++++++++
> support/scripts/scancpan | 2 +-
> 2 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/package/pkg-perl.mk b/package/pkg-perl.mk
> index 6577588..a06c068 100644
> --- a/package/pkg-perl.mk
> +++ b/package/pkg-perl.mk
> @@ -39,7 +39,15 @@ PERL_RUN = PERL5LIB= $(HOST_DIR)/usr/bin/perl
>
> define inner-perl-package
>
> +# Target packages need both the perl interpreter on the target (for
> +# runtime) and the perl interpreter on the host (for
> +# compilation). However, host packages only need the perl
> +# interpreter on the host.
> +ifeq ($(4),target)
> +$(2)_DEPENDENCIES += host-perl perl
> +else
> $(2)_DEPENDENCIES += host-perl
> +endif
>
> #
> # Configure step. Only define it if not already defined by the package
> diff --git a/support/scripts/scancpan b/support/scripts/scancpan
> index c7180c8..947d248 100755
> --- a/support/scripts/scancpan
> +++ b/support/scripts/scancpan
> @@ -655,7 +655,7 @@ while (my ($distname, $dist) = each %dist) {
> # the auth part is not used, because we use $(BR2_CPAN_MIRROR)
> my($filename, $directories, $suffix) = fileparse( $path, q{tar.gz}, q{tgz} );
> $directories =~ s|/$||;
> - my $dependencies = join q{ }, qw( = perl ),
> + my $dependencies = join q{ }, qw( = ),
> map( { q{host-} . fsname( $_ ); } sort @{$deps_build{$distname}} ),
> map( { fsname( $_ ); } sort @{$deps_runtime{$distname}} );
> my $host_dependencies = join q{ }, qw( = ),
>
prev parent reply other threads:[~2015-09-07 12:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-02 19:31 [Buildroot] [PATCH 1/2] pkg-perl: refactor perl infrastructure Francois Perrad
2015-09-02 19:31 ` [Buildroot] [PATCH 2/2] pkg-perl: remove the dependency generated by scancpan Francois Perrad
2015-09-02 19:59 ` Thomas Petazzoni
2015-09-04 12:18 ` François Perrad
2015-09-07 12:37 ` Vicente Olivert Riera [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55ED84EC.7090701@imgtec.com \
--to=vincent.riera@imgtec.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.