confirming dependencies when uninstall package #4245

aodag · 2017-01-21T08:51:58Z

when i uninstall package, i want to know packages depends to that.
this PR adds checking packages depends to uninstalling, and confirming.

make confirm_dependencies to helper function
remove uninstalled package from installed_packages
change algorithm to detecting dependencies
add functional tests for various situations

xavfernandez

We'll need a few tests for this

xavfernandez · 2017-01-28T15:31:40Z

pip/req/req_install.py

+                [dist for d in dist.requires() if d.key == self.req.name])
+        if not dep_keys:
+            return True
+        print("%s is depended from:" % self.req)


You should be using logger.info to provide feedback to the user.

xavfernandez · 2017-02-03T09:24:39Z

pip/req/req_install.py

+            return True
+        logger.info("%s is depended from:" % self.req)
+        for dep in dep_keys:
+            print(dep)


erm, same comment for this print :o

xavfernandez · 2017-02-03T09:25:15Z

A rebase on master and a few tests would also be needed 👍

aodag · 2017-04-08T16:23:44Z

this PR is affected from 893f0b0.

xavfernandez · 2017-04-08T17:57:39Z

Yes, we are trying to simplify the RequirementSet and RequirementInstall object.
confirm_dependencies could also be moved as an helper to pip/commands/uninstall.py

xavfernandez · 2017-04-08T17:59:59Z

Please also note that we don't use CHANGES.txt anymore for the changelog, cf https://pip.pypa.io/en/latest/development/#adding-a-news-entry

BrownTruck · 2017-05-19T06:00:20Z

Hello!

I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the master branch into this pull request or rebase this pull request against master then it will eligible for code review and hopefully merging!

aodag · 2017-07-14T08:03:48Z

@xavfernandez please review this PR!

zaazbb · 2017-08-20T07:51:56Z

need confirming dependencies when upgrade package also.
see #4681

There has been a test added.

aodag · 2017-08-20T09:33:42Z

@zaazbb OK, I'm sure to need confirming on upgrading packaging too.
I'll consider this PR can include that feature.

pradyunsg · 2017-08-20T13:01:54Z

I'd suggest you make another PR for that -- that PR would need discussion. :)

aodag · 2017-08-21T03:53:46Z

@pradyunsg i think so.
yes, this PR is completed.

BrownTruck · 2017-09-01T13:15:07Z

Hello!

I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the master branch into this pull request or rebase this pull request against master then it will eligible for code review and hopefully merging!

pradyunsg · 2017-10-24T09:43:16Z

Pinging @aodag! :)

pradyunsg · 2017-10-24T09:46:02Z

Linking to #4001

aodag · 2017-12-09T14:04:07Z

@pfmoore i rewrote to check once for everything and added tests for some situations.

aodag · 2017-12-13T07:27:26Z

travis-ci terminated the test for pypy3.

The job exceeded the maximum time limit for jobs, and has been terminated.

aodag · 2018-02-15T03:48:48Z

What else can I do?

aodag · 2018-03-08T06:25:55Z

@pfmoore what should i do somthing?

pradyunsg · 2018-03-08T06:29:23Z

Hey @aodag!

Sorry for the lack of a response lately. My suggestion would be to wait for a bit for #5000 and then, once it's merged, reuse relavent parts of it to run a check, once, before the installation.

BrownTruck · 2018-05-23T04:15:01Z

Hello!

I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the master branch into this pull request or rebase this pull request against master then it will eligible for code review and hopefully merging!

…-deps

aodag · 2018-06-09T06:35:26Z

BrownTruck, you are my best friend.

BrownTruck · 2018-07-27T19:00:02Z

Hello!

I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the master branch into this pull request or rebase this pull request against master then it will eligible for code review and hopefully merging!

pradyunsg · 2019-05-23T04:28:25Z

Closing this since it's bitrotten. Please file a new PR if there's still interest in this.

xavfernandez previously requested changes Jan 29, 2017

View reviewed changes

xavfernandez reviewed Feb 3, 2017

View reviewed changes

aodag force-pushed the feature-uninstall-deps branch from cec6f4e to 32f6645 Compare February 12, 2017 10:39

dstufft added the needs rebase or merge PR has conflicts with current master label Apr 1, 2017

aodag changed the title ~~confirming dependencies when uninstall package~~ WIP: confirming dependencies when uninstall package Apr 8, 2017

aodag force-pushed the feature-uninstall-deps branch from 7b48f3e to 66ca197 Compare April 8, 2017 19:06

pypa-bot removed the needs rebase or merge PR has conflicts with current master label Apr 8, 2017

BrownTruck added the needs rebase or merge PR has conflicts with current master label May 19, 2017

added feature to confirm dependencies of uninstalling packages

17bc4b7

aodag force-pushed the feature-uninstall-deps branch from aa54594 to 17bc4b7 Compare July 14, 2017 06:35

