From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 7/8] package: change ordering of steps
Date: Wed, 28 Sep 2011 16:28:07 +0200 [thread overview]
Message-ID: <20110928162807.10dcbaef@skate> (raw)
In-Reply-To: <CAAXf6LUszDVY6zf4dLx6SPknjzQEeXR-yP=zAuBw3FSWaaNkww@mail.gmail.com>
Le Wed, 28 Sep 2011 16:13:00 +0200,
Thomas De Schampheleire <patrickdepinguin+buildroot@gmail.com> a ?crit :
> I'm experiencing a problem now with Xenomai:
> * First of all, there was no dependencies to linux anymore in the
> Xenomai .mk file, which caused incorrect ordering of builds. To
> address this, I copied the approach of RTAI that Thomas added (add a
> linux dependency to xenomai-patch and add a xenomai dependency to
> linux).
Normally, the Xenomai package in package/xenomai/ does not depend on
Linux being extracted/configured/compiled. It's the opposite: the Linux
build process depends on the Xenomai package to be extracted, so that
the package can be applied.
The case of RTAI is different, because the kernel modules of RTAI are
built by package/rtai/, so RTAI depends on Linux.
Could you expand on which problem you have when the Xenomai package in
package/xenomai does not depend on Linux ?
> * after this, I stumbled upon another problem: if I removed both the
> linux and the xenomai output directories, and ran make, then linux was
> the first to come into play. After extraction of linux, the PRE_PATCH
> hook of xenomai is executed, but Xenomai is not yet extracted! Indeed,
> the current dependency tree does not enforce this.
> I can fix my scenario by changing the order of -patch and -depends :
>
> -$(1)-configure: $(1)-patch $(1)-depends \
> +$(1)-configure: $(1)-depends $(1)-patch \
>
> but I'm not sure whether this breaks other cases...
> What do you think?
Aaaah, yes, dammit. Technically speaking, it's a bit ugly that the
patch step of one package depends on the dependencies being available.
This needs some more thought I guess...
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:[~2011-09-28 14:28 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-17 20:22 [Buildroot] [pull request] Pull request for branch xenomai-rtai Thomas Petazzoni
2011-09-17 20:22 ` [Buildroot] [PATCH 1/8] linux: Add Linux Kernel extensions menu Thomas Petazzoni
2011-09-18 20:32 ` Peter Korsgaard
2011-09-17 20:22 ` [Buildroot] [PATCH 2/8] Add xenomai real-time Framework to buildroot Thomas Petazzoni
2011-09-18 20:41 ` Peter Korsgaard
2011-09-17 20:22 ` [Buildroot] [PATCH 3/8] xenomai: add support to keep or remove testsuite programs Thomas Petazzoni
2011-09-18 20:43 ` Peter Korsgaard
2011-09-17 20:22 ` [Buildroot] [PATCH 4/8] xenomai: add support to keep or remove RT-CAN utilities Thomas Petazzoni
2011-09-18 20:44 ` Peter Korsgaard
2011-09-17 20:22 ` [Buildroot] [PATCH 5/8] xenomai: add support to keep or remove Analogy utilities and libs Thomas Petazzoni
2011-09-18 20:44 ` Peter Korsgaard
2011-09-17 20:22 ` [Buildroot] [PATCH 6/8] xenomai: add support to keep or remove skins Thomas Petazzoni
2011-09-18 20:45 ` Peter Korsgaard
2011-09-19 8:24 ` Thomas De Schampheleire
2011-09-19 8:49 ` Thomas Petazzoni
2011-09-19 8:54 ` Thomas De Schampheleire
2011-09-17 20:22 ` [Buildroot] [PATCH 7/8] package: change ordering of steps Thomas Petazzoni
2011-09-18 20:47 ` Peter Korsgaard
2011-09-28 14:13 ` Thomas De Schampheleire
2011-09-28 14:28 ` Thomas Petazzoni [this message]
2011-09-28 14:44 ` Thomas De Schampheleire
2011-09-28 16:27 ` Arnout Vandecappelle
2011-11-14 15:32 ` Thomas De Schampheleire
2011-09-17 20:22 ` [Buildroot] [PATCH 8/8] Add support for the RTAI real-time extension Thomas Petazzoni
2011-09-18 20:54 ` Peter Korsgaard
2011-09-22 19:37 ` Arnout Vandecappelle
2011-09-22 19:46 ` [Buildroot] [PATCH] rtai: hide the config option Arnout Vandecappelle
2011-09-18 12:19 ` [Buildroot] [pull request] Pull request for branch xenomai-rtai Benoît Mauduit
2011-09-22 17:13 ` Arnout Vandecappelle
2011-09-19 8:21 ` Thomas De Schampheleire
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=20110928162807.10dcbaef@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox