Skip to content

Conversation

@joneshf
Copy link
Member

@joneshf joneshf commented Nov 22, 2014

@garyb can you review? If what I said in #6 makes sense, I'd like to go this way. If not, let me know.

@garyb
Copy link
Member

garyb commented Nov 22, 2014

Looks good to me, but apparently travis doesn't like it:

...
0.23s$ tar zxvf $HOME/purescript.tar.gz -C /usr/local/bin purescript/psc{,i,-docs,-make} --strip-components=1

purescript/psc-docs

tar: psc-docs: Cannot open: Permission denied

purescript/psc-make

tar: psc-make: Cannot open: Permission denied

purescript/psci

tar: psci: Cannot open: Permission denied

purescript/psc

tar: psc: Cannot open: Permission denied

tar: Exiting with failure status due to previous errors

The command "tar zxvf $HOME/purescript.tar.gz -C /usr/local/bin purescript/psc{,i,-docs,-make} --strip-components=1" failed and exited with 2 during .

Your build has been stopped.

@garyb
Copy link
Member

garyb commented Nov 22, 2014

Ah, I think this is why $PATH was extended to include a directory in $HOME for the executables rather than installing them in /usr/local/bin

@joneshf
Copy link
Member Author

joneshf commented Nov 22, 2014

Oh, right.

@joneshf
Copy link
Member Author

joneshf commented Nov 22, 2014

Okay, I'm going to merge this and close #6, the history should stay the same though.

joneshf added a commit that referenced this pull request Nov 22, 2014
@joneshf joneshf merged commit 7f86709 into purescript-contrib:master Nov 22, 2014
@joneshf joneshf deleted the better-travis branch November 22, 2014 21:36
@joneshf
Copy link
Member Author

joneshf commented Nov 22, 2014

Oh I guess it auto merges it, neat!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants