From: Jakub Narebski <jnareb@gmail.com>
To: "H.Merijn Brand" <h.m.brand@xs4all.nl>
Cc: Andy Dougherty <doughera@lafayette.edu>, git@vger.kernel.org
Subject: Re: [PATCH] Explain what 'ginstall' is
Date: Tue, 18 Dec 2007 09:20:38 +0100 [thread overview]
Message-ID: <200712180920.39633.jnareb@gmail.com> (raw)
In-Reply-To: <20071218083843.42d7a15e@pc09.procura.nl>
On Tue, 18 Dec 2007, H.Merijn Brand wrote:
> On Mon, 17 Dec 2007 17:21:08 -0800 (PST), Jakub Narebski wrote:
>
>> Andy Dougherty <doughera@lafayette.edu> writes:
>>
>>> [...]. Perhaps configure
>>> should just go looking for a suitable install program instead of
>>> assuming everyone has one.
>>
>> First, configure is, and (I think) consensus is that it should remain
>> optional. This means that Makefile "guess" section should have good
>> defaults for your operating system.
>
> And what if you OS *does* have an install, but one that is completely
> useless^Wdifferent from the GNU install? Like the utterly useless
> install on HP-UX.
Then "guess" section should have INSTALL=ginstall, or something.
Or just install with
INSTALL=/opt/bin/ginstall make install
or something like that.
>> Second, the default autoconf macro AC_PROG_INSTALL *requires* that
>> there is BSD-compatible `install' program (as 'install-sh' or
>> 'install.sh') in the sources. Adding such script is (I think) not a
>> problem;
>
> Yeah!
>
>> finding minimal portable[*1*] script is. So if you know one...
>>
>> Footnotes:
>> ----------
>> [*1*] By "portable" I mean here 'git portable', i.e. requiring only
>> those shell constructs/features that git require, not necessary
>> insanelt portable the way ./configure script is.
>
> Something like this? (gui part still missing). This is what I now
> use
>
> --8<--- make-install
Errr... please read more carefully. There is need for BSD-compatibile
`install` program as 'install-sh', not 'make-install' script. The idea
is to use system-provided 'install' if it exists and is compatibile,
because it should be faster than script version, and fallback to
provided install-sh only if system install is not found.
install-sh has to understand '-d' and '-m <mode>' switches for git
install purposes, and probably implement all three (src dest, src dir,
dir) formats.
--
Jakub Narebski
Poland
next prev parent reply other threads:[~2007-12-18 8:21 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-17 21:46 [PATCH] Explain what 'ginstall' is Andy Dougherty
2007-12-18 1:21 ` Jakub Narebski
2007-12-18 7:38 ` H.Merijn Brand
2007-12-18 8:20 ` Jakub Narebski [this message]
2007-12-18 8:38 ` H.Merijn Brand
2007-12-18 9:14 ` Jakub Narebski
2007-12-18 11:11 ` H.Merijn Brand
2007-12-18 12:32 ` Jakub Narebski
2007-12-18 13:32 ` H.Merijn Brand
2007-12-18 14:03 ` Jakub Narebski
2007-12-18 14:27 ` H.Merijn Brand
2007-12-23 9:01 ` Jan Hudec
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=200712180920.39633.jnareb@gmail.com \
--to=jnareb@gmail.com \
--cc=doughera@lafayette.edu \
--cc=git@vger.kernel.org \
--cc=h.m.brand@xs4all.nl \
/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).