Merge branch 'js/typofixes'
authorJunio C Hamano <gitster@pobox.com>
Mon, 20 Aug 2018 18:33:50 +0000 (11:33 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 20 Aug 2018 18:33:50 +0000 (11:33 -0700)
Comment update.

* js/typofixes:
remote-curl: remove spurious period
git-compat-util.h: fix typo

1  2 
git-compat-util.h
remote-curl.c
diff --combined git-compat-util.h
index 89d37095c75775a6471b0188097a58a8d1440cb1,b20fdcae0252d4347ec2041e0f98adb74765d847..5f2e90932f9990419d75d803d9552acf2f14adef
  #include <sys/sysctl.h>
  #endif
  
 +#if defined(__CYGWIN__)
 +#include "compat/cygwin.h"
 +#endif
  #if defined(__MINGW32__)
  /* pull in Windows compatibility stuff */
  #include "compat/mingw.h"
  #endif
  #ifdef NO_INTPTR_T
  /*
-  * On I16LP32, ILP32 and LP64 "long" is the save bet, however
+  * On I16LP32, ILP32 and LP64 "long" is the safe bet, however
   * on LLP86, IL33LLP64 and P64 it needs to be "long long",
   * while on IP16 and IP16L32 it is "int" (resp. "short")
   * Size needs to match (or exceed) 'sizeof(void *)'.
@@@ -284,10 -281,6 +284,10 @@@ extern char *gitdirname(char *)
  #include <openssl/err.h>
  #endif
  
 +#ifdef HAVE_SYSINFO
 +# include <sys/sysinfo.h>
 +#endif
 +
  /* On most systems <netdb.h> would have given us this, but
   * not on some systems (e.g. z/OS).
   */
  #define PRIo32 "o"
  #endif
  
 +typedef uintmax_t timestamp_t;
 +#define PRItime PRIuMAX
 +#define parse_timestamp strtoumax
 +#define TIME_MAX UINTMAX_MAX
 +
  #ifndef PATH_SEP
  #define PATH_SEP ':'
  #endif
@@@ -453,13 -441,10 +453,13 @@@ static inline int const_error(void
  
  extern void set_die_routine(NORETURN_PTR void (*routine)(const char *err, va_list params));
  extern void set_error_routine(void (*routine)(const char *err, va_list params));
 +extern void (*get_error_routine(void))(const char *err, va_list params);
 +extern void set_warn_routine(void (*routine)(const char *warn, va_list params));
 +extern void (*get_warn_routine(void))(const char *warn, va_list params);
  extern void set_die_is_recursing_routine(int (*routine)(void));
 -extern void set_error_handle(FILE *);
  
  extern int starts_with(const char *str, const char *prefix);
 +extern int istarts_with(const char *str, const char *prefix);
  
  /*
   * If the string "str" begins with the string found in "prefix", return 1.
@@@ -489,29 -474,6 +489,29 @@@ static inline int skip_prefix(const cha
        return 0;
  }
  
 +/*
 + * If the string "str" is the same as the string in "prefix", then the "arg"
 + * parameter is set to the "def" parameter and 1 is returned.
 + * If the string "str" begins with the string found in "prefix" and then a
 + * "=" sign, then the "arg" parameter is set to "str + strlen(prefix) + 1"
 + * (i.e., to the point in the string right after the prefix and the "=" sign),
 + * and 1 is returned.
 + *
 + * Otherwise, return 0 and leave "arg" untouched.
 + *
 + * When we accept both a "--key" and a "--key=<val>" option, this function
 + * can be used instead of !strcmp(arg, "--key") and then
 + * skip_prefix(arg, "--key=", &arg) to parse such an option.
 + */
 +int skip_to_optional_arg_default(const char *str, const char *prefix,
 +                               const char **arg, const char *def);
 +
 +static inline int skip_to_optional_arg(const char *str, const char *prefix,
 +                                     const char **arg)
 +{
 +      return skip_to_optional_arg_default(str, prefix, arg, "");
 +}
 +
  /*
   * Like skip_prefix, but promises never to read past "len" bytes of the input
   * buffer, and returns the remaining number of bytes in "out" via "outlen".
@@@ -562,16 -524,6 +562,16 @@@ static inline int ends_with(const char 
        return strip_suffix(str, suffix, &len);
  }
  
 +#define SWAP(a, b) do {                                               \
 +      void *_swap_a_ptr = &(a);                               \
 +      void *_swap_b_ptr = &(b);                               \
 +      unsigned char _swap_buffer[sizeof(a)];                  \
 +      memcpy(_swap_buffer, _swap_a_ptr, sizeof(a));           \
 +      memcpy(_swap_a_ptr, _swap_b_ptr, sizeof(a) +            \
 +             BUILD_ASSERT_OR_ZERO(sizeof(a) == sizeof(b)));   \
 +      memcpy(_swap_b_ptr, _swap_buffer, sizeof(a));           \
 +} while (0)
 +
  #if defined(NO_MMAP) || defined(USE_WIN32_MMAP)
  
  #ifndef PROT_READ
@@@ -651,7 -603,7 +651,7 @@@ extern int git_lstat(const char *, stru
  #endif
  
  #define DEFAULT_PACKED_GIT_LIMIT \
 -      ((1024L * 1024L) * (size_t)(sizeof(void*) >= 8 ? 8192 : 256))
 +      ((1024L * 1024L) * (size_t)(sizeof(void*) >= 8 ? (32 * 1024L * 1024L) : 256))
  
  #ifdef NO_PREAD
  #define pread git_pread
@@@ -674,6 -626,11 +674,6 @@@ extern int gitsetenv(const char *, cons
  extern char *gitmkdtemp(char *);
  #endif
  
 -#ifdef NO_MKSTEMPS
 -#define mkstemps gitmkstemps
 -extern int gitmkstemps(char *, int);
 -#endif
 -
  #ifdef NO_UNSETENV
  #define unsetenv gitunsetenv
  extern void gitunsetenv(const char *);
@@@ -724,12 -681,10 +724,12 @@@ char *gitstrdup(const char *s)
  #endif
  
  #ifdef FREAD_READS_DIRECTORIES
 -#ifdef fopen
 -#undef fopen
 -#endif
 -#define fopen(a,b) git_fopen(a,b)
 +# if !defined(SUPPRESS_FOPEN_REDEFINITION)
 +#  ifdef fopen
 +#   undef fopen
 +#  endif
 +#  define fopen(a,b) git_fopen(a,b)
 +# endif
  extern FILE *git_fopen(const char*, const char*);
  #endif
  
@@@ -777,6 -732,8 +777,6 @@@ const char *inet_ntop(int af, const voi
  extern int git_atexit(void (*handler)(void));
  #endif
  
 -extern void release_pack_memory(size_t);
 -
  typedef void (*try_to_free_t)(size_t);
  extern try_to_free_t set_try_to_free_routine(try_to_free_t);
  
@@@ -831,17 -788,12 +831,17 @@@ extern ssize_t xpread(int fd, void *buf
  extern int xdup(int fd);
  extern FILE *xfopen(const char *path, const char *mode);
  extern FILE *xfdopen(int fd, const char *mode);
 -extern int xmkstemp(char *template);
 -extern int xmkstemp_mode(char *template, int mode);
 -extern int odb_mkstemp(char *template, size_t limit, const char *pattern);
 -extern int odb_pack_keep(char *name, size_t namesz, const unsigned char *sha1);
 +extern int xmkstemp(char *temp_filename);
 +extern int xmkstemp_mode(char *temp_filename, int mode);
  extern char *xgetcwd(void);
  extern FILE *fopen_for_writing(const char *path);
 +extern FILE *fopen_or_warn(const char *path, const char *mode);
 +
 +/*
 + * FREE_AND_NULL(ptr) is like free(ptr) followed by ptr = NULL. Note
 + * that ptr is used twice, so don't pass e.g. ptr++.
 + */
 +#define FREE_AND_NULL(p) do { free(p); (p) = NULL; } while (0)
  
  #define ALLOC_ARRAY(x, alloc) (x) = xmalloc(st_mult(sizeof(*(x)), (alloc)))
  #define REALLOC_ARRAY(x, alloc) (x) = xrealloc((x), st_mult(sizeof(*(x)), (alloc)))
@@@ -854,14 -806,6 +854,14 @@@ static inline void copy_array(void *dst
                memcpy(dst, src, st_mult(size, n));
  }
  
 +#define MOVE_ARRAY(dst, src, n) move_array((dst), (src), (n), sizeof(*(dst)) + \
 +      BUILD_ASSERT_OR_ZERO(sizeof(*(dst)) == sizeof(*(src))))
 +static inline void move_array(void *dst, const void *src, size_t n, size_t size)
 +{
 +      if (n)
 +              memmove(dst, src, st_mult(size, n));
 +}
 +
  /*
   * These functions help you allocate structs with flex arrays, and copy
   * the data directly into the array. For example, if you had:
@@@ -926,22 -870,14 +926,22 @@@ static inline char *xstrdup_or_null(con
  
  static inline size_t xsize_t(off_t len)
  {
 -      if (len > (size_t) len)
 +      size_t size = (size_t) len;
 +
 +      if (len != (off_t) size)
                die("Cannot handle files this big");
 -      return (size_t)len;
 +      return size;
  }
  
  __attribute__((format (printf, 3, 4)))
  extern int xsnprintf(char *dst, size_t max, const char *fmt, ...);
  
 +#ifndef HOST_NAME_MAX
 +#define HOST_NAME_MAX 256
 +#endif
 +
 +extern int xgethostname(char *buf, size_t len);
 +
  /* in ctype.c, for kwset users */
  extern const unsigned char tolower_trans_tbl[256];
  
@@@ -1006,23 -942,6 +1006,23 @@@ static inline int sane_iscase(int x, in
                return (x & 0x20) == 0;
  }
  
 +/*
 + * Like skip_prefix, but compare case-insensitively. Note that the comparison
 + * is done via tolower(), so it is strictly ASCII (no multi-byte characters or
 + * locale-specific conversions).
 + */
 +static inline int skip_iprefix(const char *str, const char *prefix,
 +                             const char **out)
 +{
 +      do {
 +              if (!*prefix) {
 +                      *out = str;
 +                      return 1;
 +              }
 +      } while (tolower(*str++) == tolower(*prefix++));
 +      return 0;
 +}
 +
  static inline int strtoul_ui(char const *s, int base, unsigned int *result)
  {
        unsigned long ul;
@@@ -1058,25 -977,6 +1058,25 @@@ void git_qsort(void *base, size_t nmemb
  #define qsort git_qsort
  #endif
  
 +#define QSORT(base, n, compar) sane_qsort((base), (n), sizeof(*(base)), compar)
 +static inline void sane_qsort(void *base, size_t nmemb, size_t size,
 +                            int(*compar)(const void *, const void *))
 +{
 +      if (nmemb > 1)
 +              qsort(base, nmemb, size, compar);
 +}
 +
 +#ifndef HAVE_ISO_QSORT_S
 +int git_qsort_s(void *base, size_t nmemb, size_t size,
 +              int (*compar)(const void *, const void *, void *), void *ctx);
 +#define qsort_s git_qsort_s
 +#endif
 +
 +#define QSORT_S(base, n, compar, ctx) do {                    \
 +      if (qsort_s((base), (n), sizeof(*(base)), compar, ctx)) \
 +              BUG("qsort_s() failed");                        \
 +} while (0)
 +
  #ifndef REG_STARTEND
  #error "Git requires REG_STARTEND support. Compile with NO_REGEX=NeedsStartEnd"
  #endif
@@@ -1133,18 -1033,6 +1133,18 @@@ static inline int regexec_buf(const reg
  #define HAVE_VARIADIC_MACROS 1
  #endif
  
 +/* usage.c: only to be used for testing BUG() implementation (see test-tool) */
 +extern int BUG_exit_code;
 +
 +#ifdef HAVE_VARIADIC_MACROS
 +__attribute__((format (printf, 3, 4))) NORETURN
 +void BUG_fl(const char *file, int line, const char *fmt, ...);
 +#define BUG(...) BUG_fl(__FILE__, __LINE__, __VA_ARGS__)
 +#else
 +__attribute__((format (printf, 1, 2))) NORETURN
 +void BUG(const char *fmt, ...);
 +#endif
 +
  /*
   * Preserves errno, prints a message, but gives no warning for ENOENT.
   * Returns 0 on success, which includes trying to unlink an object that does
@@@ -1178,8 -1066,8 +1178,8 @@@ int remove_or_warn(unsigned int mode, c
  int access_or_warn(const char *path, int mode, unsigned flag);
  int access_or_die(const char *path, int mode, unsigned flag);
  
 -/* Warn on an inaccessible file that ought to be accessible */
 -void warn_on_inaccessible(const char *path);
 +/* Warn on an inaccessible file if errno indicates this is an error */
 +int warn_on_fopen_errors(const char *path);
  
  #ifdef GMTIME_UNRELIABLE_ERRORS
  struct tm *git_gmtime(const time_t *);
@@@ -1202,47 -1090,6 +1202,47 @@@ struct tm *git_gmtime_r(const time_t *
  #define getc_unlocked(fh) getc(fh)
  #endif
  
 +/*
 + * Our code often opens a path to an optional file, to work on its
 + * contents when we can successfully open it.  We can ignore a failure
 + * to open if such an optional file does not exist, but we do want to
 + * report a failure in opening for other reasons (e.g. we got an I/O
 + * error, or the file is there, but we lack the permission to open).
 + *
 + * Call this function after seeing an error from open() or fopen() to
 + * see if the errno indicates a missing file that we can safely ignore.
 + */
 +static inline int is_missing_file_error(int errno_)
 +{
 +      return (errno_ == ENOENT || errno_ == ENOTDIR);
 +}
 +
  extern int cmd_main(int, const char **);
  
 +/*
 + * You can mark a stack variable with UNLEAK(var) to avoid it being
 + * reported as a leak by tools like LSAN or valgrind. The argument
 + * should generally be the variable itself (not its address and not what
 + * it points to). It's safe to use this on pointers which may already
 + * have been freed, or on pointers which may still be in use.
 + *
 + * Use this _only_ for a variable that leaks by going out of scope at
 + * program exit (so only from cmd_* functions or their direct helpers).
 + * Normal functions, especially those which may be called multiple
 + * times, should actually free their memory. This is only meant as
 + * an annotation, and does nothing in non-leak-checking builds.
 + */
 +#ifdef SUPPRESS_ANNOTATED_LEAKS
 +extern void unleak_memory(const void *ptr, size_t len);
 +#define UNLEAK(var) unleak_memory(&(var), sizeof(var))
 +#else
 +#define UNLEAK(var) do {} while (0)
 +#endif
 +
 +/*
 + * This include must come after system headers, since it introduces macros that
 + * replace system names.
 + */
 +#include "banned.h"
 +
  #endif
diff --combined remote-curl.c
index 99b0bedc6ddc670cad6e74083dc33dc0aa4b595a,954c41ba2cde43b6d4f724f03b622f1825a50ffd..fb28309e850518018667dc746c630acc1f497981
@@@ -1,11 -1,9 +1,11 @@@
  #include "cache.h"
 +#include "config.h"
  #include "remote.h"
 +#include "connect.h"
  #include "strbuf.h"
  #include "walker.h"
  #include "http.h"
 -#include "exec_cmd.h"
 +#include "exec-cmd.h"
  #include "run-command.h"
  #include "pkt-line.h"
  #include "string-list.h"
@@@ -14,8 -12,6 +14,8 @@@
  #include "credential.h"
  #include "sha1-array.h"
  #include "send-pack.h"
 +#include "protocol.h"
 +#include "quote.h"
  
  static struct remote *remote;
  /* always ends with a trailing slash */
@@@ -24,10 -20,6 +24,10 @@@ static struct strbuf url = STRBUF_INIT
  struct options {
        int verbosity;
        unsigned long depth;
 +      char *deepen_since;
 +      struct string_list deepen_not;
 +      struct string_list push_options;
 +      char *filter;
        unsigned progress : 1,
                check_self_contained_and_connected : 1,
                cloning : 1,
                dry_run : 1,
                thin : 1,
                /* One of the SEND_PACK_PUSH_CERT_* constants. */
 -              push_cert : 2;
 +              push_cert : 2,
 +              deepen_relative : 1,
 +              from_promisor : 1,
 +              no_dependents : 1;
  };
  static struct options options;
  static struct string_list cas_options = STRING_LIST_INIT_DUP;
@@@ -71,23 -60,6 +71,23 @@@ static int set_option(const char *name
                options.depth = v;
                return 0;
        }
 +      else if (!strcmp(name, "deepen-since")) {
 +              options.deepen_since = xstrdup(value);
 +              return 0;
 +      }
 +      else if (!strcmp(name, "deepen-not")) {
 +              string_list_append(&options.deepen_not, value);
 +              return 0;
 +      }
 +      else if (!strcmp(name, "deepen-relative")) {
 +              if (!strcmp(value, "true"))
 +                      options.deepen_relative = 1;
 +              else if (!strcmp(value, "false"))
 +                      options.deepen_relative = 0;
 +              else
 +                      return -1;
 +              return 0;
 +      }
        else if (!strcmp(name, "followtags")) {
                if (!strcmp(value, "true"))
                        options.followtags = 1;
                else
                        return -1;
                return 0;
 +      } else if (!strcmp(name, "push-option")) {
 +              if (*value != '"')
 +                      string_list_append(&options.push_options, value);
 +              else {
 +                      struct strbuf unquoted = STRBUF_INIT;
 +                      if (unquote_c_style(&unquoted, value, NULL) < 0)
 +                              die("invalid quoting in push-option value");
 +                      string_list_append_nodup(&options.push_options,
 +                                               strbuf_detach(&unquoted, NULL));
 +              }
 +              return 0;
  
  #if LIBCURL_VERSION_NUM >= 0x070a08
        } else if (!strcmp(name, "family")) {
                        return -1;
                return 0;
  #endif /* LIBCURL_VERSION_NUM >= 0x070a08 */
 +      } else if (!strcmp(name, "from-promisor")) {
 +              options.from_promisor = 1;
 +              return 0;
 +      } else if (!strcmp(name, "no-dependents")) {
 +              options.no_dependents = 1;
 +              return 0;
 +      } else if (!strcmp(name, "filter")) {
 +              options.filter = xstrdup(value);;
 +              return 0;
        } else {
                return 1 /* unsupported */;
        }
  }
  
  struct discovery {
 -      const char *service;
 +      char *service;
        char *buf_alloc;
        char *buf;
        size_t len;
        struct ref *refs;
 -      struct sha1_array shallow;
 +      struct oid_array shallow;
 +      enum protocol_version version;
        unsigned proto_git : 1;
  };
  static struct discovery *last_discovery;
  static struct ref *parse_git_refs(struct discovery *heads, int for_push)
  {
        struct ref *list = NULL;
 -      get_remote_heads(-1, heads->buf, heads->len, &list,
 -                       for_push ? REF_NORMAL : 0, NULL, &heads->shallow);
 +      struct packet_reader reader;
 +
 +      packet_reader_init(&reader, -1, heads->buf, heads->len,
 +                         PACKET_READ_CHOMP_NEWLINE |
 +                         PACKET_READ_GENTLE_ON_EOF);
 +
 +      heads->version = discover_version(&reader);
 +      switch (heads->version) {
 +      case protocol_v2:
 +              /*
 +               * Do nothing.  This isn't a list of refs but rather a
 +               * capability advertisement.  Client would have run
 +               * 'stateless-connect' so we'll dump this capability listing
 +               * and let them request the refs themselves.
 +               */
 +              break;
 +      case protocol_v1:
 +      case protocol_v0:
 +              get_remote_heads(&reader, &list, for_push ? REF_NORMAL : 0,
 +                               NULL, &heads->shallow);
 +              break;
 +      case protocol_unknown_version:
 +              BUG("unknown protocol version");
 +      }
 +
        return list;
  }
  
@@@ -282,10 -210,9 +282,10 @@@ static void free_discovery(struct disco
        if (d) {
                if (d == last_discovery)
                        last_discovery = NULL;
 -              free(d->shallow.sha1);
 +              free(d->shallow.oid);
                free(d->buf_alloc);
                free_refs(d->refs);
 +              free(d->service);
                free(d);
        }
  }
@@@ -317,19 -244,6 +317,19 @@@ static int show_http_message(struct str
        return 0;
  }
  
 +static int get_protocol_http_header(enum protocol_version version,
 +                                  struct strbuf *header)
 +{
 +      if (version > 0) {
 +              strbuf_addf(header, GIT_PROTOCOL_HEADER ": version=%d",
 +                          version);
 +
 +              return 1;
 +      }
 +
 +      return 0;
 +}
 +
  static struct discovery *discover_refs(const char *service, int for_push)
  {
        struct strbuf exp = STRBUF_INIT;
        struct strbuf buffer = STRBUF_INIT;
        struct strbuf refs_url = STRBUF_INIT;
        struct strbuf effective_url = STRBUF_INIT;
 +      struct strbuf protocol_header = STRBUF_INIT;
 +      struct string_list extra_headers = STRING_LIST_INIT_DUP;
        struct discovery *last = last_discovery;
        int http_ret, maybe_smart = 0;
 -      struct http_get_options options;
 +      struct http_get_options http_options;
 +      enum protocol_version version = get_protocol_version_config();
  
        if (last && !strcmp(service, last->service))
                return last;
                strbuf_addf(&refs_url, "service=%s", service);
        }
  
 -      memset(&options, 0, sizeof(options));
 -      options.content_type = &type;
 -      options.charset = &charset;
 -      options.effective_url = &effective_url;
 -      options.base_url = &url;
 -      options.no_cache = 1;
 -      options.keep_error = 1;
 -
 -      http_ret = http_get_strbuf(refs_url.buf, &buffer, &options);
 +      /*
 +       * NEEDSWORK: If we are trying to use protocol v2 and we are planning
 +       * to perform a push, then fallback to v0 since the client doesn't know
 +       * how to push yet using v2.
 +       */
 +      if (version == protocol_v2 && !strcmp("git-receive-pack", service))
 +              version = protocol_v0;
 +
 +      /* Add the extra Git-Protocol header */
 +      if (get_protocol_http_header(version, &protocol_header))
 +              string_list_append(&extra_headers, protocol_header.buf);
 +
 +      memset(&http_options, 0, sizeof(http_options));
 +      http_options.content_type = &type;
 +      http_options.charset = &charset;
 +      http_options.effective_url = &effective_url;
 +      http_options.base_url = &url;
 +      http_options.extra_headers = &extra_headers;
 +      http_options.initial_request = 1;
 +      http_options.no_cache = 1;
 +      http_options.keep_error = 1;
 +
 +      http_ret = http_get_strbuf(refs_url.buf, &buffer, &http_options);
        switch (http_ret) {
        case HTTP_OK:
                break;
                die("unable to access '%s': %s", url.buf, curl_errorstr);
        }
  
 +      if (options.verbosity && !starts_with(refs_url.buf, url.buf))
 +              warning(_("redirecting to %s"), url.buf);
 +
        last= xcalloc(1, sizeof(*last_discovery));
 -      last->service = service;
 +      last->service = xstrdup(service);
        last->buf_alloc = strbuf_detach(&buffer, &last->len);
        last->buf = last->buf_alloc;
  
                 * pkt-line matches our request.
                 */
                line = packet_read_line_buf(&last->buf, &last->len, NULL);
 +              if (!line)
 +                      die("invalid server response; expected service, got flush packet");
  
                strbuf_reset(&exp);
                strbuf_addf(&exp, "# service=%s", service);
                        ;
  
                last->proto_git = 1;
 +      } else if (maybe_smart &&
 +                 last->len > 5 && starts_with(last->buf + 4, "version 2")) {
 +              last->proto_git = 1;
        }
  
        if (last->proto_git)
        strbuf_release(&charset);
        strbuf_release(&effective_url);
        strbuf_release(&buffer);
 +      strbuf_release(&protocol_header);
 +      string_list_clear(&extra_headers, 0);
        last_discovery = last;
        return last;
  }
@@@ -487,14 -374,12 +487,14 @@@ struct rpc_state 
        char *service_url;
        char *hdr_content_type;
        char *hdr_accept;
 +      char *protocol_header;
        char *buf;
        size_t alloc;
        size_t len;
        size_t pos;
        int in;
        int out;
 +      int any_written;
        struct strbuf result;
        unsigned gzip_request : 1;
        unsigned initial_buffer : 1;
@@@ -551,8 -436,6 +551,8 @@@ static size_t rpc_in(char *ptr, size_t 
  {
        size_t size = eltsize * nmemb;
        struct rpc_state *rpc = buffer_;
 +      if (size)
 +              rpc->any_written = 1;
        write_or_die(rpc->in, ptr, size);
        return size;
  }
@@@ -617,12 -500,6 +617,12 @@@ static int probe_rpc(struct rpc_state *
        return err;
  }
  
 +static curl_off_t xcurl_off_t(ssize_t len) {
 +      if (len > maximum_signed_value_of_type(curl_off_t))
 +              die("cannot handle pushes this big");
 +      return (curl_off_t) len;
 +}
 +
  static int post_rpc(struct rpc_state *rpc)
  {
        struct active_request_slot *slot;
        headers = curl_slist_append(headers, needs_100_continue ?
                "Expect: 100-continue" : "Expect:");
  
 +      /* Add the extra Git-Protocol header */
 +      if (rpc->protocol_header)
 +              headers = curl_slist_append(headers, rpc->protocol_header);
 +
  retry:
        slot = get_active_slot();
  
        curl_easy_setopt(slot->curl, CURLOPT_NOBODY, 0);
        curl_easy_setopt(slot->curl, CURLOPT_POST, 1);
        curl_easy_setopt(slot->curl, CURLOPT_URL, rpc->service_url);
 -      curl_easy_setopt(slot->curl, CURLOPT_ENCODING, "gzip");
 +      curl_easy_setopt(slot->curl, CURLOPT_ENCODING, "");
  
        if (large_request) {
                /* The request body is large and the size cannot be predicted.
                 * and we just need to send it.
                 */
                curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDS, gzip_body);
 -              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE, gzip_size);
 +              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE_LARGE, xcurl_off_t(gzip_size));
  
        } else if (use_gzip && 1024 < rpc->len) {
                /* The client backend isn't giving us compressed data so
-                * we can try to deflate it ourselves, this may save on.
+                * we can try to deflate it ourselves, this may save on
                 * the transfer time.
                 */
                git_zstream stream;
  
                headers = curl_slist_append(headers, "Content-Encoding: gzip");
                curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDS, gzip_body);
 -              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE, gzip_size);
 +              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE_LARGE, xcurl_off_t(gzip_size));
  
                if (options.verbosity > 1) {
                        fprintf(stderr, "POST %s (gzip %lu to %lu bytes)\n",
                 * more normal Content-Length approach.
                 */
                curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDS, rpc->buf);
 -              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE, rpc->len);
 +              curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE_LARGE, xcurl_off_t(rpc->len));
                if (options.verbosity > 1) {
                        fprintf(stderr, "POST %s (%lu bytes)\n",
                                rpc->service_name, (unsigned long)rpc->len);
        curl_easy_setopt(slot->curl, CURLOPT_WRITEFUNCTION, rpc_in);
        curl_easy_setopt(slot->curl, CURLOPT_FILE, rpc);
  
 +
 +      rpc->any_written = 0;
        err = run_slot(slot, NULL);
        if (err == HTTP_REAUTH && !large_request) {
                credential_fill(&http_auth);
        if (err != HTTP_OK)
                err = -1;
  
 +      if (!rpc->any_written)
 +              err = -1;
 +
        curl_slist_free_all(headers);
        free(gzip_body);
        return err;
@@@ -818,11 -686,6 +818,11 @@@ static int rpc_service(struct rpc_stat
        strbuf_addf(&buf, "Accept: application/x-%s-result", svc);
        rpc->hdr_accept = strbuf_detach(&buf, NULL);
  
 +      if (get_protocol_http_header(heads->version, &buf))
 +              rpc->protocol_header = strbuf_detach(&buf, NULL);
 +      else
 +              rpc->protocol_header = NULL;
 +
        while (!err) {
                int n = packet_read(rpc->out, NULL, NULL, rpc->buf, rpc->alloc, 0);
                if (!n)
        free(rpc->service_url);
        free(rpc->hdr_content_type);
        free(rpc->hdr_accept);
 +      free(rpc->protocol_header);
        free(rpc->buf);
        strbuf_release(&buf);
        return err;
@@@ -863,12 -725,15 +863,12 @@@ static int fetch_dumb(int nr_heads, str
        int ret, i;
  
        ALLOC_ARRAY(targets, nr_heads);
 -      if (options.depth)
 -              die("dumb http transport does not support --depth");
 +      if (options.depth || options.deepen_since)
 +              die("dumb http transport does not support shallow capabilities");
        for (i = 0; i < nr_heads; i++)
                targets[i] = xstrdup(oid_to_hex(&to_fetch[i]->old_oid));
  
        walker = get_http_walker(url.buf);
 -      walker->get_all = 1;
 -      walker->get_tree = 1;
 -      walker->get_history = 1;
        walker->get_verbosely = options.verbosity >= 3;
        walker->get_recover = 0;
        ret = walker_fetch(walker, nr_heads, targets, NULL, NULL);
@@@ -886,41 -751,38 +886,41 @@@ static int fetch_git(struct discovery *
  {
        struct rpc_state rpc;
        struct strbuf preamble = STRBUF_INIT;
 -      char *depth_arg = NULL;
 -      int argc = 0, i, err;
 -      const char *argv[17];
 -
 -      argv[argc++] = "fetch-pack";
 -      argv[argc++] = "--stateless-rpc";
 -      argv[argc++] = "--stdin";
 -      argv[argc++] = "--lock-pack";
 +      int i, err;
 +      struct argv_array args = ARGV_ARRAY_INIT;
 +
 +      argv_array_pushl(&args, "fetch-pack", "--stateless-rpc",
 +                       "--stdin", "--lock-pack", NULL);
        if (options.followtags)
 -              argv[argc++] = "--include-tag";
 +              argv_array_push(&args, "--include-tag");
        if (options.thin)
 -              argv[argc++] = "--thin";
 -      if (options.verbosity >= 3) {
 -              argv[argc++] = "-v";
 -              argv[argc++] = "-v";
 -      }
 +              argv_array_push(&args, "--thin");
 +      if (options.verbosity >= 3)
 +              argv_array_pushl(&args, "-v", "-v", NULL);
        if (options.check_self_contained_and_connected)
 -              argv[argc++] = "--check-self-contained-and-connected";
 +              argv_array_push(&args, "--check-self-contained-and-connected");
        if (options.cloning)
 -              argv[argc++] = "--cloning";
 +              argv_array_push(&args, "--cloning");
        if (options.update_shallow)
 -              argv[argc++] = "--update-shallow";
 +              argv_array_push(&args, "--update-shallow");
        if (!options.progress)
 -              argv[argc++] = "--no-progress";
 -      if (options.depth) {
 -              struct strbuf buf = STRBUF_INIT;
 -              strbuf_addf(&buf, "--depth=%lu", options.depth);
 -              depth_arg = strbuf_detach(&buf, NULL);
 -              argv[argc++] = depth_arg;
 -      }
 -      argv[argc++] = url.buf;
 -      argv[argc++] = NULL;
 +              argv_array_push(&args, "--no-progress");
 +      if (options.depth)
 +              argv_array_pushf(&args, "--depth=%lu", options.depth);
 +      if (options.deepen_since)
 +              argv_array_pushf(&args, "--shallow-since=%s", options.deepen_since);
 +      for (i = 0; i < options.deepen_not.nr; i++)
 +              argv_array_pushf(&args, "--shallow-exclude=%s",
 +                               options.deepen_not.items[i].string);
 +      if (options.deepen_relative && options.depth)
 +              argv_array_push(&args, "--deepen-relative");
 +      if (options.from_promisor)
 +              argv_array_push(&args, "--from-promisor");
 +      if (options.no_dependents)
 +              argv_array_push(&args, "--no-dependents");
 +      if (options.filter)
 +              argv_array_pushf(&args, "--filter=%s", options.filter);
 +      argv_array_push(&args, url.buf);
  
        for (i = 0; i < nr_heads; i++) {
                struct ref *ref = to_fetch[i];
  
        memset(&rpc, 0, sizeof(rpc));
        rpc.service_name = "git-upload-pack",
 -      rpc.argv = argv;
 +      rpc.argv = args.argv;
        rpc.stdin_preamble = &preamble;
        rpc.gzip_request = 1;
  
                write_or_die(1, rpc.result.buf, rpc.result.len);
        strbuf_release(&rpc.result);
        strbuf_release(&preamble);
 -      free(depth_arg);
 +      argv_array_clear(&args);
        return err;
  }
  
@@@ -1052,9 -914,6 +1052,9 @@@ static int push_git(struct discovery *h
                argv_array_push(&args, "--quiet");
        else if (options.verbosity > 1)
                argv_array_push(&args, "--verbose");
 +      for (i = 0; i < options.push_options.nr; i++)
 +              argv_array_pushf(&args, "--push-option=%s",
 +                               options.push_options.items[i].string);
        argv_array_push(&args, options.progress ? "--progress" : "--no-progress");
        for_each_string_list_item(cas_option, &cas_options)
                argv_array_push(&args, cas_option->string);
@@@ -1125,203 -984,6 +1125,203 @@@ static void parse_push(struct strbuf *b
        free(specs);
  }
  
 +/*
 + * Used to represent the state of a connection to an HTTP server when
 + * communicating using git's wire-protocol version 2.
 + */
 +struct proxy_state {
 +      char *service_name;
 +      char *service_url;
 +      struct curl_slist *headers;
 +      struct strbuf request_buffer;
 +      int in;
 +      int out;
 +      struct packet_reader reader;
 +      size_t pos;
 +      int seen_flush;
 +};
 +
 +static void proxy_state_init(struct proxy_state *p, const char *service_name,
 +                           enum protocol_version version)
 +{
 +      struct strbuf buf = STRBUF_INIT;
 +
 +      memset(p, 0, sizeof(*p));
 +      p->service_name = xstrdup(service_name);
 +
 +      p->in = 0;
 +      p->out = 1;
 +      strbuf_init(&p->request_buffer, 0);
 +
 +      strbuf_addf(&buf, "%s%s", url.buf, p->service_name);
 +      p->service_url = strbuf_detach(&buf, NULL);
 +
 +      p->headers = http_copy_default_headers();
 +
 +      strbuf_addf(&buf, "Content-Type: application/x-%s-request", p->service_name);
 +      p->headers = curl_slist_append(p->headers, buf.buf);
 +      strbuf_reset(&buf);
 +
 +      strbuf_addf(&buf, "Accept: application/x-%s-result", p->service_name);
 +      p->headers = curl_slist_append(p->headers, buf.buf);
 +      strbuf_reset(&buf);
 +
 +      p->headers = curl_slist_append(p->headers, "Transfer-Encoding: chunked");
 +
 +      /* Add the Git-Protocol header */
 +      if (get_protocol_http_header(version, &buf))
 +              p->headers = curl_slist_append(p->headers, buf.buf);
 +
 +      packet_reader_init(&p->reader, p->in, NULL, 0,
 +                         PACKET_READ_GENTLE_ON_EOF);
 +
 +      strbuf_release(&buf);
 +}
 +
 +static void proxy_state_clear(struct proxy_state *p)
 +{
 +      free(p->service_name);
 +      free(p->service_url);
 +      curl_slist_free_all(p->headers);
 +      strbuf_release(&p->request_buffer);
 +}
 +
 +/*
 + * CURLOPT_READFUNCTION callback function.
 + * Attempts to copy over a single packet-line at a time into the
 + * curl provided buffer.
 + */
 +static size_t proxy_in(char *buffer, size_t eltsize,
 +                     size_t nmemb, void *userdata)
 +{
 +      size_t max;
 +      struct proxy_state *p = userdata;
 +      size_t avail = p->request_buffer.len - p->pos;
 +
 +
 +      if (eltsize != 1)
 +              BUG("curl read callback called with size = %"PRIuMAX" != 1",
 +                  (uintmax_t)eltsize);
 +      max = nmemb;
 +
 +      if (!avail) {
 +              if (p->seen_flush) {
 +                      p->seen_flush = 0;
 +                      return 0;
 +              }
 +
 +              strbuf_reset(&p->request_buffer);
 +              switch (packet_reader_read(&p->reader)) {
 +              case PACKET_READ_EOF:
 +                      die("unexpected EOF when reading from parent process");
 +              case PACKET_READ_NORMAL:
 +                      packet_buf_write_len(&p->request_buffer, p->reader.line,
 +                                           p->reader.pktlen);
 +                      break;
 +              case PACKET_READ_DELIM:
 +                      packet_buf_delim(&p->request_buffer);
 +                      break;
 +              case PACKET_READ_FLUSH:
 +                      packet_buf_flush(&p->request_buffer);
 +                      p->seen_flush = 1;
 +                      break;
 +              }
 +              p->pos = 0;
 +              avail = p->request_buffer.len;
 +      }
 +
 +      if (max < avail)
 +              avail = max;
 +      memcpy(buffer, p->request_buffer.buf + p->pos, avail);
 +      p->pos += avail;
 +      return avail;
 +}
 +
 +static size_t proxy_out(char *buffer, size_t eltsize,
 +                      size_t nmemb, void *userdata)
 +{
 +      size_t size;
 +      struct proxy_state *p = userdata;
 +
 +      if (eltsize != 1)
 +              BUG("curl read callback called with size = %"PRIuMAX" != 1",
 +                  (uintmax_t)eltsize);
 +      size = nmemb;
 +
 +      write_or_die(p->out, buffer, size);
 +      return size;
 +}
 +
 +/* Issues a request to the HTTP server configured in `p` */
 +static int proxy_request(struct proxy_state *p)
 +{
 +      struct active_request_slot *slot;
 +
 +      slot = get_active_slot();
 +
 +      curl_easy_setopt(slot->curl, CURLOPT_ENCODING, "");
 +      curl_easy_setopt(slot->curl, CURLOPT_NOBODY, 0);
 +      curl_easy_setopt(slot->curl, CURLOPT_POST, 1);
 +      curl_easy_setopt(slot->curl, CURLOPT_URL, p->service_url);
 +      curl_easy_setopt(slot->curl, CURLOPT_HTTPHEADER, p->headers);
 +
 +      /* Setup function to read request from client */
 +      curl_easy_setopt(slot->curl, CURLOPT_READFUNCTION, proxy_in);
 +      curl_easy_setopt(slot->curl, CURLOPT_READDATA, p);
 +
 +      /* Setup function to write server response to client */
 +      curl_easy_setopt(slot->curl, CURLOPT_WRITEFUNCTION, proxy_out);
 +      curl_easy_setopt(slot->curl, CURLOPT_WRITEDATA, p);
 +
 +      if (run_slot(slot, NULL) != HTTP_OK)
 +              return -1;
 +
 +      return 0;
 +}
 +
 +static int stateless_connect(const char *service_name)
 +{
 +      struct discovery *discover;
 +      struct proxy_state p;
 +
 +      /*
 +       * Run the info/refs request and see if the server supports protocol
 +       * v2.  If and only if the server supports v2 can we successfully
 +       * establish a stateless connection, otherwise we need to tell the
 +       * client to fallback to using other transport helper functions to
 +       * complete their request.
 +       */
 +      discover = discover_refs(service_name, 0);
 +      if (discover->version != protocol_v2) {
 +              printf("fallback\n");
 +              fflush(stdout);
 +              return -1;
 +      } else {
 +              /* Stateless Connection established */
 +              printf("\n");
 +              fflush(stdout);
 +      }
 +
 +      proxy_state_init(&p, service_name, discover->version);
 +
 +      /*
 +       * Dump the capability listing that we got from the server earlier
 +       * during the info/refs request.
 +       */
 +      write_or_die(p.out, discover->buf, discover->len);
 +
 +      /* Peek the next packet line.  Until we see EOF keep sending POSTs */
 +      while (packet_reader_peek(&p.reader) != PACKET_READ_EOF) {
 +              if (proxy_request(&p)) {
 +                      /* We would have an err here */
 +                      break;
 +              }
 +      }
 +
 +      proxy_state_clear(&p);
 +      return 0;
 +}
 +
  int cmd_main(int argc, const char **argv)
  {
        struct strbuf buf = STRBUF_INIT;
        options.verbosity = 1;
        options.progress = !!isatty(2);
        options.thin = 1;
 +      string_list_init(&options.deepen_not, 1);
 +      string_list_init(&options.push_options, 1);
  
        remote = remote_get(argv[1]);
  
                        fflush(stdout);
  
                } else if (!strcmp(buf.buf, "capabilities")) {
 +                      printf("stateless-connect\n");
                        printf("fetch\n");
                        printf("option\n");
                        printf("push\n");
                        printf("check-connectivity\n");
                        printf("\n");
                        fflush(stdout);
 +              } else if (skip_prefix(buf.buf, "stateless-connect ", &arg)) {
 +                      if (!stateless_connect(arg))
 +                              break;
                } else {
                        error("remote-curl: unknown command '%s' from git", buf.buf);
                        return 1;