git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add warning about known issues to documentation of cvsimport

From
Ferry Huberts (Pelagic) <ferry.huberts@pelagic.nl>
Date
Mar 23, 2009, 20:33 UTC
Message-ID
<49C7F233.9050205@pelagic.nl>
In-Reply-To
<20090323195304.GC26678@macbook.lan>
Heiko Voigt wrote:
Show 61 quoted lines
> The described issues are compiled from the tests by Michael Haggerty and me.
> Because it is not apparent that these can be fixed anytime soon at least warn
> unwary users not to rely on the inbuilt cvsimport to much.
> 
> Signed-off-by: Heiko Voigt <hvoigt@hvoigt.net>
> ---
>  Documentation/git-cvsimport.txt |   34 ++++++++++++++++++++++++++++++++++
>  1 files changed, 34 insertions(+), 0 deletions(-)
> 
> diff --git a/Documentation/git-cvsimport.txt b/Documentation/git-cvsimport.txt
> index b7a8c10..3123725 100644
> --- a/Documentation/git-cvsimport.txt
> +++ b/Documentation/git-cvsimport.txt
> @@ -24,6 +24,9 @@ repository, or incrementally import into an existing one.
>  Splitting the CVS log into patch sets is done by 'cvsps'.
>  At least version 2.1 is required.
>  
> +*WARNING:* for certain situations the import leads to incorrect results.
> +Please see the section <<issues,ISSUES>> for further reference.
> +
>  You should *never* do any work of your own on the branches that are
>  created by 'git-cvsimport'.  By default initial import will create and populate a
>  "master" branch from the CVS repository's main branch which you're free
> @@ -164,6 +167,37 @@ If '-v' is specified, the script reports what it is doing.
>  Otherwise, success is indicated the Unix way, i.e. by simply exiting with
>  a zero exit status.
>  
> +[[issues]]
> +ISSUES
> +------
> +Problems related to timestamps:
> +
> + * If timestamps of commits in the cvs repository are not stable enough
> +   to be used for ordering commits
> + * If any files were ever "cvs import"ed more than once (e.g., import of
> +   more than one vendor release)
> + * If the timestamp order of different files cross the revision order
> +   within the commit matching time window
> +
> +Problems related to branches:
> +
> + * Branches on which no commits have been made are not imported
> + * All files from the branching point are added to a branch even if
> +   never added in cvs
> + * files added to the source branch *after* a daughter branch was
> +   created: If previously no commit was made on the daugther branch they
> +   will erroneously be added to the daughter branch in git
> +
> +Problems related to tags:
> +
> +* Multiple tags on the same revision are not imported
> +
> +If you suspect that any of these issues may apply to the repository you
> +want to import consider using these alternative tools which proved to be
> +more stable in practise:
> +
> +* cvs2git (part of cvs2svn), `http://cvs2svn.tigris.org`
> +* parsecvs, `http://cgit.freedesktop.org/~keithp/parsecvs`
>  
>  Author
>  ------

maybe you can also add remarks about autocrlf and safecrlf? both need to be off

Previous: Heiko VoigtNext: Heiko Voigt
Message 2 of 22 in “Add warning about known issues to documentation of cvsimport”
  1. Add warning about known issues to documentation of cvsimportHeiko Voigt, Mar 23, 2009
  2. Ferry Huberts (Pelagic)Mar 23, 2009
  3. Heiko VoigtMar 30, 2009
  4. Ferry Huberts (Pelagic)Mar 31, 2009
  5. Heiko VoigtMar 31, 2009
  6. cvsimport: Add a note about crlf options to the documentationHeiko Voigt, Mar 31, 2009
  7. Ferry Huberts (Pelagic)Mar 31, 2009
  8. Jeff KingMar 24, 2009
  9. Heiko VoigtMar 30, 2009
  10. Junio C HamanoMar 31, 2009
  11. Cleanup warning about known issues in cvsimport documentationHeiko Voigt, Mar 31, 2009
  12. Jeff KingMar 31, 2009
  13. Cleanup warning about known issues in cvsimport documentationHeiko Voigt, Apr 1, 2009
  14. Jeff KingMar 31, 2009
  15. Jeff KingMar 31, 2009
  16. Junio C HamanoMar 31, 2009
  17. Documentation: use "spurious .sp" XSLT if DOCBOOK_SUPPRESS_SP is setChris Johnsen, Apr 1, 2009
  18. Jeff KingApr 1, 2009
  19. Junio C HamanoApr 2, 2009
  20. Jeff KingApr 1, 2009
  21. Chris JohnsenApr 1, 2009
  22. Jeff KingApr 1, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.