On Sun, May 8, 2011 at 23:45, Sverre Rabbelier <srabbelier@xxxxxxxxx> wrote: > Heya, > > [+msysgit people] > > On Sun, May 8, 2011 at 23:38, Ãvar ArnfjÃrà Bjarmason <avarab@xxxxxxxxx> wrote: >> On Sun, May 8, 2011 at 19:03, Sverre Rabbelier <srabbelier@xxxxxxxxx> wrote: >>> On Sun, May 8, 2011 at 14:10, Ãvar ArnfjÃrà Bjarmason <avarab@xxxxxxxxx> wrote: >>>> This adds skeleton no-op functions to git-sh-i18n.sh analogous to the >>>> gettext.c skeleton functions for C, adds *.sh scripts to the "pot" >>>> target for message extraction, and updates the git-sh-i18n--envsubst >>>> tests to use the new test_i18ncmp function. >>> >>> I seem to remember there were some concerns about performance with a >>> previous version of this series. Have you done any before/after >>> timings on, say, the git test suite? Or am I remembering incorrectly? >> >> When I benchmark this on my (Linux) system it runs at 0% difference >> between the current test suite and the one with this series. >> >> The concern was about Windows, where forks are more expensive, so >> e.g. every time we shell out to sed/awk/perl/grep or >> git-sh-i18n--envsubst we incur a larger speed hit than on Unix >> systems. > > Perhaps one of the windows devs can benchmark this then? Ãvar, where > do you have a pullable version of this series? Your github fork has > quite a few branches. It's ab/i18n-sh-only in git://github.com/avar/git.git There was some discussion about this a while ago, not sure what came out of that, but there were some msysgit patches involved IIRC. -- 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