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

Instructions for installing on CentOS + minor edits #13

Closed
wants to merge 2 commits into from

Conversation

existentialcomics
Copy link

I added instructions alongside the Ubuntu instructions for Centos users, and tested it from a fresh centos 7 server. I also added a couple other notes on things that tripped me up, postgres not being initialized, and the systemd files not having the right PATH and GEM_PATH set.

and put a note about the PATH that can mess up the systemd files.
Copy link

@yiskah yiskah left a comment

Choose a reason for hiding this comment

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

Please better distinguish when the CentOS section ends and if lines such as sudo su - postgres and psql are for both CentOS and Ubuntu. If so, please duplicate them in the Ubuntu section, or create a new heading that makes it clear it is no longer just the Centos section.

sudo apt-get install postgresql postgresql-contrib

Setup a user and database for Mastodon:
Copy link

Choose a reason for hiding this comment

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

Why did you remove this line from the Ubuntu section and keep it in the CentOS section?

Copy link
Author

Choose a reason for hiding this comment

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

It wasn't removed it just got moved down. It should be more clear now with the newest commit.


sudo postgresql-setup initdb
sudo systemctl start postgresql
sudo systemctl enable postgresql

sudo su - postgres
Copy link

Choose a reason for hiding this comment

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

It's unclear if this line is just for CentOS or for both ubuntu and CentOS

Copy link

Choose a reason for hiding this comment

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

At least on debian does not have postgresql-setup.

Copy link

@wxcafe wxcafe left a comment

Choose a reason for hiding this comment

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

With that new commit it looks good to me

d6rkaiz added a commit to d6rkaiz/documentation that referenced this pull request May 4, 2017
yookoala pushed a commit to yookoala/mastodon-docs that referenced this pull request May 6, 2017
@dariusk
Copy link

dariusk commented May 7, 2017

Reviewing these changes, I think the recent merge of #207 pretty much covers everything in here.

@dariusk dariusk closed this May 7, 2017
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.

None yet

5 participants