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

A few bug fixes and minor changes to output and exit status #36

Merged
merged 8 commits into from
Nov 25, 2015

Conversation

kadrlica
Copy link
Collaborator

Fixes a few small bugs that were introduced in the latest update (removing the "clear" option from several commands). I also found myself running several instances of easyaccess and wanting to create log files. It turned out that the way easyaccess was exiting with os._exit(0) was causing problems (http://stackoverflow.com/a/9591397/4075339). The use of os._exit was probably originally intended to deal with the forked progress bar; however, I haven't noticed any negative effect from changing to sys.exit.

@mgckind
Copy link
Owner

mgckind commented Nov 24, 2015

Thanks Alex, I'll check it soon and merge the changes...

@kadrlica
Copy link
Collaborator Author

I think that for the time being, it's safest to revert to the old os._exit(0) call. I'll keep my eyes open for any problems that this causes.

mgckind added a commit that referenced this pull request Nov 25, 2015
A few bug fixes and minor changes to output and exit status
@mgckind mgckind merged commit 182d4e7 into mgckind:master Nov 25, 2015
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