From: Eric Blake <ebb9@byu.net>
To: m4-patches@gnu.org, Jim Meyering <jim@meyering.net>, git@vger.kernel.org
Subject: cvsimport bug on branches [was: conversion to git]
Date: Thu, 20 Sep 2007 19:17:47 -0600 [thread overview]
Message-ID: <46F31BBB.1040901@byu.net> (raw)
In-Reply-To: <loom.20070920T010842-272@post.gmane.org>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
According to Eric Blake on 9/19/2007 5:13 PM:
> I'm working on converting M4 to git.
>
> The initial git conversion is complete. Feel free to check it out:
>
> http://git.savannah.gnu.org/gitweb/?p=m4.git
Unfortunately, after publishing this repository, I noticed that there is a
bug in git-cvsimport, when a vendor branch is created that contains fewer
files than the point it branched from. Those files were not removed from
the commit on the new branch, which means the git repository contains too
many files compared to what the CVS repository had for any checkout along
the branch.
For example, look at the m4-1.4.1 tag. In CVS,
http://cvs.savannah.gnu.org/viewvc/m4/?root=m4&hideattic=0&pathrev=m4-1_4_1
shows the following:
ABOUT-NLS (dead) No revisions exist on m4-1_4_1
and for the file ChangeLog, the predecessor of m4-1_4_1 (rev 1.1.1.1.2.1)
was m4-1_4 (rev 1.1.1.1):
http://cvs.savannah.gnu.org/viewvc/m4/ChangeLog?hideattic=0&view=log&root=m4&pathrev=m4-1_4_1
But after git-cvsimport (using git 1.5.3), tag m4-1.4.1 is shown on the
branch-1_4 branch, but with unrelated m4-1.4ppre2 as parent, instead of
the correct m4-1.4:
http://git.sv.gnu.org/gitweb/?p=m4.git;a=shortlog;h=ab2ce6be
and as a result, the tree includes ABOUT-NLS (which was correctly a part
of the m4-1.4ppre2 tag on the master branch):
http://git.sv.gnu.org/gitweb/?p=m4.git;a=tree;h=ab2ce6bec48;hb=ab2ce6bec4
Now that the repository is published, what's the best way to correct the
history for the git m4-1.4.1 tag, to correctly pick the m4-1.4 commit as
its parent, and to omit the files that were intentionally omitted from the
CVS branch it was imported from?
What needs to be done to git-cvsimport to make it recognize files that
were omitted from CVS branch creation points, to avoid this problem in the
first place?
- --
Don't work too hard, make some time for fun as well!
Eric Blake ebb9@byu.net
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFG8xu684KuGfSFAYARAjd8AJ0eOWw1TFGt4zpQKIDFTGRjxzI/cgCghBpM
W91wfRwc8Q2hoMO1WCWYtzM=
=zzNR
-----END PGP SIGNATURE-----
next parent reply other threads:[~2007-09-21 1:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <loom.20070920T010842-272@post.gmane.org>
2007-09-21 1:17 ` Eric Blake [this message]
2007-09-21 5:18 ` cvsimport bug on branches [was: conversion to git] Steffen Prohaska
2007-09-21 15:42 ` Linus Torvalds
2007-09-21 17:15 ` Robin Rosenberg
2007-09-21 20:22 ` Johannes Schindelin
2007-09-21 20:42 ` Robin Rosenberg
2007-09-21 21:12 ` Johannes Schindelin
2007-09-21 23:16 ` Robin Rosenberg
2007-09-22 8:07 ` fromcvs installation [was: cvsimport bug on branches [was: conversion to git]] Steffen Prohaska
2007-09-22 15:08 ` Simon 'corecode' Schubert
2007-09-22 0:05 ` cvsimport bug on branches [was: conversion to git] Martin Langhoff
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=46F31BBB.1040901@byu.net \
--to=ebb9@byu.net \
--cc=git@vger.kernel.org \
--cc=jim@meyering.net \
--cc=m4-patches@gnu.org \
/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).