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

Re: [PATCH] git-init: set core.worktree if GIT_WORK_TREE is specified

From
MLMatthias Lederhofer <matled@gmx.net>
Date
Jul 4, 2007, 09:29 UTC
Message-ID
<20070704092915.GA18597@moooo.ath.cx>
In-Reply-To
<20070703224919.GA22578@moooo.ath.cx>
> +	if (!getcwd(cwd, sizeof(cwd)) || cwd[0] != '/')
> +		die("Unable to read current working directory");

Dscho just pointed out that this causes problems on windows. The same is also in setup_git_directory_gently and was there before I touched it, introduced by Linus in d288a700. What was the original reason to do this? Are there implementations of getcwd which return a relative path?

Additionally there are other places which need to check if some user supplied path is absolute. Should we have a macro/function for that which depends on the os and checks for path[0] = '/' on unix and perhaps path[1] = ':' on windows?

Previous: Matthias LederhoferNext: Johannes Sixt
Message 2 of 7 in “git-init: set core.worktree if GIT_WORK_TREE is specified”
  1. git-init: set core.worktree if GIT_WORK_TREE is specifiedMatthias Lederhofer, Jul 3, 2007
  2. Matthias LederhoferJul 4, 2007
  3. Johannes SixtJul 4, 2007
  4. Johannes SchindelinJul 4, 2007
  5. Linus TorvaldsJul 4, 2007
  6. Junio C HamanoJul 4, 2007
  7. Linus TorvaldsJul 4, 2007

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.