Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

[brew doctor] remove check_for_pydistutils_cfg_in_home #31827

Closed
wants to merge 1 commit into from
Closed

[brew doctor] remove check_for_pydistutils_cfg_in_home #31827

wants to merge 1 commit into from

Conversation

wilmoore
Copy link
Contributor

This check is essentially noise since the associated issue is closed. Further, I don't think the check_for_pydistutils_cfg_in_home was correctly depicting the bug as reported anyhow. The real issue as noted by the OP of the issue was that he had put home in that file which was causing the issue.

The check_for_pydistutils_cfg_in_home method is not adequate. It actually just checks that the file ~/.pydistutils.cfg exists. This was not the issue.

Further, I have a ~/.pydistutils.cfg file and the contents are the following:

[install]
prefix=

I was only able to install powerline after having this set. I'm sure there are a ton of different variations out there. This being the case, perhaps brew cannot or should not attempt to get into such things since there are too many variables in play.

@jacknagel
Copy link
Contributor

The check_for_pydistutils_cfg_in_home method is not adequate. It actually just checks that the file ~/.pydistutils.cfg exists. This was not the issue.

Well, the point of doctor checks is not really to diagnose specific issues, but warn about things that may cause problems. It's a debugging tool.

@wilmoore
Copy link
Contributor Author

...the point of doctor checks is not really to diagnose specific issues, but warn about things that may cause problems.

That's fair; however, in this check, it may be that it is doing more harm than good --or-- at least causing some confusion.

@jacknagel
Copy link
Contributor

The referenced bug was consolidated into http://bugs.python.org/issue4655, which is still open. Since the check was added in response to real issues which do not yet appear to have been resolved by upstream, I prefer to err on the side of caution.

@jacknagel jacknagel closed this Sep 24, 2014
@wilmoore wilmoore deleted the drop-doctor-check-for-pydistutils branch October 15, 2014 06:05
@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants