public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 13/13] RFC: Deprecate MAKEALL
Date: Tue, 5 Aug 2014 11:48:31 -0700	[thread overview]
Message-ID: <53E126FF.3040503@freescale.com> (raw)
In-Reply-To: <CAPnjgZ3d6mpnQ8jgftB0gNdTtu7L9CEtoSi9CNxXm3RHznycug@mail.gmail.com>

On 08/05/2014 11:41 AM, Simon Glass wrote:
> Hi York,
> 
> On 5 August 2014 10:43, York Sun <yorksun@freescale.com> wrote:
>> On 08/05/2014 07:47 AM, Simon Glass wrote:
>>> Since buildman now includes most of the features of MAKEALL it is probably
>>> time to talk about deprecating MAKEALL.
>>>
>>> Comments welcome.
>>>
>>
>> Simon,
>>
>> I know buildman has been out for a while. I just rely too much on MAKEALL
>> automation and am reluctant to try buildman.
>>
>> Reading buildman/README, it seems buildman always builds a branch and its
>> upstream commit. I am hoping you can help me to understand how to use buildman
>> in my environment with gerrit and Jenkins. For every patch (internal
>> development), we use gerrit to conduct review and Jenkins to test. If you are
>> not familiar with either, the simplest way to understand is a script will run on
>> every commit. In the script I use MAKEALL to build all concerned (hundreds)
>> targets for _this_ commit. The dependency is maintained by gerrit. The result is
>> fed back to gerrit to show the author (and reviewers) if a failure happens.
>>
>> If using buildman and the upstream commit is always built, a great amount of
>> time will be consumed with no benefit. If you see a better way to use buildman,
>> I can give it a try.
> 
> With the v3 or v4 series you can omit the -b option and it will build
> the currently checked-out commit for the selected boards. It might be
> useful in that the problems are stored in files as well as displayed
> on the command line.

I use the .ERR file from MAKEALL. It would be nice to have similar log, in
individual files.

> 
> So perhaps you could try that?

I would like to try that.

> 
> BTW where does this information get published?
>

When we use gerrit and Jenkins, the build log is stored in Jenkins log, and
emails were sent with the failure log.

York

  reply	other threads:[~2014-08-05 18:48 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-05 14:46 [U-Boot] [PATCH v4 0/13] Add some missing buildman features and deprecate MAKEALL Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 01/13] buildman: Fix a few typos Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 02/13] buildman: Add some notes about moving from MAKEALL Simon Glass
2014-08-05 22:34   ` York Sun
2014-08-05 23:07     ` Simon Glass
2014-08-05 23:21       ` York Sun
2014-08-06  2:15         ` Simon Glass
2014-08-06  2:53           ` York Sun
2014-08-06 14:20             ` Simon Glass
2014-08-06 15:06               ` Tom Rini
2014-08-07 12:12                 ` Simon Glass
2014-08-07 13:14                   ` Masahiro Yamada
2014-08-08 10:59                     ` Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 03/13] buildman: Allow building of current source tree Simon Glass
2014-08-05 18:54   ` Tom Rini
2014-08-05 18:58     ` Simon Glass
2014-08-05 19:01       ` Tom Rini
2014-08-05 14:46 ` [U-Boot] [PATCH v4 04/13] buildman: Move BuilderThread code to its own file Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 05/13] buildman: Sort command line options Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 06/13] buildman: Refactor output options Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 07/13] buildman: Add verbose option to display errors as they happen Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 08/13] buildman: Remove unused non-incremental build method code Simon Glass
2014-08-05 14:46 ` [U-Boot] [PATCH v4 09/13] buildman: Add an option to specify the buildman config file Simon Glass
2014-08-05 14:47 ` [U-Boot] [PATCH v4 10/13] buildman: Add a message indicating there are no errors Simon Glass
2014-08-05 14:47 ` [U-Boot] [PATCH v4 11/13] buildman: Search for *cc instead of *gcc for the compiler Simon Glass
2014-08-05 17:41   ` Jeroen Hofstee
2014-08-05 18:25     ` Simon Glass
2014-08-05 14:47 ` [U-Boot] [PATCH v4 12/13] buildman: Add a few more toolchain examples to the README Simon Glass
2014-08-05 14:47 ` [U-Boot] [PATCH v4 13/13] RFC: Deprecate MAKEALL Simon Glass
2014-08-05 16:43   ` York Sun
2014-08-05 18:41     ` Simon Glass
2014-08-05 18:48       ` York Sun [this message]
2014-08-05 18:52         ` Simon Glass
2014-08-05 18:55           ` York Sun
2014-08-05 18:59             ` Simon Glass
2014-08-05 19:01               ` York Sun
2014-08-05 19:10                 ` Tom Rini
2014-08-05 22:06                   ` York Sun
2014-08-08 21:12                     ` Tom Rini
2014-08-08 21:19                       ` York Sun
2014-08-08 21:30                         ` Tom Rini
2014-08-08 21:36                           ` York Sun

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=53E126FF.3040503@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox