Re: [PATCH] t3706: Add test for wildcard vs literal pathspec

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

> reset_git_repo &&

I couldn't reproduce this as I don't have that function here. Is it a
command that you defined locally?

> I get issues with these test cases,
> Specially at git ls-tree -r --name-only HEAD > actual_files
> 
> For some reason the test_cmp doesn't get resolved

You can check the output of those commands by calling your test script
with the `-v` flag.

If `test_cmp a b` is false, you can check why by placing a temporary
`diff a b` for displaying what changed, of course running with a `-v`.

> Is it a bad way to do? or is there a better way?

I don't see any problem based on that code snippet (except for the
reset_git_repo that is not defined), but it looks like that there are
other changes compared to the code you sent in the patch, so I don't
have enough context.





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux