[issue28845] Clean up known issues for AIX
New submission from Eric N. Vander Weele: This patch cleans up Misc/README.AIX for addressed known issues. Issues that have been marked fixed: #11184, #11185 Issues resolved by new AIX version: #1745108 Issues resolved, but not yet marked fixed/closed: #11188 Additionally, it looks like #10709 can be closed out as well. For #1745108 and #11188, I have verified they are addressed as of Python 3.5.2 on AIX 7.1 locally. The Python Buildbot is failing to build the curses module, but I believe Setup.local is needed for _curses and _curses_panel. I have gotten the aforementioned curses modules building locally and will figure out the appropriate channels getting Python's PPC64 AIX Buildbot updated independently. ---------- assignee: docs@python components: Documentation files: cleanup-readme-aix.patch keywords: patch messages: 282105 nosy: David.Edelsohn, docs@python, ericvw priority: normal severity: normal status: open title: Clean up known issues for AIX type: enhancement versions: Python 2.7, Python 3.3, Python 3.4, Python 3.5, Python 3.6, Python 3.7 Added file: http://bugs.python.org/file45712/cleanup-readme-aix.patch _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Changes by Zachary Ware <zachary.ware@gmail.com>: ---------- nosy: +Michael.Felt stage: -> patch review versions: -Python 3.3, Python 3.4 _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Julien Palard added the comment: You're also removing:
* Python has not been fully tested on AIX when compiled as a 64 bit application.
I do not have an AIX at hand but can someone at least confirm that all test passes? Having a few issues fixed does not mean that Python has been fully tested. ---------- nosy: +mdk _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Eric N. Vander Weele added the comment:
Having a few issues fixed does not mean that Python has been fully tested.
I uploaded cleanup-readme-aix2.patch, which revives back the line removed in question and only removes lines which reference issues. I just noticed the results of Python's tests on my AIX box differ from Python's PPC64 Buildbot. I'll investigate separately and submit patches for getting the tests to pass, which then it may be more appropriate to remove that line. ---------- Added file: http://bugs.python.org/file45713/cleanup-readme-aix2.patch _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Julien Palard added the comment: Look like issue1745108 has only be closed for being "outdated" (msg240919 ), it does not clearly mean that the bug does no longer exists. Have you been able to test the example in it? ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Eric N. Vander Weele added the comment: I have been able to test the example without a segmentation fault. $ python3.5 Python 3.5.2 (default, Nov 17 2016, 10:45:58) [C] on aix7 Type "help", "copyright", "credits" or "license" for more information.
import curses from curses import panel def mkpanel(scr): ... win = curses.newwin(8,8,1,1) ... pan = panel.new_panel(win) ... curses.wrapper(mkpanel)
---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Julien Palard added the comment: LGTM but no AIX to test it. ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
David Edelsohn added the comment: There is an AIX system in the Python buildbot farm. Why do you say no AIX to test? ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Michael Felt added the comment: FWIW: just build python-2.7.13 using xlC - and ncurses-6.0 installed. root@x064:[/data/prj/python/python2-2.7.13] root@x064:[/data/prj/python/python2-2.7.13]./python Python 2.7.13 (default, Jan 3 2017, 11:16:59) [C] on aix5 Type "help", "copyright", "credits" or "license" for more information.
import curses from curses import panel def mkpanel(scr): ... win = curses.newwin(8,8,1,1) ... pan = panel.new_panel(win) ... curses.wrapper(mkpanel)
So, even for AIX 5.3 and Python2.7 this test seems solved. If the above is all that is needed! ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Michael Felt added the comment: I request that you review issue27435 - in particular msg284557 - as I feel ctypes implementation for AIX is broken - at least as far as the supporting routines are concerned. When you know the "magic" one can call ctypes.CDLL() and it 'works'. ctypes.util.find_library() always returns None (maybe it works if a stack of GNU tools are also loaded, but not on a 'generic' AIX install) ctypes.LibraryLoader() is always successful (i.e., no Traceback) but I am (personally) unsure of the actual return value. It feels like a false positive. Thank you for your time and consideration! p.s. - a "fix/patch" is nearly accepted for Python3.7 (see issue26439) - needs more for the docs - however, I feel is is a great short-coming to limit this to versions that are yet to come! ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Eric N. Vander Weele added the comment:
I request that you review issue27435 - in particular msg284557 - as I feel ctypes implementation for AIX is broken - at least as far as the supporting routines are concerned.
I believe this request is outside the scope of this particular issue.
[...] AIX to test it.
It appears there is an AIX system in Python's buildbot farm to validate the curses example. Is the next step have a core maintainer conduct the validation on an AIX system before this can move forward? ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Changes by Eric N. Vander Weele <ericvw@gmail.com>: ---------- pull_requests: +1764 _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Zachary Ware added the comment: New changeset 7c2f82d158a19ec9ee6b7e259c243e1f06f0434f by Zachary Ware (Eric N. Vander Weele) in branch 'master': bpo-28845: Clean up known issues for AIX (GH-1670) https://github.com/python/cpython/commit/7c2f82d158a19ec9ee6b7e259c243e1f06f... ---------- nosy: +zach.ware _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
Zachary Ware added the comment: Thanks for the patch! I'm going to decline backporting this; let's just focus on the future for this. ---------- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: -Python 2.7, Python 3.5, Python 3.6 _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue28845> _______________________________________
participants (5)
-
David Edelsohn -
Eric N. Vander Weele
-
Julien Palard
-
Michael Felt
-
Zachary Ware