On Wed, Dec 10, 2014 at 3:36 AM, Eric Sunshine <sunsh...@sunshineco.com> wrote: > On Wed, Dec 10, 2014 at 2:34 AM, Jeff King <p...@peff.net> wrote: >> Below is a another iteration on the patch. The actual code changes are >> the same as the strbuf one, but the tests take care to avoid assuming >> the filesystem can handle such a long path. Testing on Windows and OS X >> is appreciated. > > All three new tests fail on OS X. Thus far brief examination of the > first failing tests shows that 'expect' and 'actual' differ: > > expect: > long > master > > actual: > master
The failure manifests as soon as the refname hits length 1024, at which point for-each-ref stops reporting it. MAX_PATH on OS X is 1024, so some part of the machinery invoked by for-each-ref likely is rejecting refnames longer than that (even when coming from packed-refs). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
