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

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

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 15, 2016, 10:48 UTC
Message-ID
<alpine.DEB.2.20.1607151246190.6426@virtualbox>
In-Reply-To
<CANaxB-y6XW7-ps1G2CB=N9CiD0VnSLhSer2nsf+2R4pm5TyswA@mail.gmail.com>
Hi Andrey,
On Fri, 15 Jul 2016, Andrey Vagin wrote:
> What will be wrong if 'git status' will reports these [fifo/socket] files?
`git status` is intended to give you an idea what to commit next. And...
> What will be wrong if 'git add' will returns an error instead of
> skipping them silently?

... we *cannot* commit fifos or sockets. There is simply no representation in Git's data model for them.

Having said that, I would welcome a patch that makes `git add` complain about arguments that could not be added (and are not directories).

As to the patch in question, for the above-mentioned reasons, I think we want to keep the existing behavior instead.

Ciao, Johannes

Previous: Andrey VaginNext: Johannes Sixt
Message 4 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.