Merge branch 'js/runtime-prefix'
authorJunio C Hamano <gitster@pobox.com>
Tue, 8 May 2018 06:59:34 +0000 (15:59 +0900)
committerJunio C Hamano <gitster@pobox.com>
Tue, 8 May 2018 06:59:34 +0000 (15:59 +0900)
* js/runtime-prefix:
Avoid multiple PREFIX definitions
git_setup_gettext: plug memory leak
gettext: avoid initialization if the locale dir is not present

1  2 
Makefile
exec-cmd.c
gettext.c
diff --combined Makefile
index 4541788e50c4faa424c6df717dfcc6bc78fa570b,068a22f40bf1f3a4542d7e0f0bf6ba1b8f12745c..ad880d1fc57212fc6b47aeea792a58129b61238e
+++ b/Makefile
@@@ -464,26 -464,6 +464,26 @@@ all:
  # When using RUNTIME_PREFIX, define HAVE_WPGMPTR if your platform offers
  # the global variable _wpgmptr containing the absolute path of the current
  # executable (this is the case on Windows).
 +#
 +# Define DEVELOPER to enable more compiler warnings. Compiler version
 +# and family are auto detected, but could be overridden by defining
 +# COMPILER_FEATURES (see config.mak.dev)
 +#
 +# When DEVELOPER is set, DEVOPTS can be used to control compiler
 +# options.  This variable contains keywords separated by
 +# whitespace. The following keywords are are recognized:
 +#
 +#    no-error:
 +#
 +#        suppresses the -Werror that implicitly comes with
 +#        DEVELOPER=1. Useful for getting the full set of errors
 +#        without immediately dying, or for logging them.
 +#
 +#    extra-all:
 +#
 +#        The DEVELOPER mode enables -Wextra with a few exceptions. By
 +#        setting this flag the exceptions are removed, and all of
 +#        -Wextra is used.
  
  GIT-VERSION-FILE: FORCE
        @$(SHELL_PATH) ./GIT-VERSION-GEN
  # CFLAGS and LDFLAGS are for the users to override from the command line.
  
  CFLAGS = -g -O2 -Wall
 -DEVELOPER_CFLAGS = -Werror \
 -      -Wdeclaration-after-statement \
 -      -Wno-format-zero-length \
 -      -Wold-style-definition \
 -      -Woverflow \
 -      -Wpointer-arith \
 -      -Wstrict-prototypes \
 -      -Wunused \
 -      -Wvla
  LDFLAGS =
  ALL_CFLAGS = $(CPPFLAGS) $(CFLAGS)
  ALL_LDFLAGS = $(LDFLAGS)
@@@ -690,6 -679,7 +690,6 @@@ PROGRAM_OBJS += imap-send.
  PROGRAM_OBJS += sh-i18n--envsubst.o
  PROGRAM_OBJS += shell.o
  PROGRAM_OBJS += show-index.o
 -PROGRAM_OBJS += upload-pack.o
  PROGRAM_OBJS += remote-testsvn.o
  
  # Binary suffix, set to .exe for Windows builds
@@@ -738,7 -728,6 +738,7 @@@ TEST_PROGRAMS_NEED_X += test-dump-untra
  TEST_PROGRAMS_NEED_X += test-fake-ssh
  TEST_PROGRAMS_NEED_X += test-line-buffer
  TEST_PROGRAMS_NEED_X += test-parse-options
 +TEST_PROGRAMS_NEED_X += test-pkt-line
  TEST_PROGRAMS_NEED_X += test-svn-fe
  TEST_PROGRAMS_NEED_X += test-tool
  
@@@ -821,13 -810,11 +821,13 @@@ LIB_OBJS += branch.
  LIB_OBJS += bulk-checkin.o
  LIB_OBJS += bundle.o
  LIB_OBJS += cache-tree.o
 +LIB_OBJS += chdir-notify.o
  LIB_OBJS += checkout.o
  LIB_OBJS += color.o
  LIB_OBJS += column.o
  LIB_OBJS += combine-diff.o
  LIB_OBJS += commit.o
 +LIB_OBJS += commit-graph.o
  LIB_OBJS += compat/obstack.o
  LIB_OBJS += compat/terminal.o
  LIB_OBJS += config.o
