Junio C Hamano <gitster@xxxxxxxxx> wrote: > Eric Wong <e@xxxxxxxxx> writes: > > Junio C Hamano <gitster@xxxxxxxxx> wrote: > >> Duy Nguyen <pclouds@xxxxxxxxx> writes: > >> > On Mon, Jul 25, 2016 at 6:56 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > >> >> On Mon, Jul 25, 2016 at 9:21 AM, Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> wrote: > > about the problem sooner. > > > >> >>> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> > >> >>> --- > >> >>> Tested with fbsd 10.3, kvm image. But I suppose it's the same as real > >> >>> fbsd. > >> >> > >> >> Thanks; and we know that older (but not too old that we no longer care about) > >> >> FreeBSD all have /usr/local/bin/perl? > >> > > >> > I'm no fbsd expert but from the first sentence in [1] "Perl has been > >> > removed from base more than ten years ago..." I would assume it meant > >> > "... removed from base, _to_ ports system" which means /usr/local for > >> > all package installation (for ten years for perl). So I think we are > >> > good. > >> > >> I guess we didn't follow through > >> > >> http://public-inbox.org/git/%3C20160720025630.GA71874%40plume%3E/ > >> > >> and allowed the thread to drift into a tangent? > > > > +Cc Dscho > > > > I've been meaning to followup on that, but had connectivity > > problems to my VM last week. I still prefer we use numeric > > comparisons for version numbers since numbers are... numeric. > > IOW, I prefer we go with my original patch. > > I tend to agree with you if we have to do "systems older than this > should use /usr/bin, others should use /usr/local/bin", but this > different incarnation of the same topic seems to claim that older > ones had /usr/local/bin forever anyway, and that was what made the > patch interesting. I dug around the freebsd history (git://github.com/freebsd/freebsd.git) (~1.5G) and the 3.x and 4.x releases with perl in base still contained many references to /usr/local/bin/perl It also seems FreeBSD 2.x releases were also perl-less in base; so yes, I'm alright with Duy's patch :) -- 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