mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Mansfield <lkml@dm.cobite.com>
To: Larry McVoy <lm@bitmover.com>
Cc: Andrea Arcangeli <andrea@suse.de>, <linux-kernel@vger.kernel.org>
Subject: Re: [ANNOUNCE] BK->CVS (real time mirror)
Date: Mon, 17 Mar 2003 20:48:16 -0500 (EST)	[thread overview]
Message-ID: <Pine.LNX.4.44.0303172037530.6286-100000@admin> (raw)
In-Reply-To: <20030317233332.GC529@work.bitmover.com>

On Mon, 17 Mar 2003, Larry McVoy wrote:

> On Tue, Mar 18, 2003 at 12:25:44AM +0100, Andrea Arcangeli wrote:
> > yes, this is very helpful thanks ;). I'd suggest you to also parse the
> > logic tag and to print a warning if there's an error and not only to
> > trust the timestamps. 
> 
> The time stamps we're talking about are *in* the revision history. 
> We do all checkins to all files with the same timestamp in the same
> changeset.  

Ok.  A version of 'cvsps' which can correctly parse Larry's log format
into patchsets is up on the website (www.cobite.com/cvsps).  It's version
2.0b3.

Larry's timestamps actually made this hack really easy.  All files 
committed with the exact time are recreated into a patchset.  Here, for 
example is 'patchset 8156':

--------------------
PatchSet 8156
Date: 2003/03/05 03:11:19
Author: torvalds
Branch: HEAD
Tag: v2_5_64
Log:
Linux 2.5.64

BKrev: 3e656ad75XghvjRCVNEGWy20cX0qwg

Members:
        ChangeSet:1.8156->1.8157
        Makefile:1.340->1.341


(note, this is a pretty boring patchset, I just wanted to show it 
basically works).

The patchset id is in-sync with Larry's ChangeSet commits (but off by one
from the beginning).

You can do:  'cvsps -s 8156 -g' to generate a diff of this entire
patchset, and even get the correct results.

> 
> If you thought that we were talking about on disk timestamps, that's 
> way too fragile but these are fine.
> 
> > certain logic tag out of the tree. This logic tag will be the
> > "changeset" number for us, but one that is also persistent and no only
> > unique 
> 
> (Logical tag 1.XXXX) 
> 

The checkin (Logical Tag x.yyy) log messages are currently not validated, 
and are discarded.  Only the 'main' message with the BKrev: is associated 
with each patchset.

> is in each file's checkin comments and the 1.XXXX is the ChangeSet file's
> rev for that changeset.

Seems to work.

> > I also wonder if it wouldn't be better if Larry would simply tag the CVS
> > with the logic tag number since the first place, rather than writing it

Not necessary, each changeset is available via cvsps with 'cvsps -s 
<logical change number>' as well as searching by file, by date, by tag, by 
log message etc.

> That means that *all* files get tags.  There would be 8300 x 15,000 files
> times sizeof(tag).  That's too big.
> 

Uggh.

David

