Re: [PATCH] Disable filemode rebase tests in t3400-rebase.sh if no filesystem mode support

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH] Disable filemode rebase tests in t3400-rebase.sh if no filesystem mode support

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:46:09

Alex Riesen [off-list ref] writes:
Signed-off-by: Alex Riesen <redacted>
---

I'm honestly sorry for my contributions. They seem to be exclusively
in the "make Git work in Windows" area.  Depressing.

 t/t3400-rebase.sh |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
I wonder if you should be fixing the tests with "update-index --chmod"
instead...

Re: [PATCH] Disable filemode rebase tests in t3400-rebase.sh if no filesystem mode support

From: Alex Riesen <hidden>
Date: 2016-06-15 22:46:09

2009/2/11 Junio C Hamano [off-list ref]:
Alex Riesen [off-list ref] writes:
quoted
Signed-off-by: Alex Riesen <redacted>
---

I'm honestly sorry for my contributions. They seem to be exclusively
in the "make Git work in Windows" area.  Depressing.

 t/t3400-rebase.sh |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
I wonder if you should be fixing the tests with "update-index --chmod"
instead...
Well, mostly these should work independent of the filesystem (except
for the cases when the index is checked out later). They seem to, and
they look like they should, too. See t4014-format-patch, for one.

But you're right. I should have reviewed them earlier. Anyway, there is
not many tests which have "--chmod" in them:
t4006-diff-mode.sh, t4014-format-patch.sh, t6031-merge-recursive.sh,
and t9500-gitweb-standalone-no-errors.sh.
The t4006 does not bother to look at working tree (good),
t4014 is good (looks like its ignoring modes altogether),
t6031 uses --chmod to augment the chmod useless on FAT-like fs
(seem to be an earlier fix for the same problem).

What's more interesting is the use of chmod for index mode change purpose,
which should be reviewed regularly (and I do that only when it breaks.
Lazy boom, I am...)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help