From: Andreas Ericsson <ae@op5.se>
To: Junio C Hamano <gitster@pobox.com>
Cc: Steven Penny <svnpenn@gmail.com>, git@vger.kernel.org
Subject: Re: Build without BUILT_INS
Date: Tue, 24 Apr 2012 10:32:14 +0200 [thread overview]
Message-ID: <4F96650E.7040900@op5.se> (raw)
In-Reply-To: <xmqqmx61hmtu.fsf@junio.mtv.corp.google.com>
On 04/24/2012 04:42 AM, Junio C Hamano wrote:
> Steven Penny<svnpenn@gmail.com> writes:
>
>> I would like to build Git for Windows without the hard links in libexec/git-core
>>
>> I tried doing
>>
>> make install prefix=/c/git BUILT_INS=
>>
>> and I get this error
>
> Unsurprising.
>
> What are you trying to do? Are you trying to build a git that does not
> know any of the built-in commands like "checkout", "commit", etc? Or do
> you still want a full-featured git, but because you know the code to
> perform "git foo" for any builtin command "foo" is contained in git
> executable itself, you want to "rm -f git-foo" from the filesystem?
>
> The latter is a bad idea to begin with, as it will break older scripts
> that trust the promise that by having git --exec-path early on the $PATH
> older forms "git-foo" will always work.
>
> If you really want to do so, the right way is not by mucking BUILT_INS
> to empty. I do not know how GfW is built, but if it uses our usual
> Makefile, find the place where it installs the git-foo forms of links
> after it installs the main git executable, and comment it out.
>
> And no, it will *not* be a compilation option, as the resulting
> installation will break the age-old promise we made to script writers.
>
Otoh, we did warn that we were going to deprecate the 'git-foo' form back
in 2005 when the git wrapper was rewritten in C, so perhaps we should
consider marking it as deprecated now so we can get rid of them altogether
altogether in another seven years. ;)
--
Andreas Ericsson andreas.ericsson@op5.se
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231
Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.
next prev parent reply other threads:[~2012-04-24 8:32 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-24 2:19 Build without BUILT_INS Steven Penny
2012-04-24 2:42 ` Junio C Hamano
2012-04-24 8:32 ` Andreas Ericsson [this message]
2012-04-24 19:41 ` Junio C Hamano
2012-04-25 3:11 ` Steven Penny
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F96650E.7040900@op5.se \
--to=ae@op5.se \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=svnpenn@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).