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

Uninstall deletes cherokee.conf #953

Closed
danielniccoli opened this issue Apr 3, 2013 · 4 comments
Closed

Uninstall deletes cherokee.conf #953

danielniccoli opened this issue Apr 3, 2013 · 4 comments

Comments

@danielniccoli
Copy link
Contributor

Executing make uninstall deletes cherokee.conf. I am uncertain whether this is common practice, but I did not expect that.

@skinkie
Copy link
Member

skinkie commented Jul 27, 2013

The behavior is controlled here below, but do validate if we should keep or delete.

https://github.com/cherokee/webserver/blob/master/Makefile.am#L143

@danielniccoli
Copy link
Contributor Author

Some months ago I asked a question in the Debian mailing list and in the IRC channel #debian on freenode. The question was whether make uninstall should delete configuration files. The answer was that there would not be any policy that tells you whether it should, or should not be deleted. It's essentially the decision of the programmers.

I believe that it should not be deleted. Here is why:

  1. I usually do use make uninstall before installing a new version of Cherokee. That requires me to always rename the config file before doing so, and renaming it back afterwards.
  2. If I apt-get remove or aptitude remove a package, the config file is usually not deleted unless I apt... purge the package. That also led me into opening this ticket.

@skinkie
Copy link
Member

skinkie commented Jul 28, 2013

My suggestion: come up with a patch that has make purge which deletes everything and make uninstall that keeps the config.

skinkie added a commit that referenced this issue Jan 2, 2014
@skinkie skinkie closed this as completed Jan 2, 2014
@danielniccoli
Copy link
Contributor Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants