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

add build tools install instructions for CentOS #210

Merged
merged 1 commit into from
Jul 11, 2017

Conversation

jshimko
Copy link
Contributor

@jshimko jshimko commented Jul 7, 2017

Tested and confirmed working on CentOS 7.

To test:

# hop into a fresh CentOS image
docker run -it --rm centos:7 /bin/bash

# install Reaction dependencies
yum groupinstall "Development Tools"
yum install epel-release
yum install GraphicsMagick nodejs

npm i -g reaction-cli

reaction init

cd reaction 

# because we're currently root inside the container...
export METEOR_ALLOW_SUPERUSER=true

reaction

@jshimko jshimko requested a review from brent-hoover July 7, 2017 18:40
@brent-hoover
Copy link
Collaborator

When I follow these instructions everything works until I start reaction and then I get the following error and Reaction doesn't start

You are attempting to run Meteor as the 'root' superuser. If you are
developing, this is almost certainly *not* what you want to do and will likely
result in incorrect file permissions. However, if you are running this command
in a build process (CI, etc.), or you are absolutely sure you know what you are
doing, set the METEOR_ALLOW_SUPERUSER environment variable or pass
--allow-superuser to proceed.

Even with METEOR_ALLOW_SUPERUSER or --allow-superuser, permissions in your app
directory will be incorrect if you ever attempt to perform any Meteor tasks as
a normal user. If you need to fix your permissions, run the following command
from the root of your project:

  sudo chown -Rh <username> .meteor/local

@jshimko
Copy link
Contributor Author

jshimko commented Jul 8, 2017

Yeah sorry, forgot to mention that. You're root by default in a container, so you need to tag on Meteor's --allow-superuser flag or run export METEOR_ALLOW_SUPERUSER=true first.

@brent-hoover
Copy link
Collaborator

Wouldn't we want to add that to the docs as well?

@jshimko
Copy link
Contributor Author

jshimko commented Jul 10, 2017

No, I don't think so. There's nothing OS-specific about that warning. It happens on every Linux/Unix OS. And I think the warning itself does a pretty good job of explaining your options. Also, anyone that's developing on a Linux server should probably already be aware of the dangers of running everything as root.

@brent-hoover brent-hoover merged commit d78b016 into development Jul 11, 2017
@brent-hoover brent-hoover deleted the centos-install branch July 11, 2017 01:30
@spencern spencern mentioned this pull request Jul 17, 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.

2 participants