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.

Sign up for GitHub

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

Jump to bottom

Fix AttributeError: module 'os' has no attribute 'mkdirs'. #391

Merged
merged 1 commit into from
Jul 19, 2023
Merged

Fix AttributeError: module 'os' has no attribute 'mkdirs'. #391

merged 1 commit into from
Jul 19, 2023

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Jul 18, 2023 •
edited
Loading

Fixes #390
Fixes #392

Traceback (most recent call last):
File "/usr/share/hackingtool/hackingtool.py", line 105, in
os.mkdirs(archive, exist_ok=True)
^^^^^^^^^
AttributeError: module 'os' has no attribute 'mkdirs'.

https://docs.python.org/3/library/os.html#os.makedirs

@Z4nzu This is an urgent fix. Sorry about this typo.

@cclauss cclauss changed the title Typo makedirs()!!! Fix AttributeError: module 'os' has no attribute 'mkdirs'. Jul 18, 2023
@zuzu1f
Copy link

zuzu1f commented Jul 18, 2023

thank you

@cclauss
Copy link
Contributor Author

cclauss commented Jul 19, 2023

@Z4nzu Please review because without this change the current code crashes like

@cclauss cclauss mentioned this pull request Jul 19, 2023
Closed
@Z4nzu Z4nzu self-requested a review July 19, 2023 10:18
@Z4nzu Z4nzu merged commit ca8dcdb into Z4nzu:master Jul 19, 2023
2 checks passed
@cclauss cclauss deleted the patch-4 branch July 19, 2023 10:25
Menacei pushed a commit to Menacei/hackingtool that referenced this pull request Aug 12, 2023
@Z4nzu @Menacei
Merge pull request Z4nzu#391 from cclauss/patch-4
383bba7 
Fix AttributeError: module 'os' has no attribute 'mkdirs'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Z4nzu Z4nzu Z4nzu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

I need help solving this issue
3 participants
@cclauss @zuzu1f @Z4nzu