git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [Question] .git folder file updates for changing head commit

From
Taylor Blau <me@ttaylorr.com>
Date
Mar 23, 2022, 20:25 UTC
Message-ID
<YjuCSpAHD6xHkb5G@nand.local>
In-Reply-To
<6fa76f28-063b-8964-c0a2-642dae88f1b3@huawei.com>
On Wed, Mar 23, 2022 at 03:19:06PM +0000, John Garry wrote:
Show 5 quoted lines
> For building the linux perf tool we use the git head commit id as part of
> the tool version sting. To save time in re-building, the Makefile rule has a
> dependency on .git/HEAD for rebuilding. An alternative approach would be to
> compare git log output to check current versus previous build head commit,
> but that is seen as inefficient time-wise.

Having a Makefile recipe that depends on $GIT_DIR/HEAD seems strange to me.

Presumably your Makefile rules would map out which parts of your program depend on each other, and would get invalidated when the source itself changes, no?

Perhaps you also care about the commit you're building from in order to embed something into your program. But it seems like you could inject the output of "git rev-parse HEAD" when you construct the version identifier whenever you do need to rebuild.

Thanks, Taylor

Previous: John GarryNext: John Garry
Message 2 of 9 in “[Question] .git folder file updates for changing head commit”
  1. John GarryMar 23, 2022
  2. Taylor BlauMar 23, 2022
  3. John GarryMar 24, 2022
  4. Ævar Arnfjörð BjarmasonMar 27, 2022
  5. John GarryMar 30, 2022
  6. Glen ChooMar 23, 2022
  7. John GarryMar 24, 2022
  8. Andreas SchwabMar 23, 2022
  9. John GarryMar 24, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.