From owner-freebsd-ports@freebsd.org Fri Jun 15 10:21:02 2018 Return-Path: Delivered-To: freebsd-ports@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 196B310142EF for ; Fri, 15 Jun 2018 10:21:02 +0000 (UTC) (envelope-from freebsd@grem.de) Received: from mail.grem.de (outcast.grem.de [213.239.217.27]) by mx1.freebsd.org (Postfix) with SMTP id 6F1B3856C4 for ; Fri, 15 Jun 2018 10:21:01 +0000 (UTC) (envelope-from freebsd@grem.de) Received: (qmail 6408 invoked by uid 89); 15 Jun 2018 10:21:00 -0000 Received: from unknown (HELO bsd64.grem.de) (mg@grem.de@194.97.158.70) by mail.grem.de with ESMTPA; 15 Jun 2018 10:21:00 -0000 Date: Fri, 15 Jun 2018 12:20:59 +0200 From: Michael Gmelin To: Mahmoud Al-Qudsi Cc: Michael Gmelin , Adam Weinberger , freebsd-ports@freebsd.org Subject: Re: Removing git dependencies on perl5 and python27 Message-ID: <20180615122059.7cacad99@bsd64.grem.de> In-Reply-To: References: <0DF706C2-DD9D-45DB-968F-8599D88D7622@grem.de> X-Mailer: Claws Mail 3.15.1 (GTK+ 2.24.31; amd64-portbld-freebsd10.3) X-Face: $wrgCtfdVw_H9WAY?S&9+/F"!41z'L$uo*WzT8miX?kZ~W~Lr5W7v?j0Sde\mwB&/ypo^}> +a'4xMc^^KroE~+v^&^#[B">soBo1y6(TW6#UZiC]o>C6`ej+i Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAJFBMVEWJBwe5BQDl LASZU0/LTEWEfHbyj0Txi32+sKrp1Mv944X8/fm1rS+cAAAACXBIWXMAAAsTAAAL EwEAmpwYAAAAB3RJTUUH3wESCxwC7OBhbgAAACFpVFh0Q29tbWVudAAAAAAAQ3Jl YXRlZCB3aXRoIFRoZSBHSU1QbbCXAAAAAghJREFUOMu11DFvEzEUAGCfEhBVFzuq AKkLd0O6VrIQsLXVSZXoWE5N1K3DobBBA9fQpRWc8OkWouaIjedWKiyREOKs+3PY fvalCNjgLVHeF7/3bMtBzV8C/VsQ8tecEgCcDgrzjekwKZ7TwsJZd/ywEKwwP+ZM 8P3drTsAwWn2mpWuDDuYiK1bFs6De0KUUFw0tWxm+D4AIhuuvZqtyWYeO7jQ4Aea 7jUqI+ixhQoHex4WshEvSXdood7stlv4oSuFOC4tqGcr0NjEqXgV4mMJO38nld4+ xKNxRDon7khyKVqY7YR4d+Cg0OMrkWXZOM7YDkEfKiilCn1qYv4mighZiynuHHOA Wq9QJq+BIES7lMFUtcikMnkDGHUoncA+uHgrP0ctIEqfwLHzeSo+eUA66AqzwN6n 2ZHJhw6Qh/PoyC/QENyEyC/AyNjq74Bs+3UH0xYwzDUC4B97HgLocg1QLYgDDO1v f3UX9Y307Ew4AHh67YAFFsxEpkXwpXY3eIgMhAAE3R19L919nNnuD2wlPcDE3UeT L2ytEICQib9BXgS2fU8PrD82ToYO1OEmMSnYTjSqSv9wdC0tPYC+rQRQD9ESnldF CyqfmiYW+tlALt8gH2xrMdC/youbjzPXEun+/ReXsMCDyve3dZc09fn2Oas8oXGc Jj6/fOeK5UmSMPmf/jL+GD8BEj0k/Fn6IO4AAAAASUVORK5CYII= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Jun 2018 10:21:02 -0000 Hm, seems like you're right, which is good news for me, I always built it without perl until this commit: r396048 | garga | 2015-09-04 00:00:03 +0200 (Fri, 04 Sep 2015) | 5 lines - Update to 2.5.1 - Fix build with NO_PERL set backporting a commit from git repo (82aec45) - Add git-submodile, git-am and git-request-pull to the list of commands that doesn't work without perl @@ -86,7 +86,7 @@ mv $@+ $@ else # NO_PERL -$(SCRIPT_PERL_GEN) git-instaweb: % : unimplemented.sh -+$(SCRIPT_PERL_GEN) git-instaweb git-submodule git-am git-request-pull: % : unimplemented.sh ++$(SCRIPT_PERL_GEN) git-instaweb git-submodule git-request-pull: % : unimplemented.sh Seems like it was removed again here: r416890 | garga | 2016-06-14 16:52:40 +0200 (Tue, 14 Jun 2016) | 4 lines Update devel/git to 2.9.0 @@ -88,7 +38,7 @@ mv $@+ $@ else # NO_PERL -$(SCRIPT_PERL_GEN) git-instaweb: % : unimplemented.sh -+$(SCRIPT_PERL_GEN) git-instaweb git-submodule git-request-pull: % : unimplemented.sh ++$(SCRIPT_PERL_GEN) git-instaweb git-request-pull: % : unimplemented.sh Thanks, Michael On Fri, 15 Jun 2018 03:10:37 -0500 Mahmoud Al-Qudsi wrote: > On Fri, Jun 15, 2018 at 2:57 AM, Michael Gmelin > wrote: > > Last time I checked, building git without Perl broke submodules > > (which is a core feature that should work with a default > > installation). > > I fully agree. Fortunately, (at least at a first glance) that does not > seem to be the case. > > I just built and installed git from source on a clean FreeBSD 12 > install: > > ``` > mqudsi@freebsd12:/easyre % mkdir test > mqudsi@freebsd12:/easyre % cd test > mqudsi@freebsd12:/easyre/test % git init . > Initialized empty Git repository in /easyre/test/.git/ > mqudsi@freebsd12:/easyre/test % git submodule add > https://github.com/neosmart/pkg-graph > Cloning into '/easyre/test/pkg-graph'... > remote: Counting objects: 41, done. > remote: Compressing objects: 100% (27/27), done. > remote: Total 41 (delta 16), reused 38 (delta 13), pack-reused 0 > Unpacking objects: 100% (41/41), done. > mqudsi@freebsd12:/easyre/test % perl > perl: Command not found. > mqudsi@freebsd12:/easyre/test % perl5 > perl5: Command not found. > mqudsi@freebsd12:/easyre/test % fd '^perl$' / > /easyre/git/perl > /easyre/git/t/t4034/perl > mqudsi@freebsd12:/easyre/test % git --version > git version 2.18.0.rc2.dirty > mqudsi@freebsd12:/easyre/test % > ``` > > Mahmoud Al-Qudsi > NeoSmart Technologies -- Michael Gmelin