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

Re: [PATCH] setup: trace bare repository setups

From
Glen Choo <chooglen@google.com>
Date
May 8, 2023, 22:19 UTC
Message-ID
<kl6lo7mu1n5v.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To
<ZE/03oSVAmSFLh7c@google.com>
Josh Steadmon <steadmon@google.com> writes:
> Ah, sorry for misunderstanding. I've restored the test along with a
> test_unconfig line for V3; however, doesn't this just turn into a "change
> detector" test whose only purpose will be to fail if/when we change the
> default value for this config option?

Maybe a difference in opinion: "change detection" sounds like non-essential changes to program behavior can cause a test to fail, but default value handling seems quite essential to me. Not to mention that default value handling is easy to break, so IMO guarding against accidental regressions is pretty useful.

As for practical churn, I don't expect that we'll intentionally change this default often.

Previous: Josh SteadmonNext: Glen Choo
Message 7 of 16 in “setup: trace bare repository setups”
  1. setup: trace bare repository setupsJosh Steadmon, Apr 27, 2023
  2. Junio C HamanoApr 27, 2023
  3. Josh SteadmonApr 28, 2023
  4. Josh SteadmonApr 28, 2023
  5. Junio C HamanoApr 28, 2023
  6. Josh SteadmonMay 1, 2023
  7. Glen ChooMay 8, 2023
  8. Glen ChooApr 27, 2023
  9. Josh SteadmonApr 28, 2023
  10. setup: trace bare repository setupsJosh Steadmon, Apr 28, 2023
  11. Glen ChooApr 28, 2023
  12. Josh SteadmonMay 1, 2023
  13. setup: trace bare repository setupsJosh Steadmon, May 1, 2023
  14. Junio C HamanoMay 5, 2023
  15. Taylor BlauMay 8, 2023
  16. Josh SteadmonMay 10, 2023

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.