[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Commit Log File

From: <kfogel_at_collab.net>
Date: 2003-05-13 22:08:11 CEST

Oh wait -- I think I was wrong.

The precise problem with the $ThisRev$ keyword is that you want the
whole keyword to go away, and be replaced by the revision number --
permanently and forever. Which is hard to do, since we can't
anticipate the revision number.

-K

"Schrom.Brian" <Brian.Schrom@IGT.com> writes:
> Then on checkout, the keywords would get expanded through whatever the
> substitution policy is.
>
> That way, the file doesn't change with every commit because the keyword
> expands differently. Is what we really want to revision, the keywords, not
> the meta data defined by the keywords?
>
> This does require parsing the file on both check in and checkout. If this
> isn't already being done, that can be (will be :) ) a huge performance
> issue. Maybe a solution is to limit keywords to the 1st line of a file, or
> first N lines of a file.
>
> I would think that the keywords would want to be condensed on the client
> before the diff is applied. And for symmetry, this would mean that the
> keywords would be expanded after the diff was "patched" on update/checkout,
> so maybe the processing being done on the client isn't that much of a
> performance issue after all. If keywords were expanded at these phases,
> they basically wrap the repository operations, so should have minimal impact
> on the core. It becomes more of a client operation than a server operation?
>
> Thanks,
>
> Brian S.
>
>
> -----Original Message-----
> From: kfogel@collab.net [mailto:kfogel@collab.net]
> Sent: Tuesday, May 13, 2003 8:26 AM
> To: mark benedetto king
> Cc: Jonathan Smith; dev@subversion.tigris.org
> Subject: Re: Commit Log File
>
>
> mark benedetto king <mbk@boredom.org> writes:
> > This is covered in:
> >
> > http://subversion.tigris.org/issues/show_bug.cgi?id=958
>
> I have since realized that this would be *very* hard to implement.
>
> For repository specialists, here is the explanation:
>
> The problem is that the file in the txn needs to know its contents
> before the txn can be finalized. For normal keywords, this is no
> problem -- the keywords are stored contracted anyway. But for the
> proposed one-time-only revision keyword, we'd need to know the
> revision number of the commit *before* the commit is done. That
> effectively means locking up the revision number, then going back and
> modifying all the files in the txn that have this special keyword,
> then finishing the commit. Yuck.
>
> So I doubt issue #958 is going to be resolved anytime soon (I'll add
> these comments to the issue).
>
> Jonathan, I'd recommend that you not store your logfile statically
> anymore. Instead, have a script that can generate the file
> automatically, for example by running 'svn log -v' and massaging the
> output.
>
> -Karl
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 13 22:51:29 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.