Re: gitattributes not read for diff-tree anymore in 2.15?
On Tue, Dec 05, 2017 at 10:16:45 -0800, Brandon Williams wrote: > Perfect, thanks! OK, attached is a shell script which recreates the issue. I haven't been able to get it to happen without the `GIT_WORK_TREE` and `GIT_INDEX_FILE` setup involved, so that seems to be important. I reran the bisect using this script and came up with this commit: commit be4ca290570f9173db64ea1f925b5b3831c6efed Author: David Turner Date: Thu Apr 20 16:41:18 2017 -0400 Increase core.packedGitLimit which seems even less relevant… Thanks, --Ben diff-tree-break.sh Description: Bourne shell script
Re: gitattributes not read for diff-tree anymore in 2.15?
On 12/05, Ben Boeckel wrote: > On Mon, Dec 04, 2017 at 15:03:55 -0800, Brandon Williams wrote: > > Reading the attributes files should be done regardless if the gitmodules > > file is read. The gitmodules file should only come into play if you are > > dealing with submodules. > > Yeah, it doesn't seem to make sense why this commit breaks us, but there > it is *shrug*. While it doesn't make the most sense, I still wouldn't be surprised if I missed something when writing that patch that inadvertently caused an issue. > > > Do you mind providing a reproduction recipe with expected outcome vs > > actual outcome and I can take a closer look. > > I'll work on one. It isn't as simple as I thought it was :) . The setup > we do before running the checks is apparently involved as running it > from the command line is not exhibiting the difference. > > --Ben Perfect, thanks! -- Brandon Williams
Re: gitattributes not read for diff-tree anymore in 2.15?
On Mon, Dec 04, 2017 at 15:03:55 -0800, Brandon Williams wrote: > Reading the attributes files should be done regardless if the gitmodules > file is read. The gitmodules file should only come into play if you are > dealing with submodules. Yeah, it doesn't seem to make sense why this commit breaks us, but there it is *shrug*. > Do you mind providing a reproduction recipe with expected outcome vs > actual outcome and I can take a closer look. I'll work on one. It isn't as simple as I thought it was :) . The setup we do before running the checks is apparently involved as running it from the command line is not exhibiting the difference. --Ben
Re: gitattributes not read for diff-tree anymore in 2.15?
On 12/04, Ben Boeckel wrote: > Hi, > > I've bisected a failure in our test suite to this commit: > > commit 557a5998df19faf8641acfc5b6b1c3c2ba64dca9 (HEAD, refs/bisect/bad) > Author: Brandon Williams > Date: Thu Aug 3 11:20:00 2017 -0700 > > submodule: remove gitmodules_config > > Now that the submodule-config subsystem can lazily read the gitmodules > file we no longer need to explicitly pre-read the gitmodules by > calling > 'gitmodules_config()' so let's remove it. > > Signed-off-by: Brandon Williams > Signed-off-by: Junio C Hamano > > Which is tags/v2.15.0-rc0~120^2. > > Our test suite is in a Rust project here: > > https://gitlab.kitware.com/utils/rust-git-checks > > and the failing test(s) can be run using: > > cargo test whitespace_all_ignored > > The test checks that when `.gitattributes` says that whitespace errors > should be ignored, they aren't reported as errors. My guess is that not > reading the gitmodules configuration also skips reading attributes > files. Is this reasoning correct? > > Thanks, > > --Ben Reading the attributes files should be done regardless if the gitmodules file is read. The gitmodules file should only come into play if you are dealing with submodules. Do you mind providing a reproduction recipe with expected outcome vs actual outcome and I can take a closer look. -- Brandon Williams