Thread (3 messages) flat view 3 messages, 3 authors, 2016-06-15

Re: [PATCH] tests: A new test prereq for testing chmod -w as root

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:49:14

Hi Ævar,

Junio C Hamano wrote:
Ævar Arnfjörð Bjarmason  [off-list ref] writes:
quoted
Some tests depend on not being able to read files after chmod -w. This
doesn't work when running the tests as root.
Obviously you meant s/read/write/ or "chmod -r" ;-)

We discussed this prerequisite in the past as "SANITY"
[...]
Also I think there was a discussion and proposed patch to support more
than one prerequisite tokens, concatenated with "," or something
Thanks for picking up this topic.  Here’s some related work:

 - Incomplete series using space-separated test prerequisites:
   http://thread.gmane.org/gmane.comp.version-control.git/145427

 - Why that series is incomplete:
   http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=104;bug=579273

 - Comma-separated test prerequisites:
   http://thread.gmane.org/gmane.comp.version-control.git/116729/focus=118434

 - Earlier.
   http://thread.gmane.org/gmane.comp.version-control.git/116729/focus=118385
    test_expect_success POSIXPERM,SANITY
Last time I looked, IFS=, as used by Hannes looked like a sane
approach.  I noticed too late and did not follow up; sorry about that.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help