pypa-bot removed the needs rebase or merge PR has conflicts with current master label Jul 14, 2017

aodag added 3 commits July 14, 2017 16:11

refactored to call get_installed_distributions each confirming

560dfe0

fix arguments

58d5699

flake8

3195be2

pradyunsg added the type: enhancement Improvements to functionality label Jul 14, 2017

fix linter

9c58308

aodag changed the title ~~WIP: confirming dependencies when uninstall package~~ confirming dependencies when uninstall package Jul 23, 2017

pradyunsg mentioned this pull request Aug 19, 2017

Ask a confirm message when upgrade or uninstall a package who depend by other packages. #4681

Closed

pradyunsg added type: feature request Request for a new feature and removed type: enhancement Improvements to functionality labels Oct 24, 2017

Merge branch 'master' into feature-uninstall-deps

137a610

aodag changed the title ~~confirming dependencies when uninstall package~~ WIP: confirming dependencies when uninstall package Oct 26, 2017

aodag added 7 commits November 1, 2017 22:19

Merge branch 'master' into feature-uninstall-deps

5c1210d

replace confirming process

a007ae4

flake8

8629666

addec canonicalize name of requirement

c7fd607

added test cases for confirmation to uninstall

362cd63

added slight word change

1fd3099

added test for recursive dependencies

de88746

aodag changed the title ~~WIP: confirming dependencies when uninstall package~~ confirming dependencies when uninstall package Dec 9, 2017

BrownTruck added the needs rebase or merge PR has conflicts with current master label May 23, 2018

Merge remote-tracking branch 'upstream/master' into feature-uninstall…

05091ad

…-deps

pypa-bot removed the needs rebase or merge PR has conflicts with current master label Jun 9, 2018

BrownTruck added the needs rebase or merge PR has conflicts with current master label Jul 27, 2018

pradyunsg closed this May 23, 2019

lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 22, 2019

lock bot locked as resolved and limited conversation to collaborators Jun 22, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

confirming dependencies when uninstall package #4245

confirming dependencies when uninstall package #4245

aodag commented Jan 21, 2017 •

edited

Loading

xavfernandez left a comment

xavfernandez Jan 28, 2017

xavfernandez Feb 3, 2017

xavfernandez commented Feb 3, 2017

aodag commented Apr 8, 2017

xavfernandez commented Apr 8, 2017 •

edited

Loading

xavfernandez commented Apr 8, 2017

BrownTruck commented May 19, 2017

aodag commented Jul 14, 2017

zaazbb commented Aug 20, 2017 •

edited

Loading

aodag commented Aug 20, 2017

pradyunsg commented Aug 20, 2017 •

edited

Loading

aodag commented Aug 21, 2017

BrownTruck commented Sep 1, 2017

pradyunsg commented Oct 24, 2017

pradyunsg commented Oct 24, 2017

aodag commented Dec 9, 2017

aodag commented Dec 13, 2017

aodag commented Feb 15, 2018

aodag commented Mar 8, 2018

pradyunsg commented Mar 8, 2018

BrownTruck commented May 23, 2018

aodag commented Jun 9, 2018

BrownTruck commented Jul 27, 2018

pradyunsg commented May 23, 2019

confirming dependencies when uninstall package #4245

confirming dependencies when uninstall package #4245

Conversation

aodag commented Jan 21, 2017 • edited Loading

xavfernandez left a comment

Choose a reason for hiding this comment

xavfernandez Jan 28, 2017

Choose a reason for hiding this comment

xavfernandez Feb 3, 2017

Choose a reason for hiding this comment

xavfernandez commented Feb 3, 2017

aodag commented Apr 8, 2017

xavfernandez commented Apr 8, 2017 • edited Loading

xavfernandez commented Apr 8, 2017

BrownTruck commented May 19, 2017

aodag commented Jul 14, 2017

zaazbb commented Aug 20, 2017 • edited Loading

aodag commented Aug 20, 2017

pradyunsg commented Aug 20, 2017 • edited Loading

aodag commented Aug 21, 2017

BrownTruck commented Sep 1, 2017

pradyunsg commented Oct 24, 2017

pradyunsg commented Oct 24, 2017

aodag commented Dec 9, 2017

aodag commented Dec 13, 2017

aodag commented Feb 15, 2018

aodag commented Mar 8, 2018

pradyunsg commented Mar 8, 2018

BrownTruck commented May 23, 2018

aodag commented Jun 9, 2018

BrownTruck commented Jul 27, 2018

pradyunsg commented May 23, 2019

aodag commented Jan 21, 2017 •

edited

Loading

xavfernandez commented Apr 8, 2017 •

edited

Loading

zaazbb commented Aug 20, 2017 •

edited

Loading

pradyunsg commented Aug 20, 2017 •

edited

Loading