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

better handling of dotfiles for ~/.common_env #1

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

Conversation

ninjabong
Copy link
Member

made it more like legacy-common so it also handles .profile as last resort, creating one if needed.

@ninjabong
Copy link
Member Author

Hrmm will need to update since CI failed with a traceback on my regex, however it's output clearly indicates the existence of both a .bash_profile and .zshrc which are not created "out-of-the-box" in ~/.

@ninjabong
Copy link
Member Author

bump

@rricard
Copy link
Contributor

rricard commented Nov 19, 2014

Let's rebuild it on travis !

In case travis throws weird errors, just relaunch it !

@hacfi
Copy link

hacfi commented Nov 30, 2014

Will this PR also hide the errors for the non-existing files?

@ninjabong
Copy link
Member Author

The errors can be safely ignored, but we do need to check for the existence of certain files before we go and blindly squash people's dotfiles, see osxc/legacy-common#8 for more reasoning behind why.

@hacfi
Copy link

hacfi commented Dec 1, 2014

Thanks for the info! At first I thought something went wrong. Love this tool!

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.

3 participants