All of lore.kernel.org
 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: Fri, 8 Aug 2014 14:36:19 -0700	[thread overview]
Message-ID: <53E542D3.5090806@freescale.com> (raw)
In-Reply-To: <20140808213049.GH19374@bill-the-cat>

On 08/08/2014 02:30 PM, Tom Rini wrote:
> On Fri, Aug 08, 2014 at 02:19:34PM -0700, York Sun wrote:
>> On 08/08/2014 02:12 PM, Tom Rini wrote:
> [snip]
>>> Oh I'm sure, but maybe we can help get everyone a better script.  For
>>> example, you've said the way your job goes currently is every commit
>>> gets a test.  But.. that's not what you need exactly it sounds like.  It
>>> sounds like you need a given gerrit issue, which can have a few commits
>>> to it, to be bisectable and compile tested.  buildman will handle this
>>> part for you, if you let it.
>>>
>> Tom,
>>
>> I agree buildman has improvements comparing with MAKEALL. I spent time
>> to make Gerrit, Jenkins and MAKEALL work together. It is not easy as
>> one would expect.  For now, if I can make builmand to build the last
>> commit, I can replace MAKEALL with buildman. But that seems not
>> working.
> 
> I believe you, I've spent a bunch of time inside of Jenkins myself (and
> then with IBM's LSF compute setup) and I'm doing some of the same
> hoop-jumping now too.  But I'm hoping maybe we can all take this as a
> chance to improve the various ad-hoc builder setups many of us have.
> 

Since we use Gerrit for internal review, each commit (proposed patch) triggers a
Jenkins event. I can change the script so no build is triggered, but I need to
explorer if buildman servers better than MAKEALL in this case. Because Jenkins
has no knowledge of the number of patches pushed, I don't think buildman has
enough information to find the "top" and start the build.

But anyway, it shouldn't stop us from moving to buildman. I just need the basic
feature available before removing MAKEALL.

York

      reply	other threads:[~2014-08-08 21:36 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
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 [this message]

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=53E542D3.5090806@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.