git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Problem with test from aa9c83c2
@ 2008-02-29 21:04 Mike Hommey
  2008-02-29 21:36 ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Hommey @ 2008-02-29 21:04 UTC (permalink / raw)
  To: git

Hi,

When run as root, the test added in aa9c83c2 fails, because root is allowed
to remove files in unreadable and unwriteable directories. Now, the
problem is that I don't know what to do with that. Should we care ?

Mike

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Problem with test from aa9c83c2
  2008-02-29 21:04 Problem with test from aa9c83c2 Mike Hommey
@ 2008-02-29 21:36 ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2008-02-29 21:36 UTC (permalink / raw)
  To: Mike Hommey; +Cc: git

Mike Hommey <mh@glandium.org> writes:

> When run as root, the test added in aa9c83c2 fails, because root is allowed
> to remove files in unreadable and unwriteable directories. Now, the
> problem is that I don't know what to do with that. Should we care ?

Of course not.  You are nuts running tests as root.

If "git foo" should refuse to remove a file for revision control
reason, and it does refuse correctly for a non-root user but it
fails to refuse when run as root, then that is a bug and we do
care about it.  IOW, I am not saying you are nuts running git as
root.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-02-29 21:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-02-29 21:04 Problem with test from aa9c83c2 Mike Hommey
2008-02-29 21:36 ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).