Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-43172: readline now passes its tests when built against libedit #24499

Merged
merged 3 commits into from Feb 12, 2021

Conversation

@gpshead
Copy link
Member

@gpshead gpshead commented Feb 10, 2021

Existing irreconcilable API differences remain in readline.get_begidx and readline.get_endidx behavior based on libreadline vs libedit use.

https://bugs.python.org/issue43172

Existing irreconcilable API differences remain in readline.get_begidx
and eadline.get_endidx behavior based on libreadline vs libedit use.
@gpshead
Copy link
Member Author

@gpshead gpshead commented Feb 10, 2021

There isn't anyone who'd consider themselves an expert on this ancient module's internals as far as I can see...

mostly waiting on CI tests and any other pair of C focused eyeballs.

@gpshead gpshead closed this Feb 10, 2021
@gpshead gpshead reopened this Feb 10, 2021
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Feb 12, 2021

🤖 New build scheduled with the buildbot fleet by @gpshead for commit f31c22a 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@gpshead
Copy link
Member Author

@gpshead gpshead commented Feb 12, 2021

buildbot tests look good, the three red ones are unrelated.

@gpshead gpshead merged commit fd053fd into python:master Feb 12, 2021
66 of 69 checks passed
66 of 69 checks passed
Docs
Details
Check for source changes
Details
Check if generated files are up to date
Details
Windows (x86)
Details
Windows (x64)
Details
macOS
Details
Ubuntu
Details
buildbot/AMD64 Windows10 PR Build done.
Details
buildbot/AMD64 FreeBSD Shared PR Build done.
Details
buildbot/s390x Fedora Refleaks PR Build done.
Details
Azure Pipelines PR #20210212.7 succeeded
Details
Travis CI - Pull Request Build Passed
Details
bedevere/issue-number Issue number 43172 found
Details
bedevere/news News entry found in Misc/NEWS.d
buildbot/AMD64 Arch Linux Asan Debug PR Build done.
Details
buildbot/AMD64 Arch Linux Asan PR Build done.
Details
buildbot/AMD64 Arch Linux TraceRefs PR Build done.
Details
buildbot/AMD64 Debian PGO PR Build done.
Details
buildbot/AMD64 Debian root PR Build done.
Details
buildbot/AMD64 Fedora Stable Clang Installed PR Build done.
Details
buildbot/AMD64 Fedora Stable Clang PR Build done.
Details
buildbot/AMD64 Fedora Stable LTO + PGO PR Build done.
Details
buildbot/AMD64 Fedora Stable LTO PR Build done.
Details
buildbot/AMD64 Fedora Stable PR Build done.
Details
buildbot/AMD64 Fedora Stable Refleaks PR Build done.
Details
buildbot/AMD64 RHEL7 LTO + PGO PR Build done.
Details
buildbot/AMD64 RHEL7 LTO PR Build done.
Details
buildbot/AMD64 RHEL7 PR Build done.
Details
buildbot/AMD64 RHEL7 Refleaks PR Build done.
Details
buildbot/AMD64 RHEL8 LTO + PGO PR Build done.
Details
buildbot/AMD64 RHEL8 LTO PR Build done.
Details
buildbot/AMD64 RHEL8 PR Build done.
Details
buildbot/AMD64 RHEL8 Refleaks PR Build done.
Details
buildbot/AMD64 Ubuntu Shared PR Build done.
Details
buildbot/PPC64LE Fedora Stable Clang Installed PR Build done.
Details
buildbot/PPC64LE Fedora Stable Clang PR Build done.
Details
buildbot/PPC64LE Fedora Stable LTO + PGO PR Build done.
Details
buildbot/PPC64LE Fedora Stable LTO PR Build done.
Details
buildbot/PPC64LE Fedora Stable PR Build done.
Details
buildbot/PPC64LE Fedora Stable Refleaks PR Build done.
Details
buildbot/PPC64LE RHEL7 LTO + PGO PR Build done.
Details
buildbot/PPC64LE RHEL7 LTO PR Build done.
Details
buildbot/PPC64LE RHEL7 PR Build done.
Details
buildbot/PPC64LE RHEL7 Refleaks PR Build done.
Details
buildbot/PPC64LE RHEL8 LTO + PGO PR Build done.
Details
buildbot/PPC64LE RHEL8 LTO PR Build done.
Details
buildbot/PPC64LE RHEL8 PR Build done.
Details
buildbot/PPC64LE RHEL8 Refleaks PR Build done.
Details
buildbot/aarch64 Fedora Stable PR Build done.
Details
buildbot/aarch64 RHEL7 PR Build done.
Details
buildbot/aarch64 RHEL8 PR Build done.
Details
buildbot/s390x Debian PR Build done.
Details
buildbot/s390x Fedora Clang Installed PR Build done.
Details
buildbot/s390x Fedora Clang PR Build done.
Details
buildbot/s390x Fedora LTO + PGO PR Build done.
Details
buildbot/s390x Fedora LTO PR Build done.
Details
buildbot/s390x Fedora PR Build done.
Details
buildbot/s390x RHEL7 LTO + PGO PR Build done.
Details
buildbot/s390x RHEL7 LTO PR Build done.
Details
buildbot/s390x RHEL7 PR Build done.
Details
buildbot/s390x RHEL7 Refleaks PR Build done.
Details
buildbot/s390x RHEL8 LTO + PGO PR Build done.
Details
buildbot/s390x RHEL8 LTO PR Build done.
Details
buildbot/s390x RHEL8 PR Build done.
Details
buildbot/s390x RHEL8 Refleaks PR Build done.
Details
buildbot/s390x SLES PR Build done.
Details
buildbot/x86 Gentoo Installed with X PR Build done.
Details
buildbot/x86 Gentoo Non-Debug with X PR Build done.
Details
buildbot/x86-64 macOS PR Build done.
Details
@gpshead gpshead deleted the gpshead:test_readline_libedit branch Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants