<p><a href="https://review.coreboot.org/27884">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27884/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27884/1//COMMIT_MSG@11">Patch Set #1, Line 11:</a> <code style="font-family:monospace,monospace">objects are not invalidated when bumping the fwid.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't believe this, actually. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">You assessment is correct, the target will only get created once and never touched again. This is the current behavior. When .config or Kconfig change we also need to blow away the $(obj) dir as well.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The coreboot ebuild currently passes along the VBOOT_FWID_VERSION via .config.<br>https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/refs/heads/master/sys-boot/coreboot/coreboot-9999.ebuild#166<br>This means we need to blow away essentially everything because the config.h changes on every build. By modifying the coreboot ebuild to write out fwid.version and not update the .config then the .config will remain constant when only making source changes. By modifying the ebuild to only re-run make oldconfig if the new.config != old.config then the config.h timestamp will not change either.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'll get the patches for the ebuild uploaded soon. I need to clean them up.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27884">change 27884</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/27884"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I955106efd648a75a1311f24ede46bd238d1517e0 </div>
<div style="display:none"> Gerrit-Change-Number: 27884 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Raul Rangel <rrangel@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Raul Rangel <rrangel@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Julius Werner <jwerner@chromium.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 07 Aug 2018 14:18:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>