Skip to content

Fix 2025/jhshrvdp to include curses.h instead of ncurses.h; Ncurses#276

Merged
lcn2 merged 1 commit into
ioccc-src:masterfrom
SirWumpus:2025_jhshrvdp
Jun 9, 2026
Merged

Fix 2025/jhshrvdp to include curses.h instead of ncurses.h; Ncurses#276
lcn2 merged 1 commit into
ioccc-src:masterfrom
SirWumpus:2025_jhshrvdp

Conversation

@SirWumpus

Copy link
Copy Markdown
Contributor

supports curses.h as an alternative name and for backwards compatibility.

supports curses.h as an alternative name and for backwards
compatibility.

@lcn2 lcn2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sigh 😞

@lcn2 lcn2 merged commit f7ef1ea into ioccc-src:master Jun 9, 2026
1 check passed
@xexyl

xexyl commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Landon will correct me if I'm wrong but I could have sworn (from Landon himself) that it's important to match the library with the header file i.e. if you use -lncurses you need to use ncurses.h and if you use -lcurses you need to use curses.h.

We went through this issue before with NetBSD not having ncurses (think that was it) but it probably was never resolved fully.

But with this one if -lcurses works perhaps the Makefile should be updated: assuming that I am correct (iirc Landon said it was a subtle point)?

I have never seen a problem incidentally with this but I guess Landon might have or knows of such problems.

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.

3 participants