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
prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox