From db@nipsi.de  Wed May 28 06:25:27 2003
Return-Path: <db@nipsi.de>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 3467C37B401
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 28 May 2003 06:25:27 -0700 (PDT)
Received: from mail.nipsi.de (b074139.adsl.hansenet.de [62.109.74.139])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 17BBB43F75
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 28 May 2003 06:25:26 -0700 (PDT)
	(envelope-from db@nipsi.de)
Received: from localhost (localhost [127.0.0.1])
  (uid 1001)
  by mail.nipsi.de with local; Wed, 28 May 2003 15:25:24 +0200
Message-Id: <courier.3ED4B8C4.0000C99E@mail.nipsi.de>
Date: Wed, 28 May 2003 15:25:24 +0200
From: Dennis Berger <db@nipsi.de>
Reply-To: Dennis Berger <db@nipsi.de>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: new beson expects ";" fixup parser
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         52754
>Category:       ports
>Synopsis:       new beson expects ";" fixup parser
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kris
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed May 28 06:30:02 PDT 2003
>Closed-Date:    Sun Jul 06 18:48:21 PDT 2003
>Last-Modified:  Sun Jul 06 18:48:21 PDT 2003
>Originator:     Dennis Berger
>Release:        FreeBSD 4.8-RELEASE i386
>Organization:
>Environment:
System: FreeBSD nipsi.home.net 4.8-RELEASE FreeBSD 4.8-RELEASE #1: Tue Apr 8 14:38:06 CEST 2003 root@nipsi.home.net:/usr/src/sys/compile/NIPSI i386


>Description:
	new version of bison expects ";"
>How-To-Repeat:
	try to compile recent port version while having a new bison installed
>Fix:
	patch is attached

--- patch-bison begins here ---
--- keynote-orig.y	Wed Jun 14 08:28:23 2000
+++ keynote.y	Wed May 28 14:47:32 2003
@@ -90,17 +90,17 @@
              | PRIVATEKEY { keynote_exceptionflag = keynote_donteval = 0; }
                         STRING { keynote_lex_remove($3);
 			         keynote_privkey = $3;
-			       }
+			       };
     
 keypredicate: /* Nothing */   { keynote_returnvalue = 0;
                                 return 0; 
                               }
        | notemptykeypredicate { keynote_returnvalue = $1;
 				return 0;
-                              }
+                              };
 
 notemptykeypredicate:  key     { $$ = $1; }
-       		     | keyexp  { $$ = $1; }
+       		     | keyexp  { $$ = $1; };
 
 keyexp: notemptykeypredicate AND { if (($1 == 0) && !keynote_justrecord)
                                      keynote_donteval = 1;
@@ -137,7 +137,7 @@
 			      $$ = get_kth($1);
 			    else
 			      $$ = 0;
-			  }  /* K-th */
+			  };  /* K-th */
 
 keylist: key
 	    { /* Don't do anything if we're just recording */ 
@@ -154,7 +154,7 @@
 		  keynote_kth_array[$1]++;
 
 	      keylistcount++;
-            }
+            };
 
 key: str        {
 		   if (keynote_donteval)
@@ -192,10 +192,10 @@
 				 break;
 			 }
 		   }
-                 }
+                 };
 
 localinit: /* Nothing */
-         | localconstants
+         | localconstants;
 
 localconstants: VARIABLE EQQ STRING 
 	  {
@@ -264,12 +264,12 @@
 
 	    if (i != RESULT_TRUE)
 	      return -1;
-	  } localconstants
+	  } localconstants ;
 
 program: prog { 
 	        keynote_returnvalue = $1;
 		return 0;
-	      }
+	      };
 
 prog:   /* Nada */ { $$ = 0; }
        | notemptyprog {
@@ -284,7 +284,7 @@
 			$$ = $1;
 		      else
 			$$ = $4;
-                    } 
+                    };
 
 notemptyprog: expr HINT afterhint
               {
@@ -299,7 +299,7 @@
 		  $$ = keynote_current_session->ks_values_num - 1;
 		else
 		  $$ = 0;
-	      }
+	      };
 
 afterhint: str {  if (keynote_exceptionflag || keynote_donteval)
 		    $$ = 0;
@@ -314,7 +314,7 @@
 		      free($1);
 		  }
                 }
-         | OPENBLOCK prog CLOSEBLOCK { $$ = $2; }
+         | OPENBLOCK prog CLOSEBLOCK { $$ = $2; };
 
 
 expr:     OPENPAREN expr CLOSEPAREN 	{ $$ = $2; }