@@@ -858,7 -845,7 +858,7 @@@ LIB_OBJS += ewah/bitmap.
  LIB_OBJS += ewah/ewah_bitmap.o
  LIB_OBJS += ewah/ewah_io.o
  LIB_OBJS += ewah/ewah_rlw.o
 -LIB_OBJS += exec_cmd.o
 +LIB_OBJS += exec-cmd.o
  LIB_OBJS += fetch-object.o
  LIB_OBJS += fetch-pack.o
  LIB_OBJS += fsck.o
@@@ -881,11 -868,9 +881,11 @@@ LIB_OBJS += list-objects-filter-options
  LIB_OBJS += ll-merge.o
  LIB_OBJS += lockfile.o
  LIB_OBJS += log-tree.o
 +LIB_OBJS += ls-refs.o
  LIB_OBJS += mailinfo.o
  LIB_OBJS += mailmap.o
  LIB_OBJS += match-trees.o
 +LIB_OBJS += mem-pool.o
  LIB_OBJS += merge.o
  LIB_OBJS += merge-blobs.o
  LIB_OBJS += merge-recursive.o
@@@ -930,7 -915,7 +930,7 @@@ LIB_OBJS += refs/packed-backend.
  LIB_OBJS += refs/ref-cache.o
  LIB_OBJS += ref-filter.o
  LIB_OBJS += remote.o
 -LIB_OBJS += replace_object.o
 +LIB_OBJS += replace-object.o
  LIB_OBJS += repository.o
  LIB_OBJS += rerere.o
  LIB_OBJS += resolve-undo.o
@@@ -938,13 -923,12 +938,13 @@@ LIB_OBJS += revision.
  LIB_OBJS += run-command.o
  LIB_OBJS += send-pack.o
  LIB_OBJS += sequencer.o
 +LIB_OBJS += serve.o
  LIB_OBJS += server-info.o
  LIB_OBJS += setup.o
  LIB_OBJS += sha1-array.o
  LIB_OBJS += sha1-lookup.o
 -LIB_OBJS += sha1_file.o
 -LIB_OBJS += sha1_name.o
 +LIB_OBJS += sha1-file.o
 +LIB_OBJS += sha1-name.o
  LIB_OBJS += shallow.o
  LIB_OBJS += sideband.o
  LIB_OBJS += sigchain.o
@@@ -967,7 -951,6 +967,7 @@@ LIB_OBJS += tree-diff.
  LIB_OBJS += tree.o
  LIB_OBJS += tree-walk.o
  LIB_OBJS += unpack-trees.o
 +LIB_OBJS += upload-pack.o
  LIB_OBJS += url.o
  LIB_OBJS += urlmatch.o
  LIB_OBJS += usage.o
@@@ -980,7 -963,7 +980,7 @@@ LIB_OBJS += walker.
  LIB_OBJS += wildmatch.o
  LIB_OBJS += worktree.o
  LIB_OBJS += wrapper.o
 -LIB_OBJS += write_or_die.o
 +LIB_OBJS += write-or-die.o
  LIB_OBJS += ws.o
  LIB_OBJS += wt-status.o
  LIB_OBJS += xdiff-interface.o
@@@ -1007,7 -990,6 +1007,7 @@@ BUILTIN_OBJS += builtin/clone.
  BUILTIN_OBJS += builtin/column.o
  BUILTIN_OBJS += builtin/commit-tree.o
  BUILTIN_OBJS += builtin/commit.o
 +BUILTIN_OBJS += builtin/commit-graph.o
  BUILTIN_OBJS += builtin/config.o
  BUILTIN_OBJS += builtin/count-objects.o
  BUILTIN_OBJS += builtin/credential.o
@@@ -1073,7 -1055,6 +1073,7 @@@ BUILTIN_OBJS += builtin/rev-parse.
  BUILTIN_OBJS += builtin/revert.o
  BUILTIN_OBJS += builtin/rm.o
  BUILTIN_OBJS += builtin/send-pack.o
 +BUILTIN_OBJS += builtin/serve.o
  BUILTIN_OBJS += builtin/shortlog.o
  BUILTIN_OBJS += builtin/show-branch.o
  BUILTIN_OBJS += builtin/show-ref.o
