Summary: | audio/ncmpcpp: ./configure fails after devel/ncurses 6.0 update | ||
---|---|---|---|
Product: | Ports & Packages | Reporter: | yamagi |
Component: | Individual Port(s) | Assignee: | Baptiste Daroussin <bapt> |
Status: | Closed FIXED | ||
Severity: | Affects Only Me | CC: | bapt, cmt, yamagi |
Priority: | --- | Keywords: | patch, regression |
Version: | Latest | Flags: | koobs:
maintainer-feedback?
(yamagi) |
Hardware: | Any | ||
OS: | Any |
Description
yamagi
2015-12-09 10:17:01 UTC
The bug is in the ncmcpp port which mixes headers from base for ncurses with headers from ports. have curses.h from ncurses 5.x (base and using other headers from ncurses from ports). Thank you. :) Now I'll need to think of a way how to fix that in ncmpcpp. Since I re-did the issue summary, how about we leave this Open until you do? :) Okay, I've reopened it. Actually the bug is more tricky than that. ncurses from ports never installed curses.h so that port always used the one from base with other headers from ports in case the port was installed. It has worked by chance for a while. I suspect other ports might have the same issue. I will modify ncurses to install curses.h to avoid such mistake. That should fix ncmcpp A commit references this bug: Author: bapt Date: Wed Dec 9 11:57:16 UTC 2015 New revision: 403376 URL: https://svnweb.freebsd.org/changeset/ports/403376 Log: Install curses.h the ncurses ports was specially configured for a while to not install the curses.h header, expecting ports to use ncurses.h. How ever lots of ports are still using curses.h instead of ncurses.h. Those ended up using curses.h from base and other headers from the port. It worked by chances until now. Modify this port to make sure curses.h is installed so the ports will use the proper set of headers instead of mixing potentially incompatible ones. While here modify ncurses*6-config to prevent it to expose -I arguments pointing to a non existing directory PR: 205155 Reported by: yamagi@yamagi.org Changes: head/devel/ncurses/Makefile head/devel/ncurses/pkg-plist I presume merge quarterly is not needed? nope I believe this also fixes bug #205041 And since using /usr/include/curses.h worked by chance I didn't realize that ncmpcpp was mixing up base and ports ncurses. My solution would have been hacking the configure script and change the only curses.h include in the code. Not nice but less invasive than fixing that broken, homegrown curses detection logic... Of course having a /usr/local/include/ncurses/curses.h is a much nicer solution and I guess that it'll help other peoples too. Thank you. :) I guess we're done here. I'll submit an updates patch to bug #205071 (ncmpcpp update) soon. |