On 11/30, Stefan Beller wrote:
quoted
Oh interesting, I wonder if there is a way to not have to perform a
chdir since taking a lock to lstat wouldn't be ideal.
I think we could rewrite is_submodule_populated to be
int is_submodule_populated_cheap_with_no_chdir(char *path)
{
return stat(path + ".git")
}
i.e. just take the presence of the .git file/dir as a hint to run
the child process?
I like this approach, its a quick (thread-safe) check to see if the
submodule is interesting. If there happens to be an error with the
submodule's .git file/directory then the child process will fail out.
--
Brandon Williams