python: Use patch (from upstream) instead of inreplace

This patch is different than my earlier ad-hoc solution, but it took
some time until upstream devs came up with the proper fix. However, we
can't directly link to the patch (from the mercurial web repo), as it
also changes the NEWS text and the patch fails to apply. Further, the
mercurial repo is only http (no "s").
master
Samuel John 2013-08-09 18:34:34 +02:00
parent 558fa32f59
commit c1bab919b9
1 changed files with 23 additions and 4 deletions

View File

@ -92,10 +92,6 @@ class Python < Formula
f.gsub! 'DEFAULT_FRAMEWORK_FALLBACK = [', "DEFAULT_FRAMEWORK_FALLBACK = [ '#{HOMEBREW_PREFIX}/Frameworks',"
end
# Fix http://bugs.python.org/issue18071
inreplace "./Lib/_osx_support.py", "compiler_so = list(compiler_so)",
"if isinstance(compiler_so, (str,unicode)): compiler_so = compiler_so.split()"
if build.with? 'brewed-tk'
ENV.append 'CPPFLAGS', "-I#{Formula.factory('tcl-tk').opt_prefix}/include"
ENV.append 'LDFLAGS', "-L#{Formula.factory('tcl-tk').opt_prefix}/lib"
@ -263,6 +259,29 @@ class Python < Formula
end
__END__
# http://bugs.python.org/issue18071 (Remove this hung for 2.7.6!)
diff --git a/Lib/_osx_support.py b/Lib/_osx_support.py
--- a/Lib/_osx_support.py
+++ b/Lib/_osx_support.py
@@ -53,7 +53,7 @@ def _find_executable(executable, path=No
def _read_output(commandstring):
- """Output from succesful command execution or None"""
+ """Output from successful command execution or None"""
# Similar to os.popen(commandstring, "r").read(),
# but without actually using os.popen because that
# function is not usable during python bootstrap.
@@ -68,7 +68,7 @@ def _read_output(commandstring):
with contextlib.closing(fp) as fp:
cmd = "%s 2>/dev/null >'%s'" % (commandstring, fp.name)
- return fp.read().decode('utf-8').strip() if not os.system(cmd) else None
+ return fp.read().strip() if not os.system(cmd) else None
# X11 header find fix (and let homebrew handle this.)
diff --git a/setup.py b/setup.py
index 716f08e..66114ef 100644
--- a/setup.py