GNOME Bugzilla – Bug 586806
Fix iconv cflags/libs in configure.ac
Last modified: 2013-09-13 01:05:15 UTC
When compiling, there is on my system a warning that /include cannot be found. This was because ICONV_CFLAGS was always set to "-I$withval/include" - this was introduced in commit 3462659b0a40d099630be394094788746f470f44 I think. The solution is dead simple : Before setting ICONV_CFLAGS / ICONV_LIBS, check that $withval is not empty.
Created attachment 137281 [details] [review] Fixes iconv cflags/libs
Oh good, I wondered where that was coming from. But wouldn't it also work to just quote $withval in the directory test? Then you don't need the -n test. if test -d "$withval"; then
Created attachment 137309 [details] [review] Quote withval in ICONV check for proper results New patch added. It's the simpler way :) Seems as if I thought a bit too complex. Will add new patch to evolution-data-server
Committed to master. http://git.gnome.org/cgit/evolution/commit/?id=9d9226188eeeffea82c676c4355e6fb169dd0782
*** Bug 588267 has been marked as a duplicate of this bug. ***