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

Dont barf if home directory is not writable #814

Merged
merged 3 commits into from
Dec 15, 2016

Conversation

zachriggle
Copy link
Member

@zachriggle zachriggle commented Dec 14, 2016

I expect that there are other places where this will blow up (e.g. Mako template caching), but this is at least one of them.

Fixes #799

@zachriggle zachriggle changed the base branch from dev to stable December 14, 2016 17:26
@zachriggle zachriggle force-pushed the issue_799 branch 2 times, most recently from 0ba0a6f to 6b4383e Compare December 14, 2016 17:26
@zachriggle zachriggle added this to the 3.2.0 milestone Dec 14, 2016
@zachriggle
Copy link
Member Author

CI passes, coverage decreased despite adding tests.

Copy link
Contributor

@TethysSvensson TethysSvensson left a comment

Choose a reason for hiding this comment

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

LGTM

@zachriggle zachriggle deleted the issue_799 branch January 4, 2017 18:00
Kyle-Kyle pushed a commit to Kyle-Kyle/pwntools that referenced this pull request Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants