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

[sd4dfg2%hotmail.com@localhost]

>Number:         34018
>Category:       pkg
>Synopsis:       pkgsrc/mail/mutt mishandles ncurses option?
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Jul 17 14:20:00 +0000 2006
>Originator:     Rob Quinn
>Release:        pkgsrc current
>Organization:
>Environment:
>Description:
Setting PKG_OPTIONS.mutt+=-ncurses doesn't make much of a difference when 
building mutt.

crn-oss:/usr/pkgsrc/mail/mutt #grep mutt /etc/mk.conf
PKG_OPTIONS.mutt+=-ncurses
crn-oss:/usr/pkgsrc/mail/mutt #bmake show-buildlink3
ncurses
openssl
iconv
gettext
gcc3-c
    iconv

>How-To-Repeat:

>Fix:
  Should the ".endif" in options.mk be a few lines down?

crn-sen:/usr/pkgsrc/mail/mutt #cvs diff options.mk 
Index: options.mk
===================================================================
RCS file: /cvsroot/pkgsrc/mail/mutt/options.mk,v
retrieving revision 1.8
diff -c -r1.8 options.mk
*** options.mk  5 Dec 2005 23:55:11 -0000       1.8
--- options.mk  17 Jul 2006 14:14:40 -0000
***************
*** 15,23 ****
  .else
  .  if !empty(PKG_OPTIONS:Mncurses)
  USE_NCURSES=          yes
- .  endif
  .  include "../../devel/ncurses/buildlink3.mk"
  CONFIGURE_ARGS+=      --with-curses=${BUILDLINK_PREFIX.ncurses}
  .endif
  
  ###
--- 15,23 ----
  .else
  .  if !empty(PKG_OPTIONS:Mncurses)
  USE_NCURSES=          yes
  .  include "../../devel/ncurses/buildlink3.mk"
  CONFIGURE_ARGS+=      --with-curses=${BUILDLINK_PREFIX.ncurses}
+ .  endif
  .endif
  
  ###




crn-sen:/usr/pkgsrc/mail/mutt #bmake show-buildlink3
openssl
iconv
gettext
gcc3-c
    iconv