1. 06 May, 2018 1 commit
  2. 05 May, 2018 1 commit
  3. 17 Apr, 2018 1 commit
  4. 01 Nov, 2017 1 commit
  5. 16 Jun, 2016 1 commit
  6. 30 May, 2016 1 commit
  7. 21 Oct, 2015 2 commits
  8. 20 Oct, 2015 1 commit
    • Brian J Brennan's avatar
      Make install script safer · c9d93757
      Brian J Brennan authored
      This changeset wraps all of the commands in tools/install.sh in a
      function and then calls that function as the last line of the
      script.
      
      The current install instructions ask the user to download the install
      script using `curl` and pass the result to `sh`. This is totally
      fine (as long as both the instructions and the script itself are served
      using HTTPS), but the script should be written in a way such that it
      doesn't start trying to actually *do* anything until the very last line.
      
      The reason is due to the way `curl` work: if the socket drops before the
      request is complete (server abruptly hangs up, client's internet flakes
      out, etc.), `curl` will return the partial data that it received. Here
      is an example of that:
      
      ![partial file execution](https://cldup.com/qU_Mnh2GmT.png)
      
      A way this might cause issues for tools/install.sh is if the connection drops
      after cloning but before the repository (L53-56). The .zshrc
      configuration will not be copied and the shell will not be changed, but
      if the user tries to run the install script again it will claim
      oh-my-zsh is already installed (L31-39).
      
      While this is not a particularly dangerous error condition (the user can
      just delete .oh-my-zsh and re-run), it can certainly be confusing for
      new users. This also helps future-proof the script for a time when it
      might need to use a "dangerous" command, e.g. `rm`, and we want to make
      sure it happens in the most transactional way possible.
      c9d93757
  9. 16 Oct, 2015 1 commit
    • Yannick Eckey's avatar
      Fix install.sh/upgrade.sh for tput-less systems · 75e619b7
      Yannick Eckey authored
      @fcrozat's original fix assumes `which` not to output anything to STDOUT
      in case the command is not found. That is not necessarily true on all
      systems. A better solution is to check the return value instead.
      
      Fixes #4376
      75e619b7
  10. 28 Sep, 2015 1 commit
  11. 21 Sep, 2015 1 commit
  12. 03 Sep, 2015 1 commit
    • Andrew Janke's avatar
      Add Cygwin support to installer. · 502f08b5
      Andrew Janke authored
      * Balk at incompatible Windows/MSYS git
      * Test for chsh presence before trying to use it
      * Replace non-portable `[[ ... ]]` and `[ x = *pattern* ]` constructs
      502f08b5
  13. 27 Aug, 2015 1 commit
  14. 12 Aug, 2015 1 commit
  15. 20 Jul, 2015 2 commits
    • leycec's avatar
      Cygwin-specific "chsh" installation issue fixed. · e8caf22b
      leycec authored
      Installation previously assumed the existence of a "chsh" command in the current
      ${PATH}. Since Cygwin does *NOT* provide this command, installation now tests
      for the existence of this command before attempting to run it.
      e8caf22b
    • leycec's avatar
      Secure umask enforced during installation. · da395c58
      leycec authored
      For safety, a umask of 022 prohibiting both group and other writability is now
      enforced during OMZ installation. In theory, this should reduce the likelihood
      of subsequent compinit() failures due to insecure directory permissions under
      all platforms except for default Cygwin installations (in which Windows ACLs
      override POSIX umasks).
      da395c58
  16. 07 Jul, 2015 1 commit
  17. 06 Jul, 2015 1 commit
  18. 05 May, 2015 1 commit
  19. 20 Feb, 2015 1 commit
  20. 14 Jan, 2015 1 commit
  21. 28 Nov, 2014 1 commit
  22. 31 Aug, 2014 1 commit
  23. 21 Aug, 2014 1 commit
  24. 20 Aug, 2014 1 commit
  25. 30 Mar, 2014 1 commit
    • Henrik Holm's avatar
      Correct redirection of output from 'hash' · f46d06da
      Henrik Holm authored
      The intention of the redirection to /dev/null is to hide the output
      'hash: no such command: git' since we rely on the exit status.
      
      However, the output goes to stderr, so it's stderr that needs to be
      redirected. For completeness, we redirect both stderr and stdout using
      '2>&1'.
      
      Example:
      
        [~]$ hash git > /dev/null
        [~]$ PATH=''
        [~]$ hash git > /dev/null
        hash: no such command: git
        [~]$ hash git > /dev/null 2>&1
        [~]$
      f46d06da
  26. 13 Mar, 2014 1 commit
  27. 08 Mar, 2014 3 commits
  28. 25 Oct, 2013 1 commit
  29. 30 Jun, 2013 1 commit
  30. 07 Apr, 2013 1 commit
  31. 02 Dec, 2012 1 commit
  32. 09 Nov, 2012 1 commit
  33. 21 Oct, 2012 1 commit
  34. 23 Jun, 2012 1 commit
  35. 04 Apr, 2012 1 commit
  36. 03 Jun, 2011 1 commit