Greg Troxel <gdt@xxxxxxxxxx> writes: > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> Greg Troxel <gdt@xxxxxxxxxx> writes: >> >>> I realize a README.foo file for N different systems could be clutter, >>> but having these checked in would provide the concise help that people >>> on any of those platforms need. >> >> Our Makefile documents knobs people on various platforms can tweak >> (PYTHON_PATH and OLD_ICONV are two examples of them), sets default >> values for them based on $(uname -S), then includes config.mak file >> the user can optionally create to override these platform defaults. >> This infrastructure is used across platforms, not just for NetBSD. > > If you have to choose a single PYTHON_PATH, the one you picked is right > (for now, and likely will be right for a long time). > > But as a general rule, I think configure tests are preferable to > OS-specific variables. I forgot to mention that we also ship configure (and keep track of configure.ac) so that optionally people can let autoconf machinery to create config.mak.autogen to be included at the same place as handcrafted config.mak in their build process. I do not offhand know if we do "for p in python python2.6 python2.7; do ..." kind of thing, though. > A large number of people on NetBSD use git, but almost all of them get > it via pkgsrc (which is also where they get perl, emacs, svn, and > everything else you didn't find in /usr/bin). The exception would be > people that want to hack on git itself. Yeah, that much I figured ;-) > People who want gnu libiconv can install the libiconv package from > pkgsrc. (I'm guessing OLD_ICONV means "POSIX iconv", without GNU > extensions (iconvctl?).) It refers to the type of the second parameter to iconv(); OLD_ICONV makes it take "const char *", as opposed to "char *", the latter of which matches http://pubs.opengroup.org/onlinepubs/9699919799/functions/iconv.html > So, it doesn't matter too much for pkgsrc what you change, because it > can be patched anyway (once, for all users). Yes. The values in the Makefile are fallback defaults, and matters only to people who build from the source. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html