@@@ -1087,7 -1068,6 +1087,7 @@@ BUILTIN_OBJS += builtin/update-index.
  BUILTIN_OBJS += builtin/update-ref.o
  BUILTIN_OBJS += builtin/update-server-info.o
  BUILTIN_OBJS += builtin/upload-archive.o
 +BUILTIN_OBJS += builtin/upload-pack.o
  BUILTIN_OBJS += builtin/var.o
  BUILTIN_OBJS += builtin/verify-commit.o
  BUILTIN_OBJS += builtin/verify-pack.o
@@@ -1109,7 -1089,7 +1109,7 @@@ include config.mak.unam
  -include config.mak
  
  ifdef DEVELOPER
 -CFLAGS += $(DEVELOPER_CFLAGS)
 +include config.mak.dev
  endif
  
  comma := ,
@@@ -2128,17 -2108,14 +2128,17 @@@ GIT-PERL-HEADER: $(PERL_HEADER_TEMPLATE
        INSTLIBDIR_EXTRA='$(PERLLIB_EXTRA_SQ)' && \
        INSTLIBDIR="$$INSTLIBDIR$${INSTLIBDIR_EXTRA:+:$$INSTLIBDIR_EXTRA}" && \
        sed -e 's=@@PATHSEP@@=$(pathsep)=g' \
 -          -e 's=@@INSTLIBDIR@@='$$INSTLIBDIR'=g' \
 -          -e 's=@@PERLLIBDIR@@='$(perllibdir_SQ)'=g' \
 +          -e "s=@@INSTLIBDIR@@=$$INSTLIBDIR=g" \
            -e 's=@@PERLLIBDIR_REL@@=$(perllibdir_relative_SQ)=g' \
            -e 's=@@GITEXECDIR_REL@@=$(gitexecdir_relative_SQ)=g' \
            -e 's=@@LOCALEDIR_REL@@=$(localedir_relative_SQ)=g' \
            $< >$@+ && \
        mv $@+ $@
  
 +.PHONY: perllibdir
 +perllibdir:
 +      @echo '$(perllibdir_SQ)'
 +
  .PHONY: gitweb
  gitweb:
        $(QUIET_SUBDIR0)gitweb $(QUIET_SUBDIR1) all
  $(OBJECTS): $(LIB_H)
  endif
  
 -exec_cmd.sp exec_cmd.s exec_cmd.o: GIT-PREFIX
 -exec_cmd.sp exec_cmd.s exec_cmd.o: EXTRA_CPPFLAGS = \
 +exec-cmd.sp exec-cmd.s exec-cmd.o: GIT-PREFIX
 +exec-cmd.sp exec-cmd.s exec-cmd.o: EXTRA_CPPFLAGS = \
        '-DGIT_EXEC_PATH="$(gitexecdir_SQ)"' \
        '-DGIT_LOCALE_PATH="$(localedir_relative_SQ)"' \
        '-DBINDIR="$(bindir_relative_SQ)"' \
-       '-DPREFIX="$(prefix_SQ)"'
+       '-DFALLBACK_RUNTIME_PREFIX="$(prefix_SQ)"'
  
  builtin/init-db.sp builtin/init-db.s builtin/init-db.o: GIT-PREFIX
  builtin/init-db.sp builtin/init-db.s builtin/init-db.o: EXTRA_CPPFLAGS = \
diff --combined exec-cmd.c
index 3b0a039083a8d377325983272e4aa38ee1b5674c,0000000000000000000000000000000000000000..02d31ee89711da7a4d5a483c6339909c05a15e08
mode 100644,000000..100644
--- /dev/null
@@@ -1,365 -1,0 +1,365 @@@
-               prefix = PREFIX;
 +#include "cache.h"
 +#include "exec-cmd.h"
 +#include "quote.h"
 +#include "argv-array.h"
 +
 +#if defined(RUNTIME_PREFIX)
 +
 +#if defined(HAVE_NS_GET_EXECUTABLE_PATH)
 +#include <mach-o/dyld.h>
 +#endif
 +
 +#if defined(HAVE_BSD_KERN_PROC_SYSCTL)
 +#include <sys/param.h>
 +#include <sys/types.h>
 +#include <sys/sysctl.h>
 +#endif
 +
 +#endif /* RUNTIME_PREFIX */
 +
 +#define MAX_ARGS 32
 +
 +static const char *system_prefix(void);
 +
 +#ifdef RUNTIME_PREFIX
 +
 +/**
 + * When using a runtime prefix, Git dynamically resolves paths relative to its
 + * executable.
 + *
 + * The method for determining the path of the executable is highly
 + * platform-specific.
 + */
 +
 +/**
 + * Path to the current Git executable. Resolved on startup by
 + * 'git_resolve_executable_dir'.
 + */
 +static const char *executable_dirname;
 +
 +static const char *system_prefix(void)
 +{
 +      static const char *prefix;
 +
 +      assert(executable_dirname);
 +      assert(is_absolute_path(executable_dirname));
 +
 +      if (!prefix &&
 +          !(prefix = strip_path_suffix(executable_dirname, GIT_EXEC_PATH)) &&
 +          !(prefix = strip_path_suffix(executable_dirname, BINDIR)) &&
 +          !(prefix = strip_path_suffix(executable_dirname, "git"))) {
-       return PREFIX;
++              prefix = FALLBACK_RUNTIME_PREFIX;
 +              trace_printf("RUNTIME_PREFIX requested, "
 +                              "but prefix computation failed.  "
 +                              "Using static fallback '%s'.\n", prefix);
 +      }
 +      return prefix;
 +}
 +
 +/*
 + * Resolves the executable path from argv[0], only if it is absolute.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path_from_argv0(struct strbuf *buf, const char *argv0)
 +{
 +      const char *slash;
 +
 +      if (!argv0 || !*argv0)
 +              return -1;
 +
 +      slash = find_last_dir_sep(argv0);
 +      if (slash) {
 +              trace_printf("trace: resolved executable path from argv0: %s\n",
 +                           argv0);
 +              strbuf_add_absolute_path(buf, argv0);
 +              return 0;
 +      }
 +      return -1;
 +}
 +
 +#ifdef PROCFS_EXECUTABLE_PATH
 +/*
 + * Resolves the executable path by examining a procfs symlink.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path_procfs(struct strbuf *buf)
 +{
 +      if (strbuf_realpath(buf, PROCFS_EXECUTABLE_PATH, 0)) {
 +              trace_printf(
 +                      "trace: resolved executable path from procfs: %s\n",
 +                      buf->buf);
 +              return 0;
 +      }
 +      return -1;
 +}
 +#endif /* PROCFS_EXECUTABLE_PATH */
 +
 +#ifdef HAVE_BSD_KERN_PROC_SYSCTL
 +/*
 + * Resolves the executable path using KERN_PROC_PATHNAME BSD sysctl.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path_bsd_sysctl(struct strbuf *buf)
 +{
 +      int mib[4];
 +      char path[MAXPATHLEN];
 +      size_t cb = sizeof(path);
 +
 +      mib[0] = CTL_KERN;
 +      mib[1] = KERN_PROC;
 +      mib[2] = KERN_PROC_PATHNAME;
 +      mib[3] = -1;
 +      if (!sysctl(mib, 4, path, &cb, NULL, 0)) {
 +              trace_printf(
 +                      "trace: resolved executable path from sysctl: %s\n",
 +                      path);
 +              strbuf_addstr(buf, path);
 +              return 0;
 +      }
 +      return -1;
 +}
 +#endif /* HAVE_BSD_KERN_PROC_SYSCTL */
 +
 +#ifdef HAVE_NS_GET_EXECUTABLE_PATH
 +/*
 + * Resolves the executable path by querying Darwin application stack.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path_darwin(struct strbuf *buf)
 +{
 +      char path[PATH_MAX];
 +      uint32_t size = sizeof(path);
 +      if (!_NSGetExecutablePath(path, &size)) {
 +              trace_printf(
 +                      "trace: resolved executable path from Darwin stack: %s\n",
 +                      path);
 +              strbuf_addstr(buf, path);
 +              return 0;
 +      }
 +      return -1;
 +}
 +#endif /* HAVE_NS_GET_EXECUTABLE_PATH */
 +
 +#ifdef HAVE_WPGMPTR
 +/*
 + * Resolves the executable path by using the global variable _wpgmptr.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path_wpgmptr(struct strbuf *buf)
 +{
 +      int len = wcslen(_wpgmptr) * 3 + 1;
 +      strbuf_grow(buf, len);
 +      len = xwcstoutf(buf->buf, _wpgmptr, len);
 +      if (len < 0)
 +              return -1;
 +      buf->len += len;
 +      return 0;
 +}
 +#endif /* HAVE_WPGMPTR */
 +
 +/*
 + * Resolves the absolute path of the current executable.
 + *
 + * Returns 0 on success, -1 on failure.
 + */
 +static int git_get_exec_path(struct strbuf *buf, const char *argv0)
 +{
 +      /*
 +       * Identifying the executable path is operating system specific.
 +       * Selectively employ all available methods in order of preference,
 +       * preferring highly-available authoritative methods over
 +       * selectively-available or non-authoritative methods.
 +       *
 +       * All cases fall back on resolving against argv[0] if there isn't a
 +       * better functional method. However, note that argv[0] can be
 +       * used-supplied on many operating systems, and is not authoritative
 +       * in those cases.
 +       *
 +       * Each of these functions returns 0 on success, so evaluation will stop
 +       * after the first successful method.
 +       */
 +      if (
 +#ifdef HAVE_BSD_KERN_PROC_SYSCTL
 +              git_get_exec_path_bsd_sysctl(buf) &&
 +#endif /* HAVE_BSD_KERN_PROC_SYSCTL */
 +
 +#ifdef HAVE_NS_GET_EXECUTABLE_PATH
 +              git_get_exec_path_darwin(buf) &&
 +#endif /* HAVE_NS_GET_EXECUTABLE_PATH */
 +
 +#ifdef PROCFS_EXECUTABLE_PATH
 +              git_get_exec_path_procfs(buf) &&
 +#endif /* PROCFS_EXECUTABLE_PATH */
 +
 +#ifdef HAVE_WPGMPTR
 +              git_get_exec_path_wpgmptr(buf) &&
 +#endif /* HAVE_WPGMPTR */
 +
 +              git_get_exec_path_from_argv0(buf, argv0)) {
 +              return -1;
 +      }
 +
 +      if (strbuf_normalize_path(buf)) {
 +              trace_printf("trace: could not normalize path: %s\n", buf->buf);
 +              return -1;
 +      }
 +
 +      return 0;
 +}
 +
 +void git_resolve_executable_dir(const char *argv0)
 +{
 +      struct strbuf buf = STRBUF_INIT;
 +      char *resolved;
 +      const char *slash;
 +
 +      if (git_get_exec_path(&buf, argv0)) {
 +              trace_printf(
 +                      "trace: could not determine executable path from: %s\n",
 +                      argv0);
 +              strbuf_release(&buf);
 +              return;
 +      }
 +
 +      resolved = strbuf_detach(&buf, NULL);
 +      slash = find_last_dir_sep(resolved);
 +      if (slash)
 +              resolved[slash - resolved] = '\0';
 +
 +      executable_dirname = resolved;
 +      trace_printf("trace: resolved executable dir: %s\n",
 +                   executable_dirname);
 +}
 +
 +#else
 +
 +/*
 + * When not using a runtime prefix, Git uses a hard-coded path.
 + */
 +static const char *system_prefix(void)
 +{
++      return FALLBACK_RUNTIME_PREFIX;
 +}
 +
 +/*
 + * This is called during initialization, but No work needs to be done here when
 + * runtime prefix is not being used.
 + */
 +void git_resolve_executable_dir(const char *argv0)
 +{
 +}
 +
 +#endif /* RUNTIME_PREFIX */
 +
 +char *system_path(const char *path)
 +{
 +      struct strbuf d = STRBUF_INIT;
 +
 +      if (is_absolute_path(path))
 +              return xstrdup(path);
 +
 +      strbuf_addf(&d, "%s/%s", system_prefix(), path);
 +      return strbuf_detach(&d, NULL);
 +}
 +
 +static const char *exec_path_value;
 +
 +void git_set_exec_path(const char *exec_path)
 +{
 +      exec_path_value = exec_path;
 +      /*
 +       * Propagate this setting to external programs.
 +       */
 +      setenv(EXEC_PATH_ENVIRONMENT, exec_path, 1);
 +}
 +
 +/* Returns the highest-priority location to look for git programs. */
 +const char *git_exec_path(void)
 +{
 +      if (!exec_path_value) {
 +              const char *env = getenv(EXEC_PATH_ENVIRONMENT);
 +              if (env && *env)
 +                      exec_path_value = xstrdup(env);
 +              else
 +                      exec_path_value = system_path(GIT_EXEC_PATH);
 +      }
 +      return exec_path_value;
 +}
 +
 +static void add_path(struct strbuf *out, const char *path)
 +{
 +      if (path && *path) {
 +              strbuf_add_absolute_path(out, path);
 +              strbuf_addch(out, PATH_SEP);
 +      }
 +}
 +
 +void setup_path(void)
 +{
 +      const char *exec_path = git_exec_path();
 +      const char *old_path = getenv("PATH");
 +      struct strbuf new_path = STRBUF_INIT;
 +
 +      git_set_exec_path(exec_path);
 +      add_path(&new_path, exec_path);
 +
 +      if (old_path)
 +              strbuf_addstr(&new_path, old_path);
 +      else
 +              strbuf_addstr(&new_path, _PATH_DEFPATH);
 +
 +      setenv("PATH", new_path.buf, 1);
 +
 +      strbuf_release(&new_path);
 +}
 +
 +const char **prepare_git_cmd(struct argv_array *out, const char **argv)
 +{
 +      argv_array_push(out, "git");
 +      argv_array_pushv(out, argv);
 +      return out->argv;
 +}
 +
 +int execv_git_cmd(const char **argv)
 +{
 +      struct argv_array nargv = ARGV_ARRAY_INIT;
 +
 +      prepare_git_cmd(&nargv, argv);
 +      trace_argv_printf(nargv.argv, "trace: exec:");
 +
 +      /* execvp() can only ever return if it fails */
 +      sane_execvp("git", (char **)nargv.argv);
 +
 +      trace_printf("trace: exec failed: %s\n", strerror(errno));
 +
 +      argv_array_clear(&nargv);
 +      return -1;
 +}
 +
 +int execl_git_cmd(const char *cmd, ...)
 +{
 +      int argc;
 +      const char *argv[MAX_ARGS + 1];
 +      const char *arg;
 +      va_list param;
 +
 +      va_start(param, cmd);
 +      argv[0] = cmd;
 +      argc = 1;
 +      while (argc < MAX_ARGS) {
 +              arg = argv[argc++] = va_arg(param, char *);
 +              if (!arg)
 +                      break;
 +      }
 +      va_end(param);
 +      if (MAX_ARGS <= argc)
 +              return error("too many args to run %s", cmd);
 +
 +      argv[argc] = NULL;
 +      return execv_git_cmd(argv);
 +}
diff --combined gettext.c
index baba28343c311594e048d62e190ded09e49ed5b0,4f59dfa3d14e2d84ad3def9f9670dcc1c42d2db6..7272771c8e445da194ea608443d8bc9c891b6b33
+++ b/gettext.c
@@@ -3,7 -3,7 +3,7 @@@
   */
  
  #include "cache.h"
 -#include "exec_cmd.h"
 +#include "exec-cmd.h"
  #include "gettext.h"
  #include "strbuf.h"
  #include "utf8.h"
@@@ -159,15 -159,23 +159,23 @@@ static void init_gettext_charset(const 
  void git_setup_gettext(void)
  {
        const char *podir = getenv(GIT_TEXT_DOMAIN_DIR_ENVIRONMENT);
+       char *p = NULL;
  
        if (!podir)
-               podir = system_path(GIT_LOCALE_PATH);
+               podir = p = system_path(GIT_LOCALE_PATH);
+       if (!is_directory(podir)) {
+               free(p);
+               return;
+       }
  
        bindtextdomain("git", podir);
        setlocale(LC_MESSAGES, "");
        setlocale(LC_TIME, "");
        init_gettext_charset("git");
        textdomain("git");
+       free(p);
  }
  
  /* return the number of columns of string 's' in current locale */