Leopold Toetsch wrote:
Nick Glencross wrote:
Let me try reposting the patch, which gives me the opportunity to bit
twiddle a bit more:
* Removed the mmap nonsense which was sent by accident
* Renamed config.c to config_string.c to make it less generic
* Moved a couple externs from a core parrot library into the main
executable, where they sit better
Thanks for considering this patch,
I've now applied it locally, but there are 2 problems:
- small C99-ism (decl of interp in imcc/main.c:main()) and worse:
- segfault:
./miniparrot config_lib.pasm > runtime/parrot/include/config.fpmc
make: *** [runtime/parrot/include/config.fpmc] Error 139
Please check the patch, thanks.
It took me a moment to see what what you meant by the first point, but
yes, I've been a twit and put the Parrot_set_config_string line too
early in the function!
As for the second more serious issue, that's very odd! I assume that
you've tried building from scratch... (in particular no old
src/null_config.c) Also, what platform is this on?
Thanks.
[No point resubmitting the patch to fix the first point until we have an
explanation for the second, and after your nod of approval]
Nick
