Stop failing sitecustomize if cellar not in path

[deiga]

This fixes issues when calling python scripts print 'import sitecustomize' failed; use -v for traceback

[samueljohn]

I don't know how this can happen, that your python is at HOMEBREW_PREFIX but has not HOMEBREW/Cellar in the sys.path.

Having said that, the fix looks reasonable to me. Indeed, in the upcoming python rework, I have a similar line in:

          if sys.path.startswith('#{HOMEBREW_PREFIX}/Cellar'):
              sys.path.remove(p)

[deiga]

yeah, I don't know either how that could have even happened, but it caused some headaches with the weird error message :)

Of course that also could be extended with an else that outputs a warning that something is wrong with the pythonpath :)

[samueljohn]

Did you first brew install python and then install the python from python.org (which also writes /usr/local/bin/python)?

We must not output from the sitecustomize.py, otherwise the output of executable python scripts is no longer parseable.

[deiga]

No, I only had system python installed and then I installed brew install python, I never downloaded python or installed from another source

[samueljohn]

will be addressed by #18588.

[samueljohn]

I pulled this as a kind of hot-fix until #18588 arrives.

[samueljohn]

Thank you!