Re: pkg/34018: pkgsrc/mail/mutt mishandles ncurses option?

[Thomas Klausner <wiz%NetBSD.org@localhost>]

The following reply was made to PR pkg/34018; it has been noted by GNATS.

From: Thomas Klausner <wiz%NetBSD.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/34018: pkgsrc/mail/mutt mishandles ncurses option?
Date: Mon, 17 Jul 2006 19:29:27 +0200

 --ctP54qlpMx3WjD+/
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 On Mon, Jul 17, 2006 at 02:20:01PM +0000, sd4dfg2%hotmail.com@localhost wrote:
 > >Number:         34018
 > >Category:       pkg
 > >Synopsis:       pkgsrc/mail/mutt mishandles ncurses option?
 
 I think the attached patch is more in the spirit of the option.
 Does this fix the problem for you?
  Thomas
 
 --ctP54qlpMx3WjD+/
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename="mutt-options.diff"
 
 Index: options.mk
 ===================================================================
 RCS file: /cvsroot/pkgsrc/mail/mutt/options.mk,v
 retrieving revision 1.8
 diff -u -r1.8 options.mk
 --- options.mk 5 Dec 2005 23:55:11 -0000       1.8
 +++ options.mk 17 Jul 2006 17:28:39 -0000
 @@ -1,21 +1,26 @@
  # $NetBSD: options.mk,v 1.8 2005/12/05 23:55:11 rillig Exp $
  
 -PKG_OPTIONS_VAR=      PKG_OPTIONS.mutt
 -PKG_SUPPORTED_OPTIONS=        slang ncurses ssl buffy-size
 -PKG_SUGGESTED_OPTIONS=        ssl
 +PKG_OPTIONS_VAR=              PKG_OPTIONS.mutt
 +PKG_OPTIONS_OPTIONAL_GROUPS=  display
 +PKG_OPTIONS_GROUP.display=    slang ncurses
 +PKG_SUPPORTED_OPTIONS=                ssl buffy-size
 +PKG_SUGGESTED_OPTIONS=                ncurses ssl
  
  .include "../../mk/bsd.options.mk"
  
  ###
 -### Slang and ncurses
 +### Slang
  ###
  .if !empty(PKG_OPTIONS:Mslang)
  .  include "../../devel/libslang/buildlink3.mk"
  CONFIGURE_ARGS+=      --with-slang=${BUILDLINK_PREFIX.libslang}
 -.else
 -.  if !empty(PKG_OPTIONS:Mncurses)
 +.endif
 +
 +###
 +### ncurses
 +###
 +.if !empty(PKG_OPTIONS:Mncurses)
  USE_NCURSES=          yes
 -.  endif
  .  include "../../devel/ncurses/buildlink3.mk"
  CONFIGURE_ARGS+=      --with-curses=${BUILDLINK_PREFIX.ncurses}
  .endif
 
 --ctP54qlpMx3WjD+/--