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

Re: [PATCH] completion: Add PS1 configuration for submodules

From
Jens Lehmann <jens.lehmann@web.de>
Date
Dec 7, 2010, 21:08 UTC
Message-ID
<4CFEA249.907@web.de>
In-Reply-To
<0E479F18-B26A-4216-A71E-C65EAB41A74A@sb.org>
Am 07.12.2010 21:31, schrieb Kevin Ballard:
Show 19 quoted lines
> On Dec 7, 2010, at 4:15 AM, Ævar Arnfjörð Bjarmason wrote:
> 
>> On Tue, Dec 7, 2010 at 00:22, Scott Kyle <scott@appden.com> wrote:
>>> For those who often work on repositories with submodules, the dirty
>>> indicator for unstaged changes will almost always show because development
>>> is simultaneously happening on those submodules. The config option
>>> diff.ignoreSubmodules is not appropriate for this use because it has larger
>>> implications.
>>
>> Wouldn't it be a lot better to instead add support for showing
>> submodule dirtyness as distinct from the main tree's dirtyness? Then
>> you could easily spot if you had either your tree / submodule tree
>> changes, without just ignoring them.
> 
> That sounds like a good idea, but it doesn't necessarily have to come with
> this patch. Scott's use case here is he has a submodule that is _always_ dirty,
> and he simply doesn't want to see that stuff in the PS1. Having an option to
> show it separately would be very useful for me, but should perhaps be written
> as a separate patch.

I'm not sure if I understand your case correctly, but if there is only one submodule that is always dirty and everybody knows that but nobody cares, won't it make sense to change the "submodule.<name>.ignore" config option for that peculiar submodule via .git/config or .gitmodules?

Previous: Kevin BallardNext: Scott Kyle
Message 5 of 15 in “completion: Add PS1 configuration for submodules”
  1. completion: Add PS1 configuration for submodulesScott Kyle, Dec 6, 2010
  2. SZEDER GáborDec 7, 2010
  3. Ævar Arnfjörð BjarmasonDec 7, 2010
  4. Kevin BallardDec 7, 2010
  5. Jens LehmannDec 7, 2010
  6. Scott KyleDec 7, 2010
  7. Jens LehmannDec 7, 2010
  8. Jonathan NiederDec 7, 2010
  9. Scott KyleDec 7, 2010
  10. Jonathan NiederDec 12, 2010
  11. Jens LehmannDec 13, 2010
  12. Scott KyleDec 21, 2010
  13. Scott KyleDec 7, 2010
  14. Kevin BallardDec 7, 2010
  15. Ævar Arnfjörð BjarmasonDec 8, 2010

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.