git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andreas Ericsson <ae@op5.se>
To: Matthieu Moy <Matthieu.Moy@imag.fr>
Cc: Jakub Narebski <jnareb@gmail.com>,
	Dan Loewenherz <dloewenherz@gmail.com>,
	takeshin <admirau@gmail.com>,
	git@vger.kernel.org
Subject: Re: update @version in file
Date: Wed, 13 May 2009 19:38:39 +0200	[thread overview]
Message-ID: <4A0B059F.1020509@op5.se> (raw)
In-Reply-To: <vpqeiutrl09.fsf@bauges.imag.fr>

Matthieu Moy wrote:
> Jakub Narebski <jnareb@gmail.com> writes:
> 
>> 2. If you wanted however to have *file* version (like in CVS), then it
>>    is impossible because Git doesn't store such info
> 
> What could make sense is the commit id where the file was last
> modified (i.e. the first item that appears when one runs "git log" on
> the file). Not exactly as simple as it could be because of merge, but
> not completely meaningless either.
> 

That would also incur an impossible performance penalty. A *LOT* of
projects have a LICENSE file that hasn't been changed since the project
was put under version control in the first place. Is git suppose to
dig through history for every revision just to make sure that the
LICENSE file hasn't been changed since 1999?

Now think about such a feature for a project with 20k+ files (not even
very large, as projects go) and see how much time you lose when
switching branches. Then ask yourself: What's more important: Developer
time or version numbers in files that can be hacked in to match the
entire project's version when releasing?

100% of all discussions about this on git@vger has ended up with the
conclusion that it's not worth wasting a couple of seconds of a
developer's time for this feature to exist. That's why this feature
doesn't exist in git yet.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Register now for Nordic Meet on Nagios, June 3-4 in Stockholm
 http://nordicmeetonnagios.op5.org/

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.

  reply	other threads:[~2009-05-13 17:38 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-13  8:59 update @version in file takeshin
2009-05-13  9:11 ` Andreas Ericsson
2009-05-13  9:22   ` takeshin
2009-05-13  9:42   ` Jakub Narebski
2009-05-13 16:04     ` Dan Loewenherz
2009-05-13 16:56       ` Jakub Narebski
2009-05-13 17:05         ` Matthieu Moy
2009-05-13 17:38           ` Andreas Ericsson [this message]
2009-05-13 23:02           ` Jakub Narebski
2009-05-14  7:08             ` Matthieu Moy

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=4A0B059F.1020509@op5.se \
    --to=ae@op5.se \
    --cc=Matthieu.Moy@imag.fr \
    --cc=admirau@gmail.com \
    --cc=dloewenherz@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jnareb@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).