git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: parsecvs tool now creates git repositories

From
Linus Torvalds <torvalds@osdl.org>
Date
Apr 3, 2006, 04:38 UTC
Message-ID
<Pine.LNX.4.64.0604022122430.3781@g5.osdl.org>
In-Reply-To
<1144037456.2303.92.camel@neko.keithp.com>
On Sun, 2 Apr 2006, Keith Packard wrote:
Show 10 quoted lines
> On Sun, 2006-04-02 at 21:31 +0200, Jan-Benedict Glaw wrote:
> 
> > lex.l: In function ÿÿparse_dataÿÿ:
> > lex.l:90: error: ÿÿyytext_ptrÿÿ undeclared (first use in this function)
> > lex.l:90: error: (Each undeclared identifier is reported only once
> > lex.l:90: error: for each function it appears in.)
> > make: *** [lex.o] Error 1
> 
> I think this is a bug in your version of flex; I'm using standard lex
> conventions here. I don't know how to make it work for you.
I need something like this to make it work with flex/lex..
The "-l" tells flex to be more traditional.
The "clean" rule is obvious.

And the "yylineno" is a lot more traditional than yyget_lineno(), which doesn't work for me at all. I think that's some issue with flex' support for re-entrant parsers.

Whether it works after this, I dunno. But at least it compiles.
		Linus
---
diff --git a/Makefile b/Makefile
index 639353a..c7e04a5 100644
--- a/Makefile
+++ b/Makefile
@@ -4,6 +4,7 @@ GCC_WARNINGS3=-Wnested-externs -fno-stri
 GCC_WARNINGS=$(GCC_WARNINGS1) $(GCC_WARNINGS2) $(GCC_WARNINGS3)
 CFLAGS=-O0 -g $(GCC_WARNINGS)
 YFLAGS=-d
+LFLAGS=-l
 
 SRCS=gram.y lex.l cvs.h parsecvs.c cvsutil.c revlist.c atom.c revcvs.c git.c
 
@@ -20,4 +21,4 @@ lex.o: lex.c
 y.tab.h: gram.c
 
 clean:
-	rm -f $(OBJS) y.tab.h gram.c parsecvs
+	rm -f $(OBJS) y.tab.h gram.c parsecvs lex.c
diff --git a/lex.l b/lex.l
index 39cafb0..c7833a4 100644
--- a/lex.l
+++ b/lex.l
@@ -65,8 +65,7 @@ parse_data (int save);
 \t				;
 \n				;
 .				{ 
-				    fprintf (stderr, "%s: (%d) ignoring %c\n", 
-					     yyfilename, yyget_lineno (),
+				    fprintf (stderr, "%s: (%d) ignoring %c\n", yyfilename, yylineno,
 					     yytext[0]);
 				}
 %%
@@ -146,8 +145,7 @@ lex_date (cvs_number *n)
 	d = mktime (&tm);
 	if (d == 0) {
 	    int i;
-	    fprintf (stderr, "%s: (%d) unparsable date: ", yyfilename,
-		     yyget_lineno ());
+	    fprintf (stderr, "%s: (%d) unparsable date: ", yyfilename, yylineno);
 	    for (i = 0; i < n->c; i++) {
 		if (i) fprintf (stderr, ".");
 		fprintf (stderr, "%d", n->n[i]);
Previous: Keith PackardNext: Jan-Benedict Glaw
Message 5 of 22 in “parsecvs tool now creates git repositories”
  1. Keith PackardApr 2, 2006
  2. Jan-Benedict GlawApr 2, 2006
  3. Jan-Benedict GlawApr 2, 2006
  4. Keith PackardApr 3, 2006
  5. Linus TorvaldsApr 3, 2006
  6. Jan-Benedict GlawApr 3, 2006
  7. Erik MouwApr 3, 2006
  8. Keith PackardApr 3, 2006
  9. Keith PackardApr 3, 2006
  10. Erik MouwApr 3, 2006
  11. Jakub NarebskiApr 3, 2006
  12. Keith PackardApr 3, 2006
  13. Keith PackardApr 3, 2006
  14. Jeff KingApr 3, 2006
  15. Anand KumriaApr 4, 2006
  16. Martin LanghoffApr 3, 2006
  17. Keith PackardApr 4, 2006
  18. Martin LanghoffApr 4, 2006
  19. Keith PackardApr 4, 2006
  20. Martin LanghoffApr 4, 2006
  21. Keith PackardApr 4, 2006
  22. Junio C HamanoApr 4, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.