https://githubcom/geany/geany/blob/master/configureac#L35
this check is bogus and should be removed it breaks usage for example when CXX is set to ```arm-linux-musleabi -isystem /sysroot/foo```
``` $ which g++ -isystem foo /bin/which: unrecognized option: i BusyBox v1202 (2015-04-23 14:57:49 CEST) multi-call binary
Usage: which [COMMAND]
Locate a COMMAND
```
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829
Agreed it's broken, we should have a better fix for this -- I don't remember exactly, but IIRC it's something about `AC_PROG_CXX` setting `CXX` to `g++` if it didn't find no compiler at all, and us wanting to fail hard at configure time instead.
Anyway, as a workaround until it gets fixed, you can set `CXXFLAGS` to `-isystem /sysroot/foo` instead of having this in `CXX` itself.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-165582221
that workaround is not really acceptable for my distro as it would force me to create some script thats splits CXX into its components and overrides the userprovided CFLAGS. my workaround is ```sed -i 's@which $CXX@true@' configure```
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-165587205
@b4n, yes it is due to `AC_PROG_CXX` being broken. But autotools will not fix it now, too many projects will break. We should just try the `CXX` and fail if it doesn't work.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-165612960
Or we could change it to do a proper autoconf test like `AC_COMPILE_IFELSE` or one of those. It's a sure-fire way to tell if `$CXX` contains a working compiler and without any Bash-isms.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-165619783
@codebrainz yep, thats what I was thinking of.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-165629948
ping
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/829#issuecomment-231893969
Closed #829 via #1155.
github-comments@lists.geany.org