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

Fix build fails on linux #345

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix build fails on linux #345

wants to merge 1 commit into from

Conversation

kiprasmel
Copy link

@kiprasmel kiprasmel commented Mar 25, 2019

Before this change, errors had been thrown that ncursesw/curses.h is
not found etc.

This change fixes it (I've now successfully build this on my arch linux
setup)

Originally from #297 (comment)

Signed-off-by: Kipras Melnikovas [email protected]

Before this change, errors had been thrown that `ncursesw/curses.h` is
not found etc.

This change fixes it (I've now successfully build this on my arch linux
setup)

Signed-off-by: Kipras Melnikovas <[email protected]>
@zvezdochiot
Copy link

zvezdochiot commented Mar 25, 2019

libncursesw5-dev: /usr/include/ncursesw/curses.h
    |
    V
libncurses5-dev: /usr/include/ncurses.h
   ???

@kiprasmel
Copy link
Author

libncursesw5-dev: /usr/include/ncursesw/curses.h
    |
    V
libncurses5-dev: /usr/include/ncurses.h
   ???

Hello, could you explain what does this mean? Should I manually symlink /usr/include/ncursesw/curses.h to /usr/include/ncurses.h? That's probably not what you meant:D

@zvezdochiot
Copy link

This means that you replaced the libncursesw5-dev package with libncurses5-dev. It's worth it?

@kiprasmel
Copy link
Author

This means that you replaced the libncursesw5-dev package with libncurses5-dev. It's worth it?

I originally came across this #297 (comment) when trying to debug the broken build process on arch linux.

The change I made here fixed it for me, but I don't know if it doesn't break for other systems because I cannot test it out, but I for sure know that it worked for me.

@zvezdochiot
Copy link

Let's wait what @dvorka will say.

@dvorka dvorka self-assigned this Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants