On Sun, 15 Aug 2010 14:37:38 +0200 Enrico Tröger enrico.troeger@uvena.de wrote:
I started implementing the idea of indenting based on regular expressions and have noticed that geany can be compiled without regex support. Obviously in this case the indent thing wouldn't work. I don't quite understand when someone would want to compile geany without regex. By default in configure.ac you use the system regex library. If it doesn't exist, geany is compiled without regex support. Why not to automatically fallback to using the regex
If this is the case, it's a bug. At least IIRC the shipped regex support should be used as fallback. But I didn't use the autotools based build for ages. Anyway, the idea always was to build against the system's regex implementation if available and if not (like on Windows), fall back to the shipped mini implementation.
The Waf based build system builds with the shipped mini implementation if the --enable-gnu-regex option is given. Otherwise it checks whether the system provides something usable and uses this and if not, it uses the shipped variant.
If this doesn't work similar for the autotools, it's probably a bug.
I think configure.ac needs to be fixed to do this.
Then we can remove all #ifdef HAVE_REGCOMP lines (in fact we should probably remove them anyway).
Regards, Nick