Is NO_ICONV misnamed or is it broken?

Hello list,

With regards to the Makefile define/variable `NO_ICONV` - the Makefile comments imply that it should be used if "your libc doesn't properly support iconv," which could mean anything from "a patch will be applied" to "iconv won't be used."

Based off the name of the varibale, the assumption is that iconv is an optional dependency that can be omitted if compiled with NO_ICONV. However, in practice attempting to compile git with `make ... NO_ICONV=1` and libiconv not installed results in linker errors as follows:

```
~> make clean
# omitted
~> make NO_ICONV=1
# ommitted
    LINK git-credential-store
/usr/bin/ld: cannot find -liconv
cc: error: linker command failed with exit code 1 (use -v to see invocation)
gmake: *** [Makefile:2327: git-credential-store] Error 1
```

Am I misunderstanding the intended behavior when NO_ICONV is defined (i.e. it does not remove the dependency on libiconv) or is this a bug and iconv should not, in fact, be required?

Many thanks,

Mahmoud Al-Qudsi NeoSmart Technologies