All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bin Meng <bmeng.cn@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Buildman Kconfig issue with consecutive builds
Date: Thu, 7 Nov 2019 20:41:37 +0800	[thread overview]
Message-ID: <CAEUhbmUM-ERDtyceWQLFjTkzBNF10BKyS8U2cMQ7_YgwG9rH3g@mail.gmail.com> (raw)
In-Reply-To: <ed50b7ce-406d-eb8f-8b82-31cec3ff95b0@kontron.de>

Hi Schrempf,

On Thu, Nov 7, 2019 at 12:17 AM Schrempf Frieder
<frieder.schrempf@kontron.de> wrote:
>
> Hi,
>
> I'm having some trouble using buildman to test the impact of some
> Kconfig cleanup patches ([1]).
>
> The patches introduce a new CONFIG_SPL_* option and I try to find out
> which defconfigs need to be fixed, by comparing build sizes.
>
> Now when I added a patch to fix a defconfig I noticed that buildman
> wouldn't report the expected size changes and upon looking more closely
> I found that the added Kconfig options are still missing in u-boot-spl.cfg.
>
> The strange thing is, that when I try to build only the last commit then
> the Kconfig options are there, which is why I suspect a bug in buildman
> not handling Kconfig changes correctly with consecutive builds.
>
> Can anyone have a look what is wrong or how I can debug this issue?
>
> The issue can be reproduced with the branch at [1], running:
>
> buildman -b spi_flash_kconfig_cleanup_3 --step 0 xilinx_zynqmp_virt
>

Could you please add "-C" to the buildman command line and have a try?

> for a build where the added Kconfig options are missing in the resulting
> u-boot-spl.cfg.
>
> And:
>
> buildman -b spi_flash_kconfig_cleanup_3^..spi_flash_kconfig_cleanup_3
> xilinx_zynqmp_virt
>
> for a build of only the last commit with expected output.
>
> Thanks,
> Frieder
>
> [1]: https://github.com/fschrempf/u-boot/commits/spi_flash_kconfig_cleanup_3

Regards,
Bin

  parent reply	other threads:[~2019-11-07 12:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-06 16:16 [U-Boot] Buildman Kconfig issue with consecutive builds Schrempf Frieder
2019-11-07  7:30 ` Schrempf Frieder
2019-11-07 12:41 ` Bin Meng [this message]
2019-11-07 13:28   ` Schrempf Frieder
2019-11-07 14:02     ` Bin Meng
2019-11-07 15:15       ` Schrempf Frieder
2019-11-07 16:23         ` Simon Glass
2019-11-07 19:14           ` Schrempf Frieder
2019-11-07 19:19             ` 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=CAEUhbmUM-ERDtyceWQLFjTkzBNF10BKyS8U2cMQ7_YgwG9rH3g@mail.gmail.com \
    --to=bmeng.cn@gmail.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.