Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 6/8] perl-moo: new package
Date: Mon, 8 Oct 2018 21:51:29 +0200	[thread overview]
Message-ID: <20181008215129.6ff9da6f@windsurf> (raw)
In-Reply-To: <20181008193646.13046-7-francois.perrad@gadz.org>

Hello,

On Mon,  8 Oct 2018 21:36:44 +0200, Francois Perrad wrote:

> +PERL_MOO_VERSION = 2.003004
> +PERL_MOO_SOURCE = Moo-$(PERL_MOO_VERSION).tar.gz
> +PERL_MOO_SITE = $(BR2_CPAN_MIRROR)/authors/id/H/HA/HAARG
> +PERL_MOO_DEPENDENCIES = \
> +	perl-class-method-modifiers \
> +	perl-devel-globaldestruction \
> +	perl-module-runtime \
> +	perl-role-tiny \
> +	perl-sub-quote

Please remove all those build-time dependencies, and try to build
without them. I.e do a build with BR2_PACKAGE_PERL_MOO enabled, but run
just "make perl-moo". If it works, then those dependencies are only
runtime dependencies, not build time ones.

It is much easier to detect missing build dependencies (thanks to build
testing, either on your side or in the autobuilders) than to detect
extraneous build dependencies and remove them.

So I think scancpan should only add dependencies on host packages
(which are obviously needed at build time) and not add any dependency
on target packages. And then build testing is needed to manually add
the build-time dependencies that are needed, if any.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2018-10-08 19:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-08 19:36 [Buildroot] [PATCH 0/8] Moo & friends Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 1/8] perl-class-method-modifiers: new package Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 2/8] perl-role-tiny: " Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 3/8] perl-sub-exporter-progressive: " Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 4/8] perl-sub-quote: " Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 5/8] perl-devel-globaldestruction: " Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 6/8] perl-moo: " Francois Perrad
2018-10-08 19:51   ` Thomas Petazzoni [this message]
2018-10-09 19:43     ` Arnout Vandecappelle
2018-10-08 19:36 ` [Buildroot] [PATCH 7/8] perl-exporter-tiny: " Francois Perrad
2018-10-08 19:36 ` [Buildroot] [PATCH 8/8] perl-type-tiny: " Francois Perrad

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=20181008215129.6ff9da6f@windsurf \
    --to=thomas.petazzoni@bootlin.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox