From jkh@freebsd.org  Tue Sep 11 13:43:09 2001
Return-Path: <jkh@freebsd.org>
Received: from winston.freebsd.org (adsl-64-173-15-98.dsl.sntc01.pacbell.net [64.173.15.98])
	by hub.freebsd.org (Postfix) with ESMTP
	id 60A7B37B403; Tue, 11 Sep 2001 13:43:09 -0700 (PDT)
Received: from localhost (jkh@localhost [127.0.0.1])
	by winston.freebsd.org (8.11.6/8.11.6) with ESMTP id f8BKgfT57461;
	Tue, 11 Sep 2001 13:42:42 -0700 (PDT)
	(envelope-from jkh@freebsd.org)
Message-Id: <20010911134241X.jkh@freebsd.org>
Date: Tue, 11 Sep 2001 13:42:41 -0700
From: Jordan Hubbard <jkh@freebsd.org>
To: Alan.Judge@eircom.net
Cc: FreeBSD-gnats-submit@freebsd.org, re@freebsd.org
In-Reply-To: <200109111056.f8BAuCL48105@brazil.eng.eircom.net>
Subject: Re: STABLE sysinstall scripted installs broken since 2001/08/31
References: <200109111056.f8BAuCL48105@brazil.eng.eircom.net>

>Number:         30512
>Category:       bin
>Synopsis:       Re: STABLE sysinstall scripted installs broken since 2001/08/31
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    gnats-admin
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Sep 11 13:50:04 PDT 2001
>Closed-Date:    Sat Sep 15 09:09:12 PDT 2001
>Last-Modified:  Wed Oct 26 05:17:32 GMT 2005
>Originator:     
>Release:        
>Organization:
>Environment:
>Description:
 Whoops!  You're right, fixed, thanks!
 
 - Jordan
 
 From: Alan Judge <Alan.Judge@eircom.net>
 Subject: STABLE sysinstall scripted installs broken since 2001/08/31
 Date: Tue, 11 Sep 2001 11:56:12 +0100 (IST)
 
 > 
 > >Submitter-Id:	current-users
 > >Originator:	Alan Judge
 > >Organization:	eircom.net
 > >Confidential:	no
 > >Synopsis:	STABLE sysinstall scripted installs broken since 2001/08/31
 > >Severity:	critical
 > >Priority:	medium
 > >Category:	bin
 > >Class:		sw-bug
 > >Release:	FreeBSD 4.4-RC i386
 > >Environment:
 > System: FreeBSD brazil.eng.eircom.net 4.4-BRAZIL-0.62-BETA FreeBSD 4.4-BRAZIL-0.62-BETA #0: Thu Sep 6 13:35:07 GMT 2001 judgea@brazil.eng.eircom.net:/usr/src/sys/compile/BRAZIL-DEV-SMP i386
 > 
 > 	Tracking RELENG_4 and building custom releases for production
 > 	environment.
 > 
 > >Description:
 > 	A recent patch to release/sysinstall/msg.c by jkh has broken scripted
 > 	installs.  I think the comments in msg.c are wrong leading to the wrong
 > 	code being committed for non-interactive installs.
 > 
 > 	It seems msgYesNo follows the dialog DITEM_SUCCESS/FAILURE model and
 > 	returns 0 for success and 1 for failure.
 > 
 > >How-To-Repeat:
 > 	Attempt a scripted install using a config file.
 > 	installCommit fails quickly.
 > 
 > >Fix:
 > 
 > 	Patch below against STABLE.
 > 
 > 	I'm building a new test release now, but given the approaching
 > 	release thought it best to file rapidly so this can be
 > 	fixed before 4.4 ships.  I'll followup to the PR once I've
 > 	tested my new CD.
 > 
 > 	I'd request an immediate commit to STABLE if this looks ok.
 > 
 > Index: release/sysinstall/msg.c
 > ===================================================================
 > RCS file: /f5/misc/FreeBSD/cvs/src/release/sysinstall/Attic/msg.c,v
 > retrieving revision 1.52.2.3
 > diff -u -r1.52.2.3 msg.c
 > --- msg.c	2001/08/31 19:43:56	1.52.2.3
 > +++ msg.c	2001/09/11 10:33:55
 > @@ -218,7 +218,7 @@
 >      dialog_msgbox(NULL, errstr, -1, -1, 0);
 >  }
 >  
 > -/* Put up a message in a popup yes/no box and return 1 for YES, 0 for NO */
 > +/* Put up a message in a popup yes/no box and return 0 for YES, 1 for NO */
 >  int
 >  msgYesNo(char *fmt, ...)
 >  {
 > @@ -238,13 +238,13 @@
 >  	msgInfo(NULL);
 >      }
 >      if (variable_get(VAR_NONINTERACTIVE))
 > -	return 1;	/* If non-interactive, return YES all the time */
 > +	return 0;	/* If non-interactive, return YES all the time */
 >      ret = dialog_yesno("User Confirmation Requested", errstr, -1, -1);
 >      restorescr(w);
 >      return ret;
 >  }
 >  
 > -/* Put up a message in a popup no/yes box and return 1 for YES, 0 for NO */
 > +/* Put up a message in a popup no/yes box and return 0 for YES, 1 for NO */
 >  int
 >  msgNoYes(char *fmt, ...)
 >  {
 > @@ -264,7 +264,7 @@
 >  	msgInfo(NULL);
 >      }
 >      if (variable_get(VAR_NONINTERACTIVE))
 > -	return 0;	/* If non-interactive, return NO all the time */
 > +	return 1;	/* If non-interactive, return NO all the time */
 >      ret = dialog_noyes("User Confirmation Requested", errstr, -1, -1);
 >      restorescr(w);
 >      return ret;
>How-To-Repeat:
>Fix:
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: dd 
State-Changed-When: Sat Sep 15 09:09:12 PDT 2001 
State-Changed-Why:  
Not a new PR. 

http://www.FreeBSD.org/cgi/query-pr.cgi?pr=30512 
>Unformatted:
