Skip to content

Comments

sysdig: fix pending upstream inclusion for ncurses-6.3#148626

Merged
7c6f434c merged 1 commit intoNixOS:masterfrom
trofi:fix-sysdig-for-ncurses-6.3
Dec 4, 2021
Merged

sysdig: fix pending upstream inclusion for ncurses-6.3#148626
7c6f434c merged 1 commit intoNixOS:masterfrom
trofi:fix-sysdig-for-ncurses-6.3

Conversation

@trofi
Copy link
Contributor

@trofi trofi commented Dec 4, 2021

Without the change the build fails as:

sysdig/userspace/sinspui/cursescomponents.cpp: In member function 'void curses_textbox::print_no_data()':
sysdig/userspace/sinspui/cursescomponents.cpp:878:15: error: format not a string literal and no format arguments [-Werror=format-security]
  878 |   wstr.c_str());
      |               ^

Without the change the build fails as:

    sysdig/userspace/sinspui/cursescomponents.cpp: In member function 'void curses_textbox::print_no_data()':
    sysdig/userspace/sinspui/cursescomponents.cpp:878:15: error: format not a string literal and no format arguments [-Werror=format-security]
      878 |   wstr.c_str());
          |               ^
@trofi trofi mentioned this pull request Dec 4, 2021
@ofborg ofborg bot requested a review from 7c6f434c December 4, 2021 19:47
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Dec 4, 2021
@7c6f434c 7c6f434c merged commit 0e3be5c into NixOS:master Dec 4, 2021
@trofi trofi deleted the fix-sysdig-for-ncurses-6.3 branch December 5, 2021 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants