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-45067 - Verify the version of ncurses for extended color support feature usage. #28260

Merged
merged 5 commits into from Sep 9, 2021

Conversation

orsenthil
Copy link
Member

@orsenthil orsenthil commented Sep 9, 2021

[bpo-45067](https://bugs.python.org/issue45067): Fix ncurses compilation in CentOS7. Verify ncurses version for extended color content support.

The function extended_color_content was introduced in 2017.
The ncurses-devel package in CentOS 7 had a older version ncurses resulted in
compilation error. For compiling ncurses with extended color support, we
verify the version of the ncurses library.

https://bugs.python.org/issue45067

…of ncurses

for extended color support feature usage.

The function extended_color_content was introduced in 2017.  The ncurses-devel
package in CentOS 7 had a older version ncurses resulted in compilation error.
For compiling ncurses with extended color support, we verify the version of the
ncurses library.
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

The comment at the #endif line does not match the current or new condition. Since I think that we do not want to repeat a large condition (which can out of sync again), would not be better to remove that comment?

Modules/_cursesmodule.c Outdated Show resolved Hide resolved
@orsenthil orsenthil merged commit 7944307 into python:main Sep 9, 2021
12 checks passed
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 9, 2021

Thanks @orsenthil for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒🤖

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 9, 2021

Thanks @orsenthil for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒🤖

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 9, 2021

Sorry, @orsenthil, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 794430700defb913512f871b701a888aa730de81 3.9

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Sep 9, 2021

GH-28263 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this issue Sep 9, 2021
…feature usage. (pythonGH-28260)

* issue45067 - Fix _curses compilation in CentOS 7. Verify the version of ncurses
for extended color support feature usage.

The function extended_color_content was introduced in 2017.  The ncurses-devel
package in CentOS 7 had a older version ncurses resulted in compilation error.
For compiling ncurses with extended color support, we verify the version of the
ncurses library.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
(cherry picked from commit 7944307)

Co-authored-by: Senthil Kumaran <senthil@python.org>
miss-islington added a commit that referenced this issue Sep 9, 2021
…feature usage. (GH-28260)

* issue45067 - Fix _curses compilation in CentOS 7. Verify the version of ncurses
for extended color support feature usage.

The function extended_color_content was introduced in 2017.  The ncurses-devel
package in CentOS 7 had a older version ncurses resulted in compilation error.
For compiling ncurses with extended color support, we verify the version of the
ncurses library.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
(cherry picked from commit 7944307)

Co-authored-by: Senthil Kumaran <senthil@python.org>
pablogsal pushed a commit that referenced this issue Sep 29, 2021
…feature usage. (GH-28260)

* issue45067 - Fix _curses compilation in CentOS 7. Verify the version of ncurses
for extended color support feature usage.

The function extended_color_content was introduced in 2017.  The ncurses-devel
package in CentOS 7 had a older version ncurses resulted in compilation error.
For compiling ncurses with extended color support, we verify the version of the
ncurses library.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
(cherry picked from commit 7944307)

Co-authored-by: Senthil Kumaran <senthil@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants