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

Do not litter working dir #235

Merged
merged 5 commits into from
Apr 17, 2015
Merged

Do not litter working dir #235

merged 5 commits into from
Apr 17, 2015

Conversation

hynek
Copy link
Contributor

@hynek hynek commented Apr 16, 2015

This PR cleans up the handling of temporary files such that the CWD is not filled with them.

It changes the approach to creating a directory which it kills afterwards instead of keeping track of single files.

Some adjacent lines have been also cleaned up.

No code except tests is touched by this PR.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 46aed85 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 46aed85 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 46aed85 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 2956601 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 2956601 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 2956601 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling 2956601 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@hynek
Copy link
Contributor Author

hynek commented Apr 16, 2015

One thing I would suggest is to add the snowman unconditionally to all tmp files to simplify the code some more.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling dda3ea8 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 95.28% when pulling dda3ea8 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

hynek added 3 commits April 16, 2015 14:51
Generating tmp files properly ensures the cleanup.
This allows to clean up behind ourselves more effectively and requires less
logic.
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.28%) to 92.93% when pulling c57dd68 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.28%) to 92.93% when pulling c57dd68 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.28%) to 92.93% when pulling c57dd68 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.28%) to 92.93% when pulling c57dd68 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.47%) to 94.74% when pulling f75553d on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.47%) to 94.74% when pulling f75553d on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.47%) to 94.74% when pulling f75553d on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.64%) to 92.57% when pulling f75553d on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.37%) to 91.83% when pulling f75553d on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.26%) to 92.95% when pulling 4813c0e on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.26%) to 92.95% when pulling 4813c0e on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 95.3% when pulling 4813c0e on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.62%) to 92.59% when pulling 1902c01 on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 95.3% when pulling e90680f on hynek:do-not-litter-working-dir into cf7ede8 on pyca:master.

@hynek
Copy link
Contributor Author

hynek commented Apr 16, 2015

I’ve simplified a bunch, r4r!

@reaperhulk
Copy link
Member

LGTM. The way SSL_CTX_load_verify_locations works is terrible. I haven't dug deeply yet but I bet there's a way to do that without requiring files on disk.

@reaperhulk
Copy link
Member

(But that is an issue for a future PR)

reaperhulk added a commit that referenced this pull request Apr 17, 2015
@reaperhulk reaperhulk merged commit 0145533 into pyca:master Apr 17, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants