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

Upgrade ncurses to version 6.0 #22489

Closed
jdemeyer opened this issue Mar 2, 2017 · 11 comments
Closed

Upgrade ncurses to version 6.0 #22489

jdemeyer opened this issue Mar 2, 2017 · 11 comments

Comments

@jdemeyer
Copy link

jdemeyer commented Mar 2, 2017

Tarball: http://ftp.gnu.org/gnu/ncurses/ncurses-6.0.tar.gz

CC: @sagetrac-tmonteil

Component: packages: standard

Author: Jeroen Demeyer

Branch/Commit: a454855

Reviewer: Thierry Monteil

Issue created by migration from https://trac.sagemath.org/ticket/22489

@jdemeyer jdemeyer added this to the sage-7.6 milestone Mar 2, 2017
@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2017

@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2017

New commits:

a454855Upgrade ncurses to version 6.0

@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2017

Commit: a454855

@mkoeppe
Copy link
Member

mkoeppe commented Mar 2, 2017

comment:3

I checked that this does not cause problems with the autotools package, which depends on ncurses. Haven't tested anything else.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 3, 2017

comment:4

My VM is happy, removing custom patches is always a good newe, let us just wait that some patchbot gives a green light and it will be OK for me.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 3, 2017

Reviewer: Thierry Monteil

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 3, 2017

comment:5

Replying to @sagetrac-tmonteil:

My VM is happy, removing custom patches is always a good new, let us just wait that some patchbot gives a green light and it will be OK for me.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 4, 2017

comment:6

It seems that the patchbots get stuck after downloading the tarball. Any hint ?

@tscrim
Copy link
Collaborator

tscrim commented Mar 4, 2017

comment:7

The patchbot won't run package upgrades (for security reasons). Although I'd be shocked if this caused any (nontrivial) doctest failures.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 4, 2017

comment:8

Replying to @tscrim:

The patchbot won't run package upgrades (for security reasons). Although I'd be shocked if this caused any (nontrivial) doctest failures.

It did not on my 32-bit VM, which is a weird setting. While currently at sage days 84, i cannot afford to recompile everything it on my 6 years old laptop. So, let me positive_review it now then.

@vbraun
Copy link
Member

vbraun commented Mar 5, 2017

Changed branch from u/jdemeyer/upgrade_ncurses_to_version_6_0 to a454855

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants