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 compilation error with recent ncurses versions #184

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

drmpjz
Copy link

@drmpjz drmpjz commented Mar 7, 2024

LINES/COLS is no longer a variable in ncurses, but is implemented as functions returning the resp values. Therefore they can not be assigned values anymore.
Fix the code to use local variables instead.
Also fix problems with ncursesw (addwstr) and add logic for font location in openSUSE.

@drmpjz
Copy link
Author

drmpjz commented Jun 29, 2024

Can you please build against https://github.com/drmpjz/cmatrix once more? This should fix the issue during configure (there was an extra q at the end of a line, need to get a new keyboard, the escape key doesn't always work :-( )

Not sure this will fix the addwstr error for you. My fix addresses the ncurses/ncursesw case, while from the output above it looks like you don't have a ncurses at all. But maybe I am confused, you never know :-) Can you please try a man addwstr from a terminal? Does it return something at all? If so it should list the relevant include file in the synopsis. Can you post this here please?

@drmpjz
Copy link
Author

drmpjz commented Jun 29, 2024

The autoreconf issues are something you need to check on your end (you seem to be on a newer version of auto tools). Once we found a working link, please run autoupdate on your end to get rid of the warning (and I will check on my end if it still works with the older version of autotools.
I'll check the missing symbols later. Still have a long run on my todo list for today :-)

@drmpjz
Copy link
Author

drmpjz commented Jun 29, 2024

Can you please try once more? And if you don't mind telling, what distro (and version) are you building on?

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.

1 participant