>Number:         10656
>Category:       pkg
>Synopsis:       mail/mutt does not detect color capability
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Jul 22 09:22:01 PDT 2000
>Closed-Date:
>Last-Modified:
>Originator:     Rhialto
>Release:        pkgsrc supped 22 july 2000 <NetBSD-current source date>
>Organization:
	
>Environment:
	
System: NetBSD azenomei.falu.nl 1.4.1 NetBSD 1.4.1 (AZENOMEI) #27: Wed Jun 14 01:41:57 CEST 2000 rhialto@azenomei.falu.nl:/usr/src/sys/arch/alpha/compile/AZENOMEI alpha


>Description:
	I upgrated mutt to the current version in pkgsrc (1.2.4).  The
	new version did not recognise the "color" commands in my
	.muttrc, one of which is
	
color header red default Subject:
	
	This was because the configure process did not detect
	that ncurses can do colour (and default colour). In fact it
	does, because manually setting HAVE_COLOR and
	HAVE_USE_DEFAULT_COLORS in the generated config.h does not break
	compilation, and my existing config file works again.

	configure.log reveals:

configure:2314: checking for start_color declaration
configure:2328: cc -c -O2  -I/usr/pkg/include conftest.c 1>&5
configure:2341: cc -c -O2  -I/usr/pkg/include conftest.c 1>&5
configure: In function `main':
configure:2336: `start_color' undeclared (first use in this function)
configure:2336: (Each undeclared identifier is reported only once
configure:2336: for each function it appears in.)
configure: failed program was:
#line 2332 "configure"
#include "confdefs.h"
#include (curses.h)
int main() {
#ifndef start_color
int     (*p)() = start_color;
#endif
; return 0; }

	It looks like that should be #include (ncurses.h) instead of
	#include (curses.h) in all occurrences in configure. Ideally,
	since configure first checks for the existance of (ncurses.h),
	this should depend on that.  This automagically also turns on
	HAVE_USE_DEFAULT_COLORS and some other related #defines.

	(incude syntax changed to prevend send-pr from stripping
	text between angle brackets)

	
>How-To-Repeat:
	
>Fix:
	
	I think previous versions of the pkg used libslang instead of
	ncurses. This could count as a workaround for the real problem
	(configure being broken).

-Olaf.
-- 
___ Olaf 'Rhialto' Seibert - rhialto@polder   -- Ah only did well at school
\X/ land.nl      -- tae git intae an O level class tae git away fae Begbie.
>Release-Note:
>Audit-Trail:
>Unformatted: