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

Re: [PATCH] Fix git-svnimport against libsvn_ra.

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 23, 2008, 19:09 UTC
Message-ID
<7vljzsct9t.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<200807231008.39455.p_christ@hol.gr>
Thanks.
But the one line description is not descriptive enough, don't you think?

Imagine this line (and nothing else from what you wrote in the body text) appearing among 100 other commits. Can you tell what this commit is about 3 months from now on?

Perhaps:
    svnimport: newer libsvn wants us to ask for the root with "", not "/"
> In r27729, libsvn introduced an assert which explicitly
> forbids searching the tree at "/". Luckily enough, it
> still accepts an empty string "" as the starting point.

I think you implied by saying "still", asking with an empty string had the same effect as asking for the root "/" in earlier versions of SVN, and the patch will not break people with older libsvn. Is that correct?

> http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_ra/ra_loader.c?r1=27653&r2=27729
> ---
Sign-off, please?
Previous: P. Christeas
Message 2 of 2 in “Fix git-svnimport against libsvn_ra.”
  1. Fix git-svnimport against libsvn_ra.P. Christeas, Jul 23, 2008
  2. Junio C HamanoJul 23, 2008

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.