Also, a few meta-issues with this patch. One is that the usual practice is to send patches inline rather than attaching them. Another is that this text: > The attached patch adds NO_PERL to the build system, in the same fashion > as NO_TCLTK. > > Effects: > - No perl-based scripts or code are installed: > $SCRIPT_PERL > git-instaweb > gitweb > git-cvsserver > git-svn > - git-add does NOT have interactive support. > - None of the tests for the disabled stuff gets run. > > This patch has been a optional part of the Gentoo build of Git for a > full year now, originally introduced with v1.5.4.4. should go with the patch as part of the commit message along with this: > Implement 95% of the NO_PERL functionality, to build Git without any Perl > support, because some Gentoo users want a Git without any Perl whatesoever > (Gentoo bug #214168). > > Remaining bits are doing configure.ac as well as git-remote usage in: > t5502-quickfetch.sh > t5512-ls-remote.sh > t5520-pull.sh > > Signed-off-by: Robin H. Johnson <robbat2@xxxxxxxxxx> > Updated-by: Bernd Lommerzheim <bernd@xxxxxxxxxxxxxxx> > Bugzilla-URL: http://bugs.gentoo.org/show_bug.cgi?id=214168 > Notes: Ported from 20080423-git-1.5.5.1-noperl.patch > Notes: Ported from 20080322-git-1.5.4.5-noperl.patch > Notes: Ported from 20080528-git-1.5.6.1-noperl.patch > Notes: Ported from 20080626-git-1.5.6.1-noperl.patch and 20081123-git-1.6.0.4-noperl-cvsserver.patch > Notes: Ported from 20090126-git-1.6.1.1-noperl.patch > Notes: t3701-add-interactive.sh block added 2009/04/02 as it was missed, broke > testsuite with USE=-perl. If you are using git to prepare your patch, then "git format-patch" should generate the right output. And finally, these two hunks: > diff --git a/t/t9400-git-cvsserver-server.sh b/t/t9400-git-cvsserver-server.sh > index 6a37f71..84a0d31 100755 > --- a/t/t9400-git-cvsserver-server.sh > +++ b/t/t9400-git-cvsserver-server.sh > @@ -17,6 +17,12 @@ then > [...] > diff --git a/t/t9500-gitweb-standalone-no-errors.sh b/t/t9500-gitweb-standalone-no-errors.sh > index 6ed10d0..d4580a4 100755 > --- a/t/t9500-gitweb-standalone-no-errors.sh > +++ b/t/t9500-gitweb-standalone-no-errors.sh did not apply for me, to the current 'master' or 'next'. Furthermore, the blobs they are based on aren't even present in my repo, so a 3-way merge was impossible. What did you base this patch on? -Peff -- 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