-- 
/==============================\
| David Mansfield              |
| lkml@dm.cobite.com           |
\==============================/


  parent reply	other threads:[~2003-03-18  1:37 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-03-17 23:08 David Mansfield
2003-03-17 23:25 ` Andrea Arcangeli
2003-03-17 23:33   ` Larry McVoy
2003-03-17 23:57     ` Andrea Arcangeli
2003-03-18  1:48     ` David Mansfield [this message]
2003-03-18  2:43       ` Andrea Arcangeli
  -- strict thread matches above, loose matches on Subject: below --
2003-03-13 15:38 David Mansfield
2003-03-13 15:42 ` Larry McVoy
2003-03-12 17:42 Larry McVoy
2003-03-12 18:01 ` Roman Zippel
2003-03-12 18:34 ` Ben Collins
2003-03-12 19:03   ` Sam Ravnborg
2003-03-12 19:38     ` Roman Zippel
2003-03-12 19:32   ` Nicolas Pitre
2003-03-12 19:53     ` Ben Collins
2003-03-12 20:09       ` Ben Collins
2003-03-12 20:20       ` Jeff Garzik
2003-03-12 23:58         ` Roman Zippel
2003-03-12 20:37       ` Nicolas Pitre
2003-03-13  2:57   ` Aaron Lehmann
2003-03-16  3:48   ` Andrea Arcangeli
2003-03-16 17:45     ` Roman Zippel
2003-03-16 18:54       ` Nicolas Pitre
2003-03-16 19:30         ` Shawn
2003-03-16 19:33         ` Roman Zippel
2003-03-16 21:52           ` Andrea Arcangeli
2003-03-17  1:18             ` Roman Zippel
2003-03-17  1:35               ` Larry McVoy
2003-03-17  1:56                 ` Roman Zippel
2003-03-17  9:01                 ` Henning P. Schmiedehausen
2003-03-17 17:46                 ` Daniel Phillips
2003-03-17 18:04                   ` Jeff Garzik
2003-03-17 19:32                     ` Jamie Lokier
2003-03-17 19:40                       ` David Lang
2003-03-17 20:00                         ` Jamie Lokier
2003-03-17 20:43                       ` Andrea Arcangeli
2003-03-17 20:12                     ` Roman Zippel
2003-03-17 21:56             ` Pavel Machek
2003-03-17 22:08               ` Andrea Arcangeli
2003-03-21 14:16                 ` Larry McVoy
2003-03-21 17:42                   ` Andrea Arcangeli
2003-03-21 19:40                   ` H. Peter Anvin
2003-03-22  0:15                     ` Larry McVoy
2003-03-22  0:51                       ` H. Peter Anvin
2003-03-17 17:41           ` Horst von Brand
2003-03-17 18:04             ` Petr Baudis
2003-03-12 19:21 ` Nicolas Pitre
2003-03-12 19:51   ` Larry McVoy
2003-03-12 20:08     ` Ben Collins
2003-03-12 20:14     ` Sam Ravnborg
2003-03-12 20:18       ` Larry McVoy
2003-03-12 20:46       ` Nicolas Pitre
2003-03-12 20:58         ` Larry McVoy
2003-03-12 21:08           ` Nicolas Pitre
2003-03-13  0:41             ` Larry McVoy
2003-03-12 21:18           ` Eli Carter
2003-03-13 20:45           ` Horst von Brand
2003-03-13  1:58       ` Larry McVoy
2003-03-13 23:40       ` Larry McVoy
2003-03-12 21:05     ` Daniel Jacobowitz
2003-03-12 21:18       ` Larry McVoy
2003-03-12 21:31         ` Daniel Jacobowitz
2003-03-12 21:33           ` Larry McVoy
2003-03-12 21:45         ` Kai Germaschewski
2003-03-12 22:01           ` Larry McVoy
2003-03-12 22:21             ` David Lang
2003-03-12 22:30               ` Larry McVoy
2003-03-12 23:18                 ` Andreas Dilger
2003-03-15 16:52                   ` Larry McVoy
2003-03-13 21:00             ` Horst von Brand
2003-03-13  9:43     ` Geert Uytterhoeven
2003-03-13 23:26       ` Larry McVoy
2003-03-14  8:53         ` Geert Uytterhoeven
2003-03-12  3:43 Larry McVoy
2003-03-12  4:16 ` Ben Collins
2003-03-12  8:55   ` Jens Axboe
2003-03-12 10:26     ` Andreas Dilger
2003-03-12 10:31       ` Jens Axboe
2003-03-12 10:56         ` Andreas Dilger
2003-03-12 11:15           ` Jens Axboe
2003-03-12 11:20       ` Jamie Lokier
2003-03-12 16:13       ` H. Peter Anvin
2003-03-12 16:30         ` Dana Lacoste
2003-03-12 16:47           ` John Bradford
2003-03-12 17:08           ` Roman Zippel
2003-03-12 21:50             ` Alan Cox
2003-03-13 23:30               ` Roman Zippel
2003-03-12 17:29           ` H. Peter Anvin
2003-03-12 17:57             ` John Bradford
2003-03-12 18:03               ` Larry McVoy
2003-03-12 20:49                 ` H. Peter Anvin
2003-03-13  7:59                 ` Theodore Ts'o
2003-03-13  9:58                   ` Roman Zippel
2003-03-12 16:18       ` Ben Collins
2003-03-12 16:47         ` Lars Marowsky-Bree
2003-03-12 17:34           ` Ryan Anderson
2003-03-12 18:38   ` Arador
2003-03-12 18:47     ` Ben Collins
2003-03-12 19:12       ` Andreas Dilger
2003-03-13  0:29       ` Martin J. Bligh
2003-03-13  0:56         ` Larry McVoy
2003-03-16  3:44       ` Andrea Arcangeli
2003-03-12  4:39 ` H. Peter Anvin
2003-03-12  4:56   ` Larry McVoy
2003-03-16  3:10   ` Andrea Arcangeli
2003-03-12 19:34 ` Brandon Low
2003-03-16 13:45 ` Pavel Machek
2003-03-17 14:18   ` Wayne Scott
2003-03-17 14:45     ` Pavel Machek

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.44.0303172037530.6286-100000@admin \
    --to=lkml@dm.cobite.com \
    --cc=andrea@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm@bitmover.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

all inboxes | Powered by JetHome®