From zeising@daemonic.se  Tue May  1 10:55:19 2012
Return-Path: <zeising@daemonic.se>
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id BEB351065680
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 10:55:19 +0000 (UTC)
	(envelope-from zeising@daemonic.se)
Received: from mail.lysator.liu.se (mail.lysator.liu.se [IPv6:2001:6b0:17:f0a0::3])
	by mx1.freebsd.org (Postfix) with ESMTP id 1DC758FC23
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 10:55:19 +0000 (UTC)
Received: from mail.lysator.liu.se (localhost [127.0.0.1])
	by mail.lysator.liu.se (Postfix) with ESMTP id 6B48640002
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 12:55:18 +0200 (CEST)
Received: by mail.lysator.liu.se (Postfix, from userid 1004)
	id 5DAD240003; Tue,  1 May 2012 12:55:18 +0200 (CEST)
Received: from mx.daemonic.se (mx.daemonic.se [IPv6:2001:470:dca9:0:1::3])
	(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by mail.lysator.liu.se (Postfix) with ESMTPSA id 0839640002
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 12:55:17 +0200 (CEST)
Received: from mailscanner.daemonic.se (mailscanner.daemonic.se [IPv6:2001:470:dca9:0:1::6])
	by mx.daemonic.se (Postfix) with ESMTPS id 3VhffF5sqWz8gh0
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 12:55:17 +0200 (CEST)
Received: from mx.daemonic.se ([IPv6:2001:470:dca9:0:1::3]) (using TLS with cipher CAMELLIA256-SHA)
	by mailscanner.daemonic.se (mailscanner.daemonic.se [2001:470:dca9:0:1::6]) (amavisd-new, port 10025)
	with ESMTPS id pLcX4h_sjyZS for <FreeBSD-gnats-submit@freebsd.org>;
	Tue,  1 May 2012 12:55:15 +0200 (CEST)
Received: from mail.daemonic.se (mail.daemonic.se [10.1.0.4])
	by mx.daemonic.se (Postfix) with ESMTPS id 3VhffC5Fpmz8ggv
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 12:55:15 +0200 (CEST)
Received: from vincent.daemonic.se (login.daemonic.se [IPv6:2001:470:dca9:0:1::10])
	by mail.daemonic.se (Postfix) with ESMTPS id 3VhffC4y2Nz9CvQ
	for <FreeBSD-gnats-submit@freebsd.org>; Tue,  1 May 2012 12:55:15 +0200 (CEST)
Received: (from zeising@localhost)
	by vincent.daemonic.se (8.14.5/8.14.5/Submit) id q41AtFIf068535;
	Tue, 1 May 2012 12:55:15 +0200 (CEST)
	(envelope-from zeising)
Message-Id: <201205011055.q41AtFIf068535@vincent.daemonic.se>
Date: Tue, 1 May 2012 12:55:15 +0200 (CEST)
From: Niclas Zeising <zeising@daemonic.se>
Reply-To: Niclas Zeising <zeising@daemonic.se>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: [PATCH] make usr.bin/mkesdb and usr.bin/mkcsmapper build with clang using -Werror
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         167481
>Category:       bin
>Synopsis:       [PATCH] make mkesdb(1) and mkcsmapper(1) build with clang using -Werror
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    eadler
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue May 01 11:00:24 UTC 2012
>Closed-Date:    Sun Sep 02 16:58:40 UTC 2012
>Last-Modified:  Sat Sep 15 19:00:27 UTC 2012
>Originator:     Niclas Zeising
>Release:        FreeBSD 9.0-RELEASE amd64
>Organization:
>Environment:
System: FreeBSD vincent.daemonic.se 9.0-RELEASE FreeBSD 9.0-RELEASE #0 r232364: Fri Mar 2 01:14:23 CET 2012 root@vincent.daemonic.se:/usr/obj/usr/src/sys/VINCENT amd64


	
>Description:
	usr.bin/mkesdb and usr.bin/mkcsmapper does not build with clang when -Werror is used.
>How-To-Repeat:
	
>Fix:

	Attached patch makes them build with clang -Werror.

--- head.usr.bin.buildfix.diff begins here ---
Index: head/usr.bin/mkesdb/lex.l
===================================================================
--- head/usr.bin/mkesdb/lex.l	(revision 234868)
+++ head/usr.bin/mkesdb/lex.l	(working copy)
@@ -45,6 +45,7 @@
 
 int line_number = 1;
 %}
+%option noinput
 %option	nounput
 
 %x	COMMENT
@@ -59,7 +60,7 @@
 <COMMENT>[\n]	{ line_number++; }
 <COMMENT>.	{ }
 <COMMENT><<EOF>>	{
-		yyerror("unexpected file end (unterminate comment)\n");
+		yyerror("unexpected end of file (unterminated comment)\n");
 		exit(1);
 	}
 
Index: head/usr.bin/mkcsmapper/lex.l
===================================================================
--- head/usr.bin/mkcsmapper/lex.l	(revision 234868)
+++ head/usr.bin/mkcsmapper/lex.l	(working copy)
@@ -43,6 +43,7 @@
 
 int line_number = 1;
 %}
+%option noinput
 %option	nounput
 
 %x	COMMENT
@@ -57,7 +58,7 @@
 <COMMENT>[\n]	{ line_number++; }
 <COMMENT>.	{ }
 <COMMENT><<EOF>>	{
-		yyerror("unexpected file end (unterminate comment)\n");
+		yyerror("unexpected end of file (unterminated comment)\n");
 		exit(1);
 	}
 