@@ -333,19 +333,19 @@
 	| floatexp			{ $$ = $1; }
 	| stringexp 			{ $$ = $1; }
         | TRUE	  		        { $$ = 1; }
-        | FALSE	  		        { $$ = 0; }
+        | FALSE	  		        { $$ = 0; };
 
 numexp:	  numex LT numex { $$ = $1 < $3; }
 	| numex GT numex { $$ = $1 > $3; }
 	| numex EQ numex { $$ = $1 == $3; }
 	| numex LE numex { $$ = $1 <= $3; }
 	| numex GE numex { $$ = $1 >= $3; }
-	| numex NE numex { $$ = $1 != $3; }
+	| numex NE numex { $$ = $1 != $3; };
 
 floatexp: floatex LT floatex { $$ = $1 < $3; }
 	| floatex GT floatex { $$ = $1 > $3; }
 	| floatex LE floatex { $$ = $1 <= $3; }
-	| floatex GE floatex { $$ = $1 >= $3; }
+	| floatex GE floatex { $$ = $1 >= $3; };
 
 numex:	  numex PLUS numex  { $$ = $1 + $3; }
 	| numex MINUS numex { $$ = $1 - $3; }
@@ -383,7 +383,7 @@
 						$$ = (int) floor(atof($2));
 					      free($2);
 					  }
-					}
+					};
 
 floatex:  floatex PLUS floatex  	{ $$ = ($1 + $3); }
 	| floatex MINUS floatex 	{ $$ = ($1 - $3); }
@@ -417,7 +417,7 @@
 						$$ = atof($2);
 					      free($2);
 					  }
-	                                }
+	                                };
 
 stringexp: str EQ str {
                         if (keynote_exceptionflag || keynote_donteval)
@@ -578,7 +578,7 @@
 		      free($1);
 		  }
 	      }
-	    }
+	    };
 
 str: str DOTT str    {  if (keynote_exceptionflag || keynote_donteval)
 			  $$ = (char *) NULL;
@@ -604,7 +604,7 @@
 			      return -1;
 			}
 		      }
-	| strnotconcat { $$ = $1; }
+	| strnotconcat { $$ = $1; };
 
 strnotconcat: STRING 	                { $$ = $1; }
         | OPENPAREN str CLOSEPAREN 	{ $$ = $2; }
@@ -659,7 +659,7 @@
 				if (keynote_lex_add($$, LEXTYPE_CHAR) == -1)
 				  return -1;
 			    }
-			 }
+			 };
 %%
 
 /*
--- keynote-ver-orig.y	Tue Oct 12 00:34:29 1999
+++ keynote-ver.y	Wed May 28 14:47:32 2003
@@ -48,7 +48,7 @@
        | STRING              { if (kn_add_authorizer(sessid, $1) != 0)
 				 return keynote_errno;
                                free($1);
-                             }
+                             };
 
 expr: VSTRING EQ STRING      { int i = kn_add_action(sessid, $1, $3, 0);
 
@@ -63,7 +63,7 @@
 				 return i;
 			       free($1);
 			       free($3);
-                             } expr 
+                             } expr ;
 %%
 void
 kverror(char *s)
--- patch-bison ends here ---


>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->feedback 
State-Changed-By: arved 
State-Changed-When: Fri May 30 04:57:16 PDT 2003 
State-Changed-Why:  
Can you please tell us, to which port this patch should be applied? 

http://www.freebsd.org/cgi/query-pr.cgi?pr=52754 
State-Changed-From-To: feedback->closed 
State-Changed-By: arved 
State-Changed-When: Fri Jul 4 08:15:15 PDT 2003 
State-Changed-Why:  
feedback timeout (1 month) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=52754 
State-Changed-From-To: closed->open 
State-Changed-By: arved 
State-Changed-When: Sat Jul 5 04:46:49 PDT 2003 
State-Changed-Why:  
Feedback arrived after i closed the PR. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=52754 
Responsible-Changed-From-To: freebsd-ports-bugs->kris 
Responsible-Changed-By: arved 
Responsible-Changed-When: Sun Jul 6 09:08:24 PDT 2003 
Responsible-Changed-Why:  
According to the submitter, this patch is for security/keynote, 
so over to kris, the Maintainer. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=52754 
State-Changed-From-To: open->closed 
State-Changed-By: kris 
State-Changed-When: Sun Jul 6 18:48:13 PDT 2003 
State-Changed-Why:  
Patch committed, thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=52754 
>Unformatted:
