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] NickThread Previous | Thread Next