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

[FIX] Wrong build path in install.sh #11879

Merged
merged 4 commits into from
Sep 11, 2018
Merged

Conversation

geekgonecrazy
Copy link
Contributor

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11879 August 27, 2018 19:15 Inactive
rodrigok
rodrigok previously approved these changes Aug 27, 2018
@rodrigok rodrigok changed the title [Fix] Correct build path in install.sh [FIX] Wrong build path in install.sh Aug 27, 2018
@rodrigok rodrigok added this to the 0.69.0 milestone Aug 27, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11879 August 27, 2018 19:28 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11879 August 27, 2018 19:29 Inactive
tar zxf rocket.chat.tgz && rm rocket.chat.tgz
gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 0E163286C20D07B9787EBE9FD7F9D0414FD08104

curl -SLf "https://releases.rocket.chat/latest/download/" -o rocket.chat.tgz
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the develop release is actually at https://releases.rocket.chat/develop/download/ .. or did you change to latest on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it's a script never used, and wouldn't be updated between master and develop I figured it better to have anyone using it to actually install latest stable

@sampaiodiego sampaiodiego merged commit 3b93f63 into develop Sep 11, 2018
@sampaiodiego sampaiodiego deleted the fix/broken-build-path branch September 11, 2018 12:03
This was referenced Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants