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

pkg install: not reinstall a package by default #1773

Closed
dmpetrov opened this issue Mar 22, 2019 · 0 comments · Fixed by #2142
Closed

pkg install: not reinstall a package by default #1773

dmpetrov opened this issue Mar 22, 2019 · 0 comments · Fixed by #2142
Assignees
@efiop
Labels
feature request Requesting a new feature p2-medium Medium priority, should be done, but less important

Comments

@dmpetrov
Copy link
Member

dmpetrov commented Mar 22, 2019 •
edited
Loading 

$ dvc pkg install https://github.com/dmpetrov/tag_classifier classify/
....
$ dvc pkg install https://github.com/dmpetrov/tag_classifier classify/
Package 'tag_classifier' was already installed.

Probably we need an option like --force in case the reinstallation is needed.

UPDATE:
Unit tests need to be created first #1772
@dmpetrov dmpetrov added the feature request Requesting a new feature label Mar 22, 2019
@efiop efiop added the p2-medium Medium priority, should be done, but less important label Mar 22, 2019
@efiop efiop self-assigned this Jun 12, 2019
efiop added a commit to efiop/dvc that referenced this issue Jun 17, 2019
@efiop
pkg: introduce -f|--force for pkg install
4947069 
Fixes iterative#1773

Signed-off-by: Ruslan Kuprieiev <ruslan@iterative.ai>
@efiop efiop mentioned this issue Jun 17, 2019
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@efiop efiop

Labels
feature request Requesting a new feature p2-medium Medium priority, should be done, but less important
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

pkg: introduce -f|--force for pkg install efiop/dvc
2 participants
@efiop @dmpetrov