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

make hangs at lang/m2/comp+next_c #280

Open
kenrector opened this issue Mar 10, 2023 · 1 comment
Open

make hangs at lang/m2/comp+next_c #280

kenrector opened this issue Mar 10, 2023 · 1 comment

Comments

@kenrector
Copy link

Without changing anything in the makefile, in a fresh repository downloaded today, on a Mac OS X running 10.15.7 make hangs when it gets to:

lang/m2/comp+next_c.

I did a make clean and tried again with same result.

here are the preceding n output:

main.a(l_states.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /tmp/ack-build/obj/lang/cem/cemcom.ansi/cemcom/main/cemcom/main.a(tokenfile.o) has no symbols
lang/cem/cemcom.ansi+cemcom
lang/cem/cemcom.ansi+pkg
lang/m2/comp+chartab_c
lang/m2/comp+tokenfile_g
lang/m2/comp+llgen
"/tmp/ack-build/obj/lang/m2/comp/tokenfile_g/tokenfile.g", line 49: (Warning) terminal PROGRAM not used
"/tmp/ack-build/obj/lang/m2/comp/tokenfile_g/tokenfile.g", line 50: (Warning) terminal COERCION not used
"/tmp/ack-build/obj/lang/m2/comp/tokenfile_g/tokenfile.g", line 51: (Warning) terminal CAST not used
lang/m2/comp+parameters_h
lang/m2/include+headers
lang/m2/comp+em_m2/main/char
lang/m2/comp+next_c

@langleyVa
Copy link

I worked around it by deleting line 69 (nice) in

"+next_c",
.

The whole build step seems to have been left over when in 04f723b the system of keeping markers in C files that are then fixed with sed in build, seems to have been ripped out of the modula part. The file arguments to the build step have been removed. That's why sed hangs, without file arguments it waits for stdin input.

Removing the line seems to have no adverse effects, it builds and tests fine. I don't have the time right now to get confident enough to prepare a PR with the fix, so I'll just document it here for David.

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