Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

caltech-parser/test_parselib build problems #927

Closed
jaykrell opened this issue Jan 24, 2022 · 2 comments
Closed

caltech-parser/test_parselib build problems #927

jaykrell opened this issue Jan 24, 2022 · 2 comments

Comments

@jaykrell
Copy link
Contributor

jaykrell commented Jan 24, 2022

I've been seeing like this a while.
At first I suspect a newline problem.
Now I am not sure.
It is true, maybe, we have a problem on Windows, running executables with forward slashes.
That is, CreateProcess, CreateFile, etc. all accept forward slashes, but cmd might not and system() uses cmd. I think.
We should probably futz with slashes when we run stuff.
Skip cm3 if there are no | or < or > or % or " and if first space delimited thing ends in .exe or .com, something like that..

== package C:\s\cm3\caltech-parser\parserlib\parserlib\test_parserlib ==

+++ c:\cm3\bin\cm3.exe -DM3_BACKEND_MODE=C -DBUILD_DIR=AMD64_NT -build -DROOT=C:/s/cm3 -DTARGET=AMD64_NT +++
--- building in AMD64_NT ---

ignoring ../src\m3overrides

C:/s/cm3/caltech-parser/parserlib/ktok/AMD64_NT/ktok ../src\Calc.t -o CalcTok.i3
C:/s/cm3/caltech-parser/parserlib/klex/AMD64_NT/klex ../src\Calc.l -o CalcLex.i3 -t ../src\Calc.t -ti3 CalcTok.i3
../src\Calc.l line 5: name and expression required
"c:\cm3\pkg\cit_util\src\generics.tmpl", line 37: quake runtime error: exit 1: C:/s/cm3/caltech-parser/parserlib/klex/AMD64_NT/klex ../src\Calc.l -o CalcLex.i3 -t ../src\Calc.t -ti3 CalcTok.i3

--procedure-- -line- -file---
exec --
_exec 37 c:\cm3\pkg\cit_util\src\generics.tmpl
_xCons 37 c:\cm3\pkg\parserlib\src\parser.tmpl
_lCons 82 c:\cm3\pkg\parserlib\src\parser.tmpl
_lConsUn 84 c:\cm3\pkg\parserlib\src\parser.tmpl
lexer 86 c:\cm3\pkg\parserlib\src\parser.tmpl
include_dir 5 C:\s\cm3\caltech-parser\parserlib\parserlib\test_parserlib\src\m3makefile
8 C:\s\cm3\caltech-parser\parserlib\parserlib\test_parserlib\AMD64_NT\m3make.args

Actually:
C:/s/cm3/caltech-parser/parserlib/klex/AMD64_NT/klex

does run, the error is from it.
So leave command lines alone, I think (we'll see about DOS/DJGPP, later).

@VictorMiasnikov
Copy link
Contributor

VictorMiasnikov commented Jan 24, 2022

I want write about refactoring "caltech-parser/parserlib" a long time . . .

At first we should move(?) testing outside of building.

P.S.

2)
I prefer
this style of folder structure:
C:/s/cm3/src/caltech-parser/

but we should use this:
C:/s/cm3/caltech-parser/

@jaykrell
Copy link
Contributor Author

jaykrell commented Feb 8, 2022

Duplicate of #970

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants