git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).