picrin-scheme / picrin

lightweight scheme interpreter
MIT License
414 stars 35 forks source link

Missing source files? #351

Closed dcurrie closed 7 months ago

dcurrie commented 7 years ago

In a commit on May 12, 2017, "bugfix: dyn_env is not properly restored on delim-cont call", a change was made to error.c in the generated file lib/ext but not to the corresponding source file in piclib.

On a related note, it's necessary for me to change the piclib/*.scm source files to use write-string instead of display; otherwise some of the output goes to the console instead of the pipe into the output file (on macOS). Also, the files in piclib use quotes instead of \<> around #include file names.

The build is failing on Github.

When I fix the build locally by making the changes above, as well as

the resulting picrin application hangs on launch.

All this makes me wonder if there are missing file updates on Github.

omasanori commented 7 years ago

Thank you for reporting, @dcurrie!

I also found the build issues some day and your pic_raise fix looks good to me. However, I am unsure whether the pic_fmemopen fix is correct or not since the API had been public and changed to private IIRC.

@wasabiz, your thoughts?