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

Re: [PATCH 3/3] git-parse-remote: remove unused functions

From
SBSanti Béjar <santi@agolina.net>
Date
Jun 6, 2009, 21:03 UTC
Message-ID
<adf1fd3d0906061403sd13200co286e986c67135f4b@mail.gmail.com>
In-Reply-To
<7v63f9cd4s.fsf@alter.siamese.dyndns.org>
2009/6/6 Junio C Hamano <gitster@pobox.com>:
Show 14 quoted lines
> Santi Béjar <santi@agolina.net> writes:
>
>> Signed-off-by: Santi Béjar <santi@agolina.net>
>> ---
>>  .gitignore                         |    1 -
>>  Documentation/git-parse-remote.txt |   50 ---------
>>  git-parse-remote.sh                |  204 ------------------------------------
>>  3 files changed, 0 insertions(+), 255 deletions(-)
>>  delete mode 100644 Documentation/git-parse-remote.txt
>
> I do not understand this patch.
>
> With this patch remove git-parse-remote from .gitignore (as if the entire
> file is going away)
Ops, the .gitignore removal was a mistake.
> and its documentation (again as if the entire file is
> going away), so naturally I would have expected to see removal of the file
> in the patch as well (and that means Makefile needs to be touched), but
> that is not what is happening.

I just wanted to remove the unused functions, and their documentation. But I was removing the two functions that were documented, so I removed all the file. Maybe I can just remove their documentation and leave the rest of the file.

Santi
Previous: Junio C Hamano
Message 3 of 3 in “git-parse-remote: remove unused functions”
  1. 3/3 git-parse-remote: remove unused functionsSanti Béjar, Jun 5, 2009
  2. Junio C HamanoJun 6, 2009
  3. Santi BéjarJun 6, 2009

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.