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: put tic binary in bin #28226

Merged
merged 1 commit into from
Aug 30, 2017
Merged

ncurses: put tic binary in bin #28226

merged 1 commit into from
Aug 30, 2017

Conversation

casey
Copy link
Contributor

@casey casey commented Aug 13, 2017

The tic binary wasn't being exported from the ncurses package. tic is the terminfo compiler, needed if one wants to compile new entries for the terminfo datbase.

  • Tested using sandboxing
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux

Tested by compiling a new terminfo entry and confirming that it worked.

@mention-bot
Copy link

@casey, thanks for your PR! By analyzing the history of the files in this pull request, we identified @wkennington, @vcunat and @edolstra to be potential reviewers.

@casey
Copy link
Contributor Author

casey commented Aug 23, 2017

I looked at the travis output, and it looks like it's just stalled because it's triggering a huge rebuild.

@Mic92
Copy link
Member

Mic92 commented Aug 27, 2017

Can you change the target branch from master to staging?

@casey casey changed the base branch from master to staging August 27, 2017 21:57
@casey
Copy link
Contributor Author

casey commented Aug 27, 2017

Done, rebased and changed base branch to staging.

@casey
Copy link
Contributor Author

casey commented Aug 27, 2017

Also, for future PRs, when should I target master and when staging?

@globin
Copy link
Member

globin commented Aug 27, 2017

@casey staging is for builds that trigger a lot of rebuilds.

@Ericson2314 Ericson2314 removed their request for review August 28, 2017 19:17
@fpletz fpletz added this to the 17.09 milestone Aug 30, 2017
@Mic92 Mic92 merged commit 823fb27 into NixOS:staging Aug 30, 2017
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

6 participants