From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v10 14/14] RFC: Deprecate MAKEALL
Date: Wed, 22 Oct 2014 08:16:52 -0700 [thread overview]
Message-ID: <5447CA64.3050901@freescale.com> (raw)
In-Reply-To: <CAPnjgZ1VJj17V0_Mopff9O=AKL+6GtPCeF_R-PEQmyokYn5ksw@mail.gmail.com>
On 10/22/2014 08:11 AM, Simon Glass wrote:
> Hi Wolfgang,
>
> On 22 October 2014 02:33, Wolfgang Denk <wd@denx.de> wrote:
>> Dear Tom,
>>
>> In message <20141014081724.GF25506@bill-the-cat> you wrote:
>>>
>>> No, not yet. I am going to mention in the release notes that we're
>>> going to strongly start thinking about deleting MAKEALL. We can pick up
>>> the deprecation patch early in the next merge window.
>>
>> Can we please keep at least some script (as a wrapper around
>> buildman?) that keeps the old user interface in place?
>>
>> I frequently use "git bisect run MAKEALL <board> ...", and it would
>> be nice if we could keep this working even in newer versions of the
>> code.
>>
>> Or is there a similar alternative command that works with identical
>> parameters for - say - all versions of the last two years or so?
>
> Unfortunately there is not really an equivalent, and buildman only recently
>
> - sets its return code correctly
> - supports building the current tree (previously it required a branch
> name for the commit to build)
>
> During development of buildman/patman I used to keep a separate tree.
> Then you can run the latest buildman from that tree using you current
> directory for the bisect.
>
> I could perhaps look at adjusting MAKEALL to call buildman if that
> solution isn't good enough (this will work assuming that buildman has
> been set up with toolchains, etc.). Still I think a deprecation
> warning is a good start.
>
Simon,
While you work on buildman, please consider to regenerate boards.cfg for each
build. If one patch in a series adds a board, there will be a false error for
patches before it.
York
next prev parent reply other threads:[~2014-10-22 15:16 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-28 15:43 [U-Boot] [PATCH v10 02/14] patman: Fix indentation in terminal.py Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 03/14] patman: Correct unit tests to run correctly Simon Glass
2014-09-06 16:42 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 04/14] patman: Remove the -a option Simon Glass
2014-09-06 16:42 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 05/14] patman: Use --no-pager' to stop git from forking a pager Simon Glass
2014-09-06 16:43 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 06/14] patman: Avoid changing the order of tags Simon Glass
2014-08-30 22:19 ` Masahiro YAMADA
2014-09-06 16:44 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 07/14] buildman: Set the return code to indicate build result Simon Glass
2014-09-06 16:44 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 08/14] buildman: Allow make-flags variables to include '-' and '_' Simon Glass
2014-09-06 16:45 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 09/14] buildman: Implement an option to exclude boards from the build Simon Glass
2014-09-06 16:45 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 10/14] buildman: Remove the directory prefix from each error line Simon Glass
2014-09-06 16:45 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 11/14] buildman: Add an option to show which boards caused which errors Simon Glass
2014-09-02 12:36 ` [U-Boot] [U-Boot, v10, " Tom Rini
2014-09-02 19:17 ` Simon Glass
2014-09-06 16:46 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 12/14] buildman: Separate out display of warnings and errors Simon Glass
2014-09-06 16:46 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 13/14] patman: Fix detection of git version Simon Glass
2014-09-06 16:46 ` Simon Glass
2014-08-28 15:43 ` [U-Boot] [PATCH v10 14/14] RFC: Deprecate MAKEALL Simon Glass
2014-10-14 7:38 ` Simon Glass
2014-10-14 8:17 ` Tom Rini
2014-10-22 8:33 ` Wolfgang Denk
2014-10-22 15:11 ` Simon Glass
2014-10-22 15:16 ` York Sun [this message]
2014-10-22 16:09 ` Tom Rini
2014-10-22 16:59 ` Wolfgang Denk
2014-10-22 17:19 ` Tom Rini
2014-10-22 17:37 ` Wolfgang Denk
2014-10-22 17:51 ` Simon Glass
2014-10-22 17:56 ` Wolfgang Denk
2014-10-22 17:59 ` Simon Glass
2014-10-22 18:13 ` Wolfgang Denk
2014-10-23 3:14 ` Simon Glass
2015-07-14 17:56 ` [U-Boot] [U-Boot,v10,14/14] " Tom Rini
2014-09-06 16:41 ` [U-Boot] [PATCH v10 02/14] patman: Fix indentation in terminal.py Simon Glass
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=5447CA64.3050901@freescale.com \
--to=yorksun@freescale.com \
--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 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.