From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] MAKEALL
Date: Wed, 19 Oct 2011 10:57:34 +0200 [thread overview]
Message-ID: <20111019085734.DC322140ABD2@gemini.denx.de> (raw)
In-Reply-To: <CALButC+_q+bfZuMJyXjn-GbC1XFWjvGeD-go85LVUvp4S16WnA@mail.gmail.com>
Dear Graeme Russ,
In message <CALButC+_q+bfZuMJyXjn-GbC1XFWjvGeD-go85LVUvp4S16WnA@mail.gmail.com> you wrote:
>
> > I see no clean way to implement this - ok, we could provide an
> > external tool / data base that maps boards or SoC names to
> > CROSS_COMPILE/ARCH/PATH settings, which each user has to configure for
> > his own set of tool chain settings.
>
> IMHO, for running MAKEALL, I see no problem with this. If we had a
> 'toolchains.cfg' file which could be a format like:
>
> #ARCH SOC BOARD TOOLCHAIN
> x86 sc520 - /path/to/gcc
>
> This would give new developers a head-up as to what the defacto toolchains
> are
This is IMO not flexible enough. The longer I think the more I like
the idea of providing a hook (script file name) that gets passed all
relevant parameters as arguments (target name, arch, cpu, board name,
vendor, soc, options) and that gets sourced by MAKEALL, so it is able
to modify any environment variables (PATH, ARCH, CROSS_COMPILE,
eventually more) as needed.
> We can then have 'toolchains.cfg.local' which is added to .gitignore so
> individual users can override the toolchain. But all patch submissions
> must pass MAKEALL without using toolchains.cfg.local (something like
> 'MAKEALL --no-custom-toolchains'. The first thing MAKEALL should do is
> scan toolchains.cfg (and toolchains.cfg.local if required) for each
> selected arch and check that each toolchain is available and spit out
> 'toolchain not available' warnings.
As mentioned before: a decision based on ARCH alone is not sufficient;
see for example the issues with the big endian ARM boards (PXA).
> All we need to do then is setup our build machines to do an automated
> git-pull and MAKEALL
:-)
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A stone was placed at a ford in a river with the inscription:
"When this stone is covered it is dangerous to ford here."
next prev parent reply other threads:[~2011-10-19 8:57 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-18 6:23 [U-Boot] [STATUS] "Quality" of patches / testing Wolfgang Denk
2011-10-18 6:51 ` Simon Schwarz
2011-10-18 9:22 ` Andreas Bießmann
2011-10-18 9:44 ` Wolfgang Denk
2011-10-18 13:05 ` Jason
2011-10-18 13:10 ` Jason
2011-10-18 13:13 ` Simon Schwarz
2011-10-18 13:49 ` Jason
2011-10-18 15:37 ` Jason
2011-10-18 16:12 ` Mike Frysinger
2011-10-18 13:36 ` Andreas Bießmann
2011-10-18 15:55 ` Jason
2011-10-18 14:05 ` Simon Glass
2011-10-18 16:59 ` Anton Staaf
2011-10-18 20:23 ` Wolfgang Denk
2011-10-20 0:39 ` Simon Glass
2011-10-20 15:32 ` Wolfgang Denk
2011-10-18 10:24 ` Lukasz Majewski
2011-10-18 11:02 ` Wolfgang Denk
2011-10-18 9:34 ` Wolfgang Denk
2011-10-18 13:05 ` Simon Schwarz
2011-10-18 8:49 ` Lukasz Majewski
2011-10-18 17:01 ` [U-Boot] MAKEALL Mike Frysinger
2011-10-18 17:39 ` Simon Glass
2011-10-18 17:58 ` Tom Rini
2011-10-18 18:11 ` Mike Frysinger
2011-10-18 18:31 ` Mike Frysinger
2011-10-18 18:54 ` Tom Rini
2011-10-18 19:49 ` Mike Frysinger
2011-10-18 20:07 ` Wolfgang Denk
2011-10-18 20:14 ` Mike Frysinger
2011-10-18 20:47 ` Wolfgang Denk
2011-10-18 20:55 ` Mike Frysinger
2011-10-18 21:30 ` Simon Glass
2011-10-18 22:21 ` Mike Frysinger
2011-10-19 11:36 ` Albert ARIBAUD
2011-10-19 14:25 ` Mike Frysinger
2011-10-19 19:57 ` Wolfgang Denk
2011-10-18 21:50 ` Wolfgang Denk
2011-10-18 22:18 ` Mike Frysinger
2011-10-18 22:33 ` Graeme Russ
2011-10-19 7:12 ` Andreas Bießmann
2011-10-19 8:57 ` Wolfgang Denk [this message]
2011-10-18 17:16 ` [U-Boot] [STATUS] "Quality" of patches / testing Anton Staaf
2011-10-18 17:44 ` Albert ARIBAUD
2011-10-18 18:07 ` Anton Staaf
2011-10-20 9:25 ` Detlev Zundel
-- strict thread matches above, loose matches on Subject: below --
2014-01-03 23:05 [U-Boot] MAKEALL York Sun
2014-01-04 9:21 ` Wolfgang Denk
2014-01-08 16:54 ` Simon Glass
2014-02-12 9:55 ` Albert ARIBAUD
2014-02-12 10:42 ` Masahiro Yamada
2014-02-16 4:57 ` Simon Glass
2014-02-19 14:04 ` Masahiro Yamada
2014-01-29 7:13 JYOTI DUBEY
2014-01-29 7:24 ` Anatolij Gustschin
[not found] ` <CAE0zQku6s7L=C87CjW6wTmorttnPbeXEgHg2eBx2TcV2hvBysw@mail.gmail.com>
2014-01-29 7:45 ` Anatolij Gustschin
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=20111019085734.DC322140ABD2@gemini.denx.de \
--to=wd@denx.de \
--cc=u-boot@lists.denx.de \
/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