{"action":"created","issue":{"url":"https://api.github.com/repos/libgit2/libgit2/issues/3040","labels_url":"https://api.github.com/repos/libgit2/libgit2/issues/3040/labels{/name}","comments_url":"https://api.github.com/repos/libgit2/libgit2/issues/3040/comments","events_url":"https://api.github.com/repos/libgit2/libgit2/issues/3040/events","html_url":"https://github.com/libgit2/libgit2/pull/3040","id":67709171,"number":3040,"title":"Detect Git for windows Git root directory","user":{"login":"csware","id":428133,"avatar_url":"https://avatars.githubusercontent.com/u/428133?v=3","gravatar_id":"","url":"https://api.github.com/users/csware","html_url":"https://github.com/csware","followers_url":"https://api.github.com/users/csware/followers","following_url":"https://api.github.com/users/csware/following{/other_user}","gists_url":"https://api.github.com/users/csware/gists{/gist_id}","starred_url":"https://api.github.com/users/csware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/csware/subscriptions","organizations_url":"https://api.github.com/users/csware/orgs","repos_url":"https://api.github.com/users/csware/repos","events_url":"https://api.github.com/users/csware/events{/privacy}","received_events_url":"https://api.github.com/users/csware/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":87,"created_at":"2015-04-11T00:17:34Z","updated_at":"2015-05-24T00:41:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/libgit2/libgit2/pulls/3040","html_url":"https://github.com/libgit2/libgit2/pull/3040","diff_url":"https://github.com/libgit2/libgit2/pull/3040.diff","patch_url":"https://github.com/libgit2/libgit2/pull/3040.patch"},"body":"The https://git-for-windows.github.io package uses a different directory structure compared to msysgit and also providfes a 64-bit installer which is not handles correctly right now."},"comment":{"url":"https://api.github.com/repos/libgit2/libgit2/issues/comments/104961167","html_url":"https://github.com/libgit2/libgit2/pull/3040#issuecomment-104961167","issue_url":"https://api.github.com/repos/libgit2/libgit2/issues/3040","id":104961167,"user":{"login":"kblees","id":1111200,"avatar_url":"https://avatars.githubusercontent.com/u/1111200?v=3","gravatar_id":"","url":"https://api.github.com/users/kblees","html_url":"https://github.com/kblees","followers_url":"https://api.github.com/users/kblees/followers","following_url":"https://api.github.com/users/kblees/following{/other_user}","gists_url":"https://api.github.com/users/kblees/gists{/gist_id}","starred_url":"https://api.github.com/users/kblees/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kblees/subscriptions","organizations_url":"https://api.github.com/users/kblees/orgs","repos_url":"https://api.github.com/users/kblees/repos","events_url":"https://api.github.com/users/kblees/events{/privacy}","received_events_url":"https://api.github.com/users/kblees/received_events","type":"User","site_admin":false},"created_at":"2015-05-24T00:41:47Z","updated_at":"2015-05-24T00:41:47Z","body":"> > > If yes, is this the \"system\" configuration, or is it something else? If it is not the \"system\" configuration, what is its name?\r\n\r\n> > It is the \"system\" configuration.\r\n\r\n> That would introduce a really big problem: what is /etc/gitconfig, then? And how would you change the documentation without breaking non-Windows expectations?\r\n\r\nThe documentation currently refers to *both* `/etc/gitconfig` and `$(prefix)/etc/gitconfig` as the \"system-wide\" configuration, sometimes even in the same sentence.\r\n\r\nWhere \"$(prefix)\" would be completely meaningless to ordinary users that have never seen an autoconf-style Makefile.\r\n\r\nAnd users that finally dicover git's Makefile will find out that the default is `prefix = $(HOME)`, so the \"system-wide\" configuration is actually **user-specific**!\r\n\r\nYou have to dig *much* deeper to discover that the ominous `$(prefix)` is only used if `$(sysconfdir)` is a relative path (which is the default *unless* `prefix=/usr`)...\r\n\r\nBottom line is that the current documentation is completely broken wrt. the \"system-wide\" config, and understanding it requires intimate knowledge of both the build system and the source code.\r\n\r\nI tried to fix this by removing `$(prefix)` almost everywhere, and explaining what it means in the single place where it remains."}}