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

Error when reinstalling all #216

Closed
AlexanderProd opened this issue Jan 10, 2019 · 12 comments · Fixed by #246
Closed

Error when reinstalling all #216

AlexanderProd opened this issue Jan 10, 2019 · 12 comments · Fixed by #246
Labels
bug Something isn't working reinstallation

Comments

@AlexanderProd
Copy link
Contributor

AlexanderProd commented Jan 10, 2019

When trying to reinstall all from the main menu I'm getting the following error seems like there's a problem with paths and directories.

I've been using version 2.3

Traceback (most recent call last):
  File "/usr/local/bin/shallow-backup", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/shallow_backup/__main__.py", line 159, in cli
    reinstall_all_sb(dotfiles_path, packages_path, fonts_path, configs_path)
  File "/usr/local/lib/python3.7/site-packages/shallow_backup/reinstall.py", line 129, in reinstall_all_sb
    reinstall_dots_sb(dotfiles_path)
  File "/usr/local/lib/python3.7/site-packages/shallow_backup/reinstall.py", line 26, in reinstall_dots_sb
    copyfile(file, home_path)
  File "/usr/local/Cellar/python/3.7.1/Frameworks/Python.framework/Versions/3.7/lib/python3.7/shutil.py", line 121, in copyfile
    with open(dst, 'wb') as fdst:
IsADirectoryError: [Errno 21] Is a directory: '/Users/alexanderhoerl'
@alichtman alichtman added bug Something isn't working reinstallation labels Jan 10, 2019
@alichtman
Copy link
Owner

Hmm. I'll look into this as soon as I have time. Looks like an error introduced in the last version, maybe?

@alichtman alichtman pinned this issue Jan 10, 2019
@AlexanderProd
Copy link
Contributor Author

AlexanderProd commented Jan 11, 2019

Just tried it in version 2.2 installed from pip and the bug happens there as well with the same error message. I've also created a new backup and deleted the config file.

@alichtman
Copy link
Owner

Related to #209?

@AlexanderProd
Copy link
Contributor Author

AlexanderProd commented Jan 12, 2019 via email

@alichtman
Copy link
Owner

Well, it's not necessarily specific to the backup. The root of the issue is that a copying function call fails.

@AlexanderProd
Copy link
Contributor Author

AlexanderProd commented Jan 12, 2019

Oh yeah you're right I guess they have a similar root. Does reinstalling all work for you?

@alichtman
Copy link
Owner

image

Yep, I get the same error. It's a bit weird because the error is thrown on a line that should be impossible to hit with this code. Err: Is a directory but thrown on a line that can only be hit if the path is not a directory.

image

@robbixc
Copy link
Contributor

robbixc commented Feb 26, 2019

Hey :) any news on this? I also hit the bug. I'm on ubuntu 18.04

@alichtman
Copy link
Owner

@robbixc Unfortunately, I haven't had any time to work on this. I likely won't have any time for the foreseeable future, either. I'd like to bring on another maintainer for this project... But that's a conversation for a different thread.

Short answer is: no progress.

@alichtman alichtman unpinned this issue Apr 1, 2019
@heysupratim
Copy link
Contributor

Hey just informing that this issue still exists on 3.2

@alichtman
Copy link
Owner

alichtman commented Nov 29, 2019

Happy to review a PR for this if you put one together. I don't have time to resolve this bug at the moment, and it doesn't interfere with my workflow, so it's not high priority for me. I agree it's an important bug to resolve, it just hasn't hit the top of my list in a long time. @heysupratim

@heysupratim
Copy link
Contributor

Hope this helps 🤗

alichtman added a commit that referenced this issue Jan 27, 2020
* Fixes #216

* Added Test Coverage for reinstallation of dotfiles regarding #216

* Refactor reinstallation code

Co-authored-by: Supratim <sup270792@gmail.com>
alichtman added a commit that referenced this issue Mar 22, 2020
* Fixes #216

* Added Test Coverage for reinstallation of dotfiles regarding #216

* Refactor reinstallation code

Co-authored-by: Supratim <sup270792@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reinstallation
Projects
None yet
4 participants