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

Network check #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Network check #20

wants to merge 2 commits into from

Conversation

vanosg
Copy link
Collaborator

@vanosg vanosg commented Mar 30, 2014

Check to see if network connectivity is available before running script- if we can't connect to the internet, how can we update!

Geo Van Osterom added 2 commits March 26, 2014 23:05
If there is no internet connection, subsequent downloads will (obviously) not work- lets catch this error early.
Current code structure allows the failure to occur but continues on as a success
In the off case the user has already set up an ssh key as the non-priv user, this will make sure that same user's ssh key is used
to connect with github, instead of the root key which is what would be used when this script is executed via sudo. Logname
gives us the initiating user's login name, instead of using root.
@elcojacobs
Copy link
Member

The run git as user commit, does it actually do anything?
It checks the current user and then runs git as the current user, which I would think is the default behavior?

@vanosg
Copy link
Collaborator Author

vanosg commented Mar 31, 2014

logname checks the user who initiated the sudo command. git fetch is being run as the root user, as it was called via the sudo command.

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