From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [UNSURE]Re: [PATCH 01/13] perl: new package v6
Date: Sat, 22 Sep 2012 05:38:51 +0200 [thread overview]
Message-ID: <20120922053851.25e90b1e@skate> (raw)
In-Reply-To: <CAB0FRsu9fyH_9+QtLgbXrTMA=zqmR1io5vCO6cEYFQfwgtNcxw@mail.gmail.com>
Dear Fran?ois Perrad,
On Fri, 21 Sep 2012 23:29:57 +0200, Fran?ois Perrad wrote:
> > Please explain in more details what this means, and whether this
> > patch has a chance of being upstream or not.
> >
> > Also, it seems to imply that the build process of Perl would require
> > Qemu. Or, Qemu is not amongst the requirements of Buildroot, and
> > there is no host-qemu package in the dependencies of your package
> > (and we don't have a host-qemu package). Yann E. Morin has posted a
> > target package for qemu a while ago, but it hasn't been merged so
> > far.
> >
> > That said, even so it is certainly possible to create a host-qemu
> > package, I am a bit annoyed to see the build process of a package
> > requiring qemu. Is there really no other way? I guess it uses qemu
> > to run a bunch of tests on the target system. Is there no way to
> > provide pre-defined values for those tests, on a per-architecture
> > basis, rather than running those test applications in Qemu?
> >
>
> Qemu is not used for test, but for the configure and build step.
> Perl is 'bootstrapped' by miniperl (a perl without any extension
> module), during the build process, miniperl is running via qemu (user
> mode).
>
> Perl doesn't use the "standard" autotools, Perl has a limited support
> for cross-compiling
> where the target executables are running on a real target through a
> ssh connection.
> The use of qemu (user mode) removes the need of a real target and the
> ssh connection.
>
> BR is shipped with 12 configs/qemu_*_defconfig files,
> many users have already run 'apt-get install qemu-kvm
> qemu-kvm-extras'.
>
> A host-qemu package (with the latest version) could be nice in the
> future.
I understand the need for Qemu in the Perl context (even though I
generally don't understand why they can't bring normal
cross-compilation support in the Perl build system), but the solution
you're proposing cannot be accepted as is. We need to either:
* Make qemu a hard requirement to run Buildroot, and in that case, add
a check for it in support/dependencies/dependencies.sh. Since it is
such an unusual dependency, we may special case it, and add it as a
dependency only if the Perl package is selected, a bit like we
already do for the version control tools (that are only requested as
dependencies if at least one package needing a given version control
tool is selected)
* Add a host-qemu package. We can quite certainly find a commonality
here with Yann's work on Qemu.
At the moment, your package will break on auto-builders, because they
don't necessarily have Qemu installed.
Best regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2012-09-22 3:38 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-08 12:28 [Buildroot] [PATCH 01/13] perl: new package v6 Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 02/13] perl: add an option for miniperl Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 03/13] perl: add DB_File Francois Perrad
2012-09-20 19:57 ` Thomas Petazzoni
2012-09-21 18:26 ` François Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 04/13] gdbm: new package Francois Perrad
2012-09-12 5:30 ` Arnout Vandecappelle
2012-09-08 12:28 ` [Buildroot] [PATCH 05/13] perl: add GDBM_File Francois Perrad
2012-09-20 19:59 ` Thomas Petazzoni
2012-09-21 18:36 ` François Perrad
2012-09-21 19:01 ` Thomas Petazzoni
2012-09-24 8:43 ` Arnout Vandecappelle
2012-09-08 12:28 ` [Buildroot] [PATCH 06/13] cpanminus: new package v3 Francois Perrad
2012-09-20 20:04 ` Thomas Petazzoni
2012-09-20 21:35 ` Arnout Vandecappelle
2012-09-21 18:25 ` François Perrad
2012-09-21 19:02 ` Thomas Petazzoni
2012-09-08 12:28 ` [Buildroot] [PATCH 07/13] microperl: build extensions at build time Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 08/13] microperl: bump version to 5.14.2 Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 09/13] microperl: improve configure step on 64bits platform Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 10/13] microperl: install more modules by default Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 11/13] microperl: bump version to 5.16.1 Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 12/13] microperl: remove host-microperl Francois Perrad
2012-09-08 12:28 ` [Buildroot] [PATCH 13/13] microperl: mark as DEPRECATED Francois Perrad
2012-09-20 20:22 ` Thomas Petazzoni
2012-09-21 18:53 ` François Perrad
2012-09-21 18:59 ` Thomas Petazzoni
2012-09-23 15:11 ` François Perrad
2012-09-23 15:25 ` Thomas Petazzoni
2012-09-20 19:56 ` [Buildroot] [PATCH 01/13] perl: new package v6 Thomas Petazzoni
2012-09-21 21:29 ` [Buildroot] [UNSURE]Re: " François Perrad
2012-09-22 3:38 ` Thomas Petazzoni [this message]
2012-09-24 20:20 ` [Buildroot] " Arnout Vandecappelle
2012-09-25 17:17 ` [Buildroot] [UNSURE]Re: " François 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=20120922053851.25e90b1e@skate \
--to=thomas.petazzoni@free-electrons.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.