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

ncurses: 6.0 -> 6.0-20170729 #28334

Merged
merged 1 commit into from
Aug 22, 2017
Merged

ncurses: 6.0 -> 6.0-20170729 #28334

merged 1 commit into from
Aug 22, 2017

Conversation

layus
Copy link
Member

@layus layus commented Aug 17, 2017

Ncurses has no stable release since v6.0 which was released two years ago.
Given that even debian started using unstable weekly updates, I see no issue in doing the same.

This has been tested on my package set, but not the whole nixpkgs.
⚠️ this is a small mass-rebuild.

Fixes #19785.

Bump version to include a patch that fixes NixOS#19785.
@bjornfor
Copy link
Contributor

Does upstream want the world to use their unreleased version?

@layus
Copy link
Member Author

layus commented Aug 17, 2017

@bjornfor I would say that this is the versioning scheme of ncurses. Every distro is using these, from debian to fedora. I can however ask to the main developper, but I am not sure it is worth bothering him with that.

@fpletz fpletz merged commit e6661db into NixOS:staging Aug 22, 2017
@tsaeger
Copy link
Contributor

tsaeger commented Sep 7, 2017

I noticed after the upgrade to 6.0-20170729, tig used in a zsh session with TERM=xterm-256color has display problems. No crash, just that the graph is visually off.

Downgrading ncurses back 6.0 or upgrading to 20170902 fixed the issue I experienced.

I will submit a pr.

@layus
Copy link
Member Author

layus commented Sep 7, 2017

@tsaeger Oh crap, I guess that is what I get for using unstable releases. I assume that you fix was http://invisible-island.net/ncurses/NEWS.html#t20170827.

I hope they will release an official release in the near future, but I thank you for your PR in the meantime.

@infinisil infinisil mentioned this pull request Oct 13, 2017
8 tasks
@layus layus deleted the update-terminfo branch January 31, 2018 13:30
@c0bw3b c0bw3b mentioned this pull request Nov 14, 2018
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants