{"action":"created","issue":{"url":"https://api.github.com/repos/jelmer/dulwich/issues/394","labels_url":"https://api.github.com/repos/jelmer/dulwich/issues/394/labels{/name}","comments_url":"https://api.github.com/repos/jelmer/dulwich/issues/394/comments","events_url":"https://api.github.com/repos/jelmer/dulwich/issues/394/events","html_url":"https://github.com/jelmer/dulwich/pull/394","id":119707536,"number":394,"title":"Fix thinpack support over http","user":{"login":"jsbain","id":7924558,"avatar_url":"https://avatars.githubusercontent.com/u/7924558?v=3","gravatar_id":"","url":"https://api.github.com/users/jsbain","html_url":"https://github.com/jsbain","followers_url":"https://api.github.com/users/jsbain/followers","following_url":"https://api.github.com/users/jsbain/following{/other_user}","gists_url":"https://api.github.com/users/jsbain/gists{/gist_id}","starred_url":"https://api.github.com/users/jsbain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsbain/subscriptions","organizations_url":"https://api.github.com/users/jsbain/orgs","repos_url":"https://api.github.com/users/jsbain/repos","events_url":"https://api.github.com/users/jsbain/events{/privacy}","received_events_url":"https://api.github.com/users/jsbain/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-01T12:30:10Z","updated_at":"2015-12-04T02:50:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jelmer/dulwich/pulls/394","html_url":"https://github.com/jelmer/dulwich/pull/394","diff_url":"https://github.com/jelmer/dulwich/pull/394.diff","patch_url":"https://github.com/jelmer/dulwich/pull/394.patch"},"body":"Checks fetch capabilities for thinpack support, and properly calls add_thin_pack instead of add_pack.\r\nThis is mainly for Http, but the fix had to be implemented in GitClient due to the way fetch is structured.  Fixes #226."},"comment":{"url":"https://api.github.com/repos/jelmer/dulwich/issues/comments/161857256","html_url":"https://github.com/jelmer/dulwich/pull/394#issuecomment-161857256","issue_url":"https://api.github.com/repos/jelmer/dulwich/issues/394","id":161857256,"user":{"login":"jsbain","id":7924558,"avatar_url":"https://avatars.githubusercontent.com/u/7924558?v=3","gravatar_id":"","url":"https://api.github.com/users/jsbain","html_url":"https://github.com/jsbain","followers_url":"https://api.github.com/users/jsbain/followers","following_url":"https://api.github.com/users/jsbain/following{/other_user}","gists_url":"https://api.github.com/users/jsbain/gists{/gist_id}","starred_url":"https://api.github.com/users/jsbain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsbain/subscriptions","organizations_url":"https://api.github.com/users/jsbain/orgs","repos_url":"https://api.github.com/users/jsbain/repos","events_url":"https://api.github.com/users/jsbain/events{/privacy}","received_events_url":"https://api.github.com/users/jsbain/received_events","type":"User","site_admin":false},"created_at":"2015-12-04T02:50:11Z","updated_at":"2015-12-04T02:50:11Z","body":"Is the concern about loading into memory vs a file, or not loading directly into the repo?  It seemed like another option was to use a Tempfile.  or are you considering a more massive change to allow adding a thinpack directly?  "}}