Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nix/install: Rewrite to not rely on bashisms. #16

Closed
wants to merge 4 commits into from

Commits on Aug 31, 2014

  1. nix/install: Rewrite to not rely on bashisms.

    Gets rid of [[ ... ]] and provides a fallback to "which" if the "type"
    builtin is not available.
    
    Also, this now longer relies on "set -e", because we're checking the
    errors carefully by ourselves and properly clean up whenever there was
    an error.
    
    Signed-off-by: aszlig <aszlig@redmoonstudios.org>
    aszlig committed Aug 31, 2014
    Configuration menu
    Copy the full SHA
    18cf43d View commit details
    Browse the repository at this point in the history
  2. nix/install: Don't rely on tar for decompression.

    Older versions of tar didn't detect the compression type and thus tried
    to extract an uncompressed archive and even older versions of tar didn't
    even have the -j argument at all, so let's download, decompress and
    unpack in one go.
    
    Signed-off-by: aszlig <aszlig@redmoonstudios.org>
    aszlig committed Aug 31, 2014
    Configuration menu
    Copy the full SHA
    f84c795 View commit details
    Browse the repository at this point in the history
  3. nix: Fix install instructions to not rely on bash.

    Of course, the install script from Nix itself will use bash, but that's
    an entirely different case. The problem with the install instructions
    itself is that it even relied on bash for its "<()" syntax.
    
    Signed-off-by: aszlig <aszlig@redmoonstudios.org>
    aszlig committed Aug 31, 2014
    Configuration menu
    Copy the full SHA
    2833d67 View commit details
    Browse the repository at this point in the history

Commits on Sep 23, 2014

  1. nix/install: Re-add shell script wrapper.

    As @edolstra pointed out, this is to prevent execution of a partially
    downloaded script. So let's reintroduce the wrapper with a comment to
    make sure it won't be removed again.
    
    Signed-off-by: aszlig <aszlig@redmoonstudios.org>
    aszlig committed Sep 23, 2014
    Configuration menu
    Copy the full SHA
    cd0a40a View commit details
    Browse the repository at this point in the history