onigposix.h missing
bovi opened this issue · 13 comments
Hi @mattn,
I'm getting at the moment the following error while building:
build/mrbgems/mruby-onig-regexp/src/mruby_orig_regexp.c:10:23: fatal error: onigposix.h: No such file or directory
Does it maybe make sense to integrate oniguruma directly into the GEM? What is your plan about that?
Regards
Daniel
I suppose that you have onigposix.h in /usr/include or /usr/local/include. But I will change mruby-onig-regexp to static-link gem.
onigposix.h is here: http://www.geocities.jp/kosako3/oniguruma/index.html
great. thanks!
Hi guys, is there a second way around this? I'm the author of webruby, and someone reported an issue saying that they cannot use this gem in webruby. I took a little look, the problem is that oniguruma is missing.
Just to provide a little background here: webruby uses emscripten to compile mruby, it uses its own libc and libc++ libraries instead of using the system's libs, which means that there is not likely to be an onigurama impl.
So I just want to ask here if there is another way around this problem. Is it possible to add an option here to let the users choose whether to use the one in /usr/include(and /usr/lib), or compile one on-the-fly?
Given how little oniguruma changes, wouldn't it make sense to simply bundle oniguruma in mruby-onig-regexp ? This would make building mruby-onig-regexp simpler, for example to build on mac oniguruma has to be installed, for example with brew.
thanks,
Ricardo
Btw, I made some mrbgem.rake that builds oniguruma so libonig.a would be linked when a executable needs to be built. Maybe it can be linked to shared library or DLL.
I want to use this in mruby-uv too so that libuv version problem can be solved with it.
Though it's little dirty so I would want a rubyist to refactor it.
@take-cheeze Thanks! It seems rubyinstaller for windows doesn't contains libarchive. Hmm.
@take-cheeze Is this ok? 1fbb9fb
Thanks for the Windows workaround too.
@take-cheeze Thanks for the fix! I haven't given it a test. But for my case, I think this is good enough to solve the problem.