git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: profile-fast is failing in my Git 2.2.1 build from tar in a Git repo

From
Jeff King <peff@peff.net>
Date
Jan 12, 2015, 21:08 UTC
Message-ID
<20150112210827.GA10409@peff.net>
In-Reply-To
<xmqq1tn7o63p.fsf@gitster.dls.corp.google.com>
On Tue, Jan 06, 2015 at 03:13:30PM -0800, Junio C Hamano wrote:
Show 25 quoted lines
> > The problem is in the t/perf/perf-lib.sh:test_perf_create_repo_from()
> > function, where we see this:
> >
> >   repo="$1"
> >   source="$2"
> >   source_git=$source/$(cd "$source" && git rev-parse --git-dir)
> >
> > The function is invoked as:
> >
> >   test_perf_create_repo_from '/home/psmith/src/git-2.2.1/t/perf/trash
> > directory.p5302-pack-index' /home/psmith/src/git-2.2.1/t/..
> >
> > but this doesn't work, because "$source" is not a Git repository; it's
> > the root of the unpacked tarball.
> 
> Yup, that is exactly why I said I am not surprised.
> 
> I do not think profile build is prepared to be run without having
> our history (after all, it is not test_perf_create_REPO_from, not
> test_perf_create_source_directory_of_git_from).  It wants to create
> a repository that hosts a reasonably sized but not too big a
> project, i.e. us.
> 
> The safest and cleanest fix would be to make sure that the said
> function checks if $source/.git is a repository, perhaps?

Hmph. We already dealt with this once in 93b5393 (Makefile: make perf tests optional for profile build, 2014-08-19). That check explicitly looks for ".git" (and not checking that we are in an outer git repo), so I am not sure why it is kicking in at all. But probably the fix should go there.

-Peff
Previous: Junio C HamanoNext: Jeff King
Message 3 of 4 in “profile-fast is failing in my Git 2.2.1 build from tar in a Git repo”
  1. Paul SmithJan 4, 2015
  2. Junio C HamanoJan 6, 2015
  3. Jeff KingJan 12, 2015
  4. Jeff KingJan 12, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.