From: Linus Torvalds <torvalds@osdl.org>
To: walt <wa1ter@myrealbox.com>, Sam Ravnborg <sam@ravnborg.org>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: Does git belong in root's $PATH?
Date: Sat, 7 Jan 2006 10:31:52 -0800 (PST) [thread overview]
Message-ID: <Pine.LNX.4.64.0601071023250.3169@g5.osdl.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0601070838470.6317@x2.ybpnyarg>
On Sat, 7 Jan 2006, walt wrote:
>
> When updated my kernel this morning, the same way I've been doing
> it for many months, I noticed that the -gxxxxxxx localversion
> string was missing from the new kernel's name.
>
> I finally figured out that this happened because /usr/local/bin
> is not in my root's $PATH, and the setlocalversion script depends
> on git. (The only thing I do as root is 'make install').
Ok, sounds like a build buglet to me. If you've done a "make" as a regular
user, and just do a "make install" as root, I'd argue that the "make
install" should do as little as humanly possible, and literally just
install the kernel. If it starts changing the version, that sounds a bit
fishy.
Sam, anything we can do?
That said:
> I suppose I'm asking a philosophical question here: do you
> guys install git where root can find it (as a system tool)?
I don't, but I don't use "make install" anyway, I just do "make
modules_install". I install the kernel by hand, I always have.
Of course, that's partly because I've always felt that "make install" does
too much (I think "make modules_install" is better - it really only
installs the already-built modules).
Maybe it would be best to remove the "vmlinux" dependency from "make
install" (so that "make install" will do exactly that: just install). I
think all the documentation already tells you to do a "make" and then a
"make install".
The other make targets really _are_ different: "make fdimage" depends on
vmlinux, but that's because it literally just builds the image. "make
install" is special.
Sam, what say you? I forget what the kbuild mailing list is, but maybe
you can forward this suggestion there..
Linus
next prev parent reply other threads:[~2006-01-07 18:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-07 17:21 Does git belong in root's $PATH? walt
2006-01-07 18:03 ` Andreas Ericsson
2006-01-07 20:40 ` H. Peter Anvin
2006-01-08 0:37 ` Andreas Ericsson
2006-01-08 0:39 ` H. Peter Anvin
2006-01-08 1:22 ` walt
2006-01-08 3:00 ` H. Peter Anvin
2006-01-07 18:31 ` Linus Torvalds [this message]
2006-01-07 20:49 ` H. Peter Anvin
2006-01-07 23:01 ` Sam Ravnborg
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=Pine.LNX.4.64.0601071023250.3169@g5.osdl.org \
--to=torvalds@osdl.org \
--cc=git@vger.kernel.org \
--cc=sam@ravnborg.org \
--cc=wa1ter@myrealbox.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).