<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 4 May 2020, 13:32 Michael Brown, <<a href="mailto:mcb30@ipxe.org">mcb30@ipxe.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 04/05/2020 02:10, Neil Roza wrote:<br>
>     On Sun, May 03, 2020 at 12:18:26PM +0900, Christian Nilsson wrote:<br>
>      > What if there is any local non commited changes, or config file<br>
>     changes, or<br>
>      > embedded script changes. The checksum over linked solves the<br>
>     hash, but is<br>
>      > it actually correct to use git as a source for BUILD_TIMESTAMP<br>
>     when there<br>
>      > is local changes?<br>
> <br>
>     I see the warning, but I don't see the problem.<br>
>     In case that doesn't answer the "What if question",<br>
>     please elaborate what the hidden danger is.<br>
> <br>
> No, I get it: a pristine HEAD and a dirty HEAD don't deserve the same <br>
> source date epoch. There's some ways to disambiguate these with `git <br>
> stash`, but this is starting to get complicated. It could become an ugly <br>
> `$(shell ...)` in the Makefile.housekeeping, or I could put it in a <br>
> helper shell script. What's the right decision for ipxe?<br>
<br>
BUILD_TIMESTAMP has a weaker requirement than BUILD_ID: it gets used <br>
only as a means to automatically select the newest version of iPXE if <br>
multiple iPXE drivers are loaded concurrently in a UEFI system.  It's <br>
already rounded down to the nearest minute when used for that purpose.<br>
<br>
For BUILD_TIMESTAMP (but not BUILD_ID), I am happy with using the commit <br>
date as extracted from git.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Good, can we just make sure that the purpose and usage is documented in the makefile as well?</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Michael<br>
_______________________________________________<br>
ipxe-devel mailing list<br>
<a href="mailto:ipxe-devel@lists.ipxe.org" target="_blank" rel="noreferrer">ipxe-devel@lists.ipxe.org</a><br>
<a href="https://lists.ipxe.org/mailman/listinfo/ipxe-devel" rel="noreferrer noreferrer" target="_blank">https://lists.ipxe.org/mailman/listinfo/ipxe-devel</a><br>
</blockquote></div></div></div>