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

Re: [PATCH v4 2/2] interpret-trailers: add option for in-place editing

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Jan 19, 2016, 21:45 UTC
Message-ID
<CAPig+cRHTs9q4k=CqtY2j=ZtTYMU6_SPeCHkQe4m5AGXOjg_Ww@mail.gmail.com>
In-Reply-To
<xmqqfuxt9ti3.fsf@gitster.mtv.corp.google.com>
On Tue, Jan 19, 2016 at 3:58 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 19 quoted lines
> Eric Sunshine <sunshine@sunshineco.com> writes:
>> My understanding is that SANITY is an expectation that directory
>> permissions work in an expected POSIXy way: that is, a file can't be
>> deleted when its containing directory lacks 'write', and a file can't
>> be read/accessed when the directory has neither 'read' nor 'execute'.
>> This doesn't say anything about root not being allowed to read a file
>> when the file itself lacks 'read'.
>
> In short, SANITY is "does looking at permission bits sufficient to
> anticipate what the filesystem would do?" while POSIXPERM is "can
> chmod be used to tweak permission bits of the filesystem" (a
> filesystem that lacks permission bits support would qualify as
> !POSIXPERM, as there is nothing to tweak in the first place).
>
> I suspect the comment added by f400e51c and its patch description
> stressed too much about permission of a directory affecting what we
> can do to files inside the directory, and failed to describe another
> criteria for a sane environment: "files whose permission bits say
> you shouldn't be able to read or write cannot be read or written".

You suspect correctly. It was exactly the comment added by f400e51c that misled me. (t/README does, on the other hand, mention "root", as I noticed after reading your previous response.)

Thanks for spelling all this out. Hopefully, others reading your reply (now and later) will be less confused than I.

Previous: Junio C HamanoNext: Junio C Hamano
Message 13 of 17 in “Add in-place editing support to git interpret-trailers”
  1. 0/2 Add in-place editing support to git interpret-trailersTobias Klauser, Jan 14, 2016
  2. 1/2 trailer: allow to write to files other than stdoutTobias Klauser, Jan 14, 2016
  3. 2/2 interpret-trailers: add option for in-place editingTobias Klauser, Jan 14, 2016
  4. Junio C HamanoJan 14, 2016
  5. Tobias KlauserJan 15, 2016
  6. Junio C HamanoJan 15, 2016
  7. Tobias KlauserJan 15, 2016
  8. Eric SunshineJan 18, 2016
  9. Junio C HamanoJan 19, 2016
  10. Eric SunshineJan 19, 2016
  11. Eric SunshineJan 19, 2016
  12. Junio C HamanoJan 19, 2016
  13. Eric SunshineJan 19, 2016
  14. Junio C HamanoJan 19, 2016
  15. Eric SunshineJan 20, 2016
  16. Eric SunshineJan 18, 2016
  17. Tobias KlauserJan 19, 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.