Skip to content

Fix compilation error with recent ncurses versions#184

Open
drmpjz wants to merge 7 commits into
abishekvashok:masterfrom
drmpjz:master
Open

Fix compilation error with recent ncurses versions#184
drmpjz wants to merge 7 commits into
abishekvashok:masterfrom
drmpjz:master

Conversation

@drmpjz
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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