Skip to content

[WIP] Added make uninstall#32

Open
alquerci wants to merge 1 commit into
nylen:masterfrom
alquerci:add/make_uninstall
Open

[WIP] Added make uninstall#32
alquerci wants to merge 1 commit into
nylen:masterfrom
alquerci:add/make_uninstall

Conversation

@alquerci

Copy link
Copy Markdown
Collaborator
Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Related tickets
License GNU GPLv3
  • Ensure that works

@alquerci alquerci added this to the 1.1 milestone Aug 25, 2014
@alquerci alquerci force-pushed the add/make_uninstall branch from 06f2a3b to 604bb7f Compare August 28, 2014 08:27
@alquerci alquerci force-pushed the add/make_uninstall branch from 604bb7f to ca3a839 Compare August 28, 2014 15:00
@alquerci alquerci modified the milestones: 1.1, 1.2 Sep 4, 2014
@nylen

nylen commented Sep 17, 2014

Copy link
Copy Markdown
Owner

This is a good feature. A few questions:

  • Shouldn't we remove directories as well? (/usr/share/doc/cyg-apt/, /usr/lib/python2.7/site-packages/cygapt/, etc.)
  • Shouldn't we remove egg-info files like /usr/lib/python2.7/site-packages/cygapt-1.1.0rc1_91.ge480502.dirty-py2.7.egg-info?
  • Any problem with adding -v to rm options to display progress?

@alquerci

Copy link
Copy Markdown
Collaborator Author
  • Directories should be keep because we don't know what the user do with theses.
  • For egg-info files I don't know maybe yes (about old one).
  • rm -v nice idea

We need to learn about what we are allow to do on make uninstall.

We currently work on a Makefile that follow GNU coding standards (ref) co this implementation becomes out of date.

@alquerci

Copy link
Copy Markdown
Collaborator Author

Closing in favor of #66

@alquerci alquerci closed this Sep 27, 2014
@alquerci alquerci deleted the add/make_uninstall branch September 27, 2014 15:15
@alquerci alquerci restored the add/make_uninstall branch April 11, 2015 12:29
@alquerci

Copy link
Copy Markdown
Collaborator Author

Reopen this since #69 has been planned for the version 2.0.

@alquerci alquerci reopened this Apr 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants