Zvezdan Petkovic added the comment:
Ned,
I just signed the contributor agreement form.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13631
Changes by Zvezdan Petkovic zvez...@computer.org:
--
nosy: +zvezdan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14073
___
___
Python-bugs-list
Zvezdan Petkovic zvez...@zope.com added the comment:
I did not test against a readline build.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13631
New submission from Zvezdan Petkovic zvez...@zope.com:
Problem
===
The changes in r87356 for py3k and r87358 for python-2.7 described in issue
9907 have broken parsing of the initialization file .editrc under editline
emulation of readline on Mac OS X.
Background
==
Both
Changes by Zvezdan Petkovic zvez...@zope.com:
Added file: http://bugs.python.org/file24044/readline-3.2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13631
Zvezdan Petkovic zvez...@zope.com added the comment:
Does a translation really need to be in Python?
I use .editrc file in my home directory with this content:
python:bind ^I rl_complete
and everything works fine.
--
nosy: +zvezdan
___
Python
Changes by Zvezdan Petkovic zvez...@zope.com:
--
nosy: +zvezdan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8065
___
___
Python-bugs-list
Zvezdan Petkovic zvez...@zope.com added the comment:
Ronald,
That bit is not in the trunk, should it be forward ported to the trunk?
Yes, that should be applied to trunk and 3.x to make it work on Mac OS X 10.5
(Leopard). I indicated that in msg98979.
The explanation why that part
Zvezdan Petkovic zvez...@zope.com added the comment:
OK, so issue4204 patch is causing problems.
As I mentioned before we could choose to #ifdef __FreeBSD__ or check whether
__APPLE__ is not defined.
I'm attaching a new patch for 2.6.5 (2.6 branch) that uses the __APPLE__ check
Zvezdan Petkovic zvez...@zope.com added the comment:
The patch with the same correction against trunk is attached so that Meador can
test it.
--
Added file: http://bugs.python.org/file16161/readline-libedit-trunk.patch
___
Python tracker rep
Zvezdan Petkovic zvez...@zope.com added the comment:
Meador, I just looked at your patch (it seems we were posting patches at about
the same time).
That's a good fix too.
Barry it's your call.
--
___
Python tracker rep...@bugs.python.org
http
Changes by Zvezdan Petkovic zvez...@zope.com:
Removed file: http://bugs.python.org/file16160/readline-libedit-2.6.5-1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Changes by Zvezdan Petkovic zvez...@zope.com:
Removed file: http://bugs.python.org/file16161/readline-libedit-trunk.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Changes by Zvezdan Petkovic zvez...@zope.com:
Added file: http://bugs.python.org/file16162/readline-libedit-2.6.5-fix1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Changes by Zvezdan Petkovic zvez...@zope.com:
Added file: http://bugs.python.org/file16163/readline-libedit-trunk.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Zvezdan Petkovic zvez...@zope.com added the comment:
This seems to be a 64-bit issue.
The 32-bit build passes this test.
echo ${TESTPYTHON} ${TESTPROG} ${TESTOPTS}
/Users/zvezdan/opt/snapshot/2.6.5/bin/python -E -tt
/Users/zvezdan/opt/snapshot/2.6.5/lib/python2.6/test/test_macostools.py -l
Zvezdan Petkovic zvez...@zope.com added the comment:
How about this example from Mac OS X Reference Library Property List
Programming Guide:
http://developer.apple.com/mac/library/documentation/Cocoa/Conceptual/PropertyLists/QuickStartPlist/QuickStartPlist.html
Perhaps the fact that Apple
Zvezdan Petkovic zvez...@zope.com added the comment:
Also, on a Mac computer:
- start Xcode 3.2.1 and select Help-Developer Documentation
- type in the search box in the top right corner property list
- the second hit is Property List Programming Guide; click on it
- the second page gives
Zvezdan Petkovic zvez...@zope.com added the comment:
This is a configuration bug:
/Users/barry/projects/python/python26/Modules/readline.c -o
build/temp.macosx-10.3-ppc-2.6 ...
Why is it trying to build using macosx-10.3 target?
I bet if you specify MACOSX_DEPLOYMENT_TARGET=10.5
Zvezdan Petkovic zvez...@zope.com added the comment:
Barry,
I'm sorry, I only now noticed this line in your compilation on 10.5:
/Users/barry/projects/python/python26/Modules/readline.c:41: error: conflicting
types for 'completion_matches'
This problem was introduced by a patch in issue4204
Zvezdan Petkovic zvez...@zope.com added the comment:
The readline-libedit-2.6.5.patch is attached.
The patch was applied and python built in several configurations on Mac
OS X 10.6 (Snow Leopard). There is no regression (details below).
Can somebody else test on Mac OS X 10.5 (Leopard)?
32
Zvezdan Petkovic zvez...@zope.com added the comment:
I forgot to add that the patch for 2.6.5 is based on:
http://svn.python.org/view?rev=74970view=rev
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Zvezdan Petkovic zvez...@zope.com added the comment:
Brett,
what does this command return for you?
otool -L /path/to/lib/python2.4/lib-dynload/readline.so
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Zvezdan Petkovic zvez...@zope.com added the comment:
Make that python2.6 in the command above. :-)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Zvezdan Petkovic zvez...@zope.com added the comment:
I assume you had to pass extra -I and -L flags when compiling with GNU
readline. Right?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6877
Zvezdan Petkovic zvez...@zope.com added the comment:
Brett,
IMO, your backtrace only implies that readline module was built
believing it has libedit (i.e., include files were system ones from
/usr/include).
However, the following scenario is possible. Some packaging tools
choose to divide
Zvezdan Petkovic zvez...@zope.com added the comment:
The patch `readline-libedit.patch` has the following problems:
- a typo causing an undefined variable error;
- a missing #endif;
- it doesn't refer to the new __APPLE__ specific doc string anywhere.
It fails to compile `readline.c
New submission from Zvezdan Petkovic zvez...@zope.com:
The attached patch enables compilation and use of the readline module on
Mac OS X 10.5 (Leopard) and 10.6 (Snow Leopard).
It utilizes the native editline library (used for emulation of the
readline library on Mac).
I used the patch
Zvezdan Petkovic zvez...@zope.com added the comment:
Changed type to crash because compilation of readline module without
this patch was causing Python to crash with BusError.
--
type: compile error - crash
___
Python tracker rep...@bugs.python.org
Zvezdan Petkovic zvez...@zope.com added the comment:
I worked around the issue mentioned in msg82619.
The readline patch (issue 6877) is not adversely affected by the patch
applied in this issue. This issue can remain closed AFAIC.
--
___
Python
Zvezdan Petkovic zvez...@zope.com added the comment:
When testing the patch make sure that your readline module has been
actually linked against editline library rather then some copy of GNU
readline from MacPorts or Fink.
Assuming --prefix=${HOME}/opt, the output of
otool -L ~/opt/lib
Zvezdan Petkovic zvez...@zope.com added the comment:
This patch could potentially break non-Mac OS X systems.
Fortunately, I have a patch that works with systems that use GNU
readline and systems that use editline emulation. See issue 6877.
Unfortunately, I was lingering for over a year
Zvezdan Petkovic zvez...@zope.com added the comment:
Also, the patch from issue 6877 changes setup.py in a way that enables
build of the readline module on Leopard as well.
Such build is used for about two years already (Python 2.4) by several
people in my company and nobody noticed any
Zvezdan Petkovic zvez...@zope.com added the comment:
It would also be nice if one could programmaticly detect if readline
is linked to libedit
There's this rl_library_version constant defined in editline/readline C
libraries that the attached patch uses.
Perhaps, if we can expose its value
Zvezdan Petkovic zvez...@zope.com added the comment:
Part of this patch committed in -r 67098 breaks Mac OS X 10.5,
because the completion_matches() function is already declared in
/usr/include/readline/readline.h and the new declaration conflicts with
it.
If the lack of this declaration
35 matches
Mail list logo
