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

Re: [PATCH] git-clean: remove fifo, devices, socket files

From
AVAndrew Vagin <avagin@virtuozzo.com>
Date
Jul 15, 2016, 21:36 UTC
Message-ID
<20160715213628.GA19021@outlook.office365.com>
In-Reply-To
<3a6cb604-daa0-6d34-3342-0671a0b23032@kdbg.org>
On Fri, Jul 15, 2016 at 10:33:42PM +0200, Johannes Sixt wrote:
Show 17 quoted lines
> Am 15.07.2016 um 09:46 schrieb Andrey Vagin:
> > On Thu, Jul 14, 2016 at 10:56 PM, Johannes Sixt <j6t@kdbg.org> wrote:
> > > IOW: These special files are invisible for Git unless it already knows the
> > > names. The latter case is outside 'git clean's domain, and the former case
> > > really means that special files in the working tree are left at the user's
> > > discretion.
> > 
> > I understand your points, but I don't see any reasons to ignore these files.
> > 
> > What will be wrong if 'git status' will reports these files?
> > What will be wrong if 'git add' will returns an error instead of
> > skipping them silently?
> 
> I can buy that 'git add' reports an error for special files. (And I concur
> with Dscho that the behavior should otherwise remain unchanged.) But this is
> not what the commit message sells even if the patch changes the behavior of
> 'git add', too (I haven't tested the patch).
Ok. Thank you for the feedback.
> 
> -- Hannes
> 
Previous: Johannes Sixt
Message 6 of 6 in “git-clean: remove fifo, devices, socket files”
  1. git-clean: remove fifo, devices, socket filesAndrey Vagin, Jul 15, 2016
  2. Johannes SixtJul 15, 2016
  3. Andrey VaginJul 15, 2016
  4. Johannes SchindelinJul 15, 2016
  5. Johannes SixtJul 15, 2016
  6. Andrew VaginJul 15, 2016

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.