All of lore.kernel.org
 help / color / mirror / Atom feed
From: Schrempf Frieder <frieder.schrempf@kontron.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Buildman Kconfig issue with consecutive builds
Date: Thu, 7 Nov 2019 07:30:57 +0000	[thread overview]
Message-ID: <075081e7-8a9b-4cb8-4091-6c29c6d0a63a@kontron.de> (raw)
In-Reply-To: <ed50b7ce-406d-eb8f-8b82-31cec3ff95b0@kontron.de>

On 06.11.19 17:16, Frieder Schrempf 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

Forgot to mention that my master is set to 
32d870a82203a16a6e05958e2a02287af4dd825a (which is not upstream) in this 
case.

> 
> 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

  reply	other threads:[~2019-11-07  7:30 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 [this message]
2019-11-07 12:41 ` Bin Meng
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=075081e7-8a9b-4cb8-4091-6c29c6d0a63a@kontron.de \
    --to=frieder.schrempf@kontron.de \
    --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.