--- head.usr.bin.buildfix.diff ends here ---


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->eadler 
Responsible-Changed-By: eadler 
Responsible-Changed-When: Sun May 6 14:43:41 UTC 2012 
Responsible-Changed-Why:  
I'll take it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=167481 
State-Changed-From-To: open->analyzed 
State-Changed-By: eadler 
State-Changed-When: Tue May 8 00:21:10 UTC 2012 
State-Changed-Why:  
awaiting approval 

http://www.freebsd.org/cgi/query-pr.cgi?pr=167481 
State-Changed-From-To: analyzed->open 
State-Changed-By: eadler 
State-Changed-When: Fri Jun 1 07:06:51 UTC 2012 
State-Changed-Why:  
same as bin/167480 

http://www.freebsd.org/cgi/query-pr.cgi?pr=167481 
State-Changed-From-To: open->analyzed 
State-Changed-By: eadler 
State-Changed-When: Tue Jul 3 03:25:58 UTC 2012 
State-Changed-Why:  
awaiting approval/review 

http://www.freebsd.org/cgi/query-pr.cgi?pr=167481 
State-Changed-From-To: analyzed->closed 
State-Changed-By: eadler 
State-Changed-When: Sun Sep 2 16:58:38 UTC 2012 
State-Changed-Why:  
167480 is already patched 

http://www.freebsd.org/cgi/query-pr.cgi?pr=167481 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: bin/167481: commit references a PR
Date: Tue, 11 Sep 2012 16:32:49 +0000 (UTC)

 Author: dim
 Date: Tue Sep 11 16:32:31 2012
 New Revision: 240365
 URL: http://svn.freebsd.org/changeset/base/240365
 
 Log:
   Ensure mkcsmapper and mkesdb compile with clang, if WITH_ICONV is
   defined.
   
   Submitted by:	zeising
   PR:		bin/167481
   MFC after:	3 days
   X-MFC-With:	r238197
 
 Modified:
   head/usr.bin/mkcsmapper/lex.l
   head/usr.bin/mkesdb/lex.l
 
 Modified: head/usr.bin/mkcsmapper/lex.l
 ==============================================================================
 --- head/usr.bin/mkcsmapper/lex.l	Tue Sep 11 16:30:26 2012	(r240364)
 +++ head/usr.bin/mkcsmapper/lex.l	Tue Sep 11 16:32:31 2012	(r240365)
 @@ -43,6 +43,7 @@
  
  int line_number = 1;
  %}
 +%option	noinput
  %option	nounput
  
  %x	COMMENT
 
 Modified: head/usr.bin/mkesdb/lex.l
 ==============================================================================
 --- head/usr.bin/mkesdb/lex.l	Tue Sep 11 16:30:26 2012	(r240364)
 +++ head/usr.bin/mkesdb/lex.l	Tue Sep 11 16:32:31 2012	(r240365)
 @@ -45,6 +45,7 @@
  
  int line_number = 1;
  %}
 +%option	noinput
  %option	nounput
  
  %x	COMMENT
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: bin/167481: commit references a PR
Date: Sat, 15 Sep 2012 18:53:21 +0000 (UTC)

 Author: dim
 Date: Sat Sep 15 18:53:00 2012
 New Revision: 240537
 URL: http://svn.freebsd.org/changeset/base/240537
 
 Log:
   MFC r238197:
   
     Fix spelling
   
     PR:		bin/167480
     Submitted by:	zeising
     Approved by:	cperciva
   
   MFC r240365:
   
     Ensure mkcsmapper and mkesdb compile with clang, if WITH_ICONV is
     defined.
   
     PR:		bin/167481
     Submitted by:	zeising
 
 Modified:
   stable/9/usr.bin/mkcsmapper/lex.l
   stable/9/usr.bin/mkesdb/lex.l
 Directory Properties:
   stable/9/usr.bin/mkcsmapper/   (props changed)
   stable/9/usr.bin/mkesdb/   (props changed)
 
 Modified: stable/9/usr.bin/mkcsmapper/lex.l
 ==============================================================================
 --- stable/9/usr.bin/mkcsmapper/lex.l	Sat Sep 15 18:34:12 2012	(r240536)
 +++ stable/9/usr.bin/mkcsmapper/lex.l	Sat Sep 15 18:53:00 2012	(r240537)
 @@ -43,6 +43,7 @@
  
  int line_number = 1;
  %}
 +%option	noinput
  %option	nounput
  
  %x	COMMENT
 @@ -57,7 +58,7 @@ int line_number = 1;
  <COMMENT>[\n]	{ line_number++; }
  <COMMENT>.	{ }
  <COMMENT><<EOF>>	{
 -		yyerror("unexpected file end (unterminate comment)\n");
 +		yyerror("unexpected file end (unterminated comment)\n");
  		exit(1);
  	}
  
 
 Modified: stable/9/usr.bin/mkesdb/lex.l
 ==============================================================================
 --- stable/9/usr.bin/mkesdb/lex.l	Sat Sep 15 18:34:12 2012	(r240536)
 +++ stable/9/usr.bin/mkesdb/lex.l	Sat Sep 15 18:53:00 2012	(r240537)
 @@ -45,6 +45,7 @@
  
  int line_number = 1;
  %}
 +%option	noinput
  %option	nounput
  
  %x	COMMENT
 @@ -59,7 +60,7 @@ int line_number = 1;
  <COMMENT>[\n]	{ line_number++; }
  <COMMENT>.	{ }
  <COMMENT><<EOF>>	{
 -		yyerror("unexpected file end (unterminate comment)\n");
 +		yyerror("unexpected file end (unterminated comment)\n");
  		exit(1);
  	}
  
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 
>Unformatted:
