{"thread":{"id":"3119","subject":"[PATCH] AsciiDoc fixes for the git-svnimport manpage","startedAt":"2006-01-21T19:06:19Z","lastAt":"2006-01-21T19:31:09Z","messageCount":3,"participants":["Florian Weimer","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"14987","messageId":"E1F0O43-0002pn-7O@mid.deneb.enyo.de","threadId":"3119","inReplyTo":null,"subject":"[PATCH] AsciiDoc fixes for the git-svnimport manpage","fromName":"Florian Weimer","fromEmail":"fw@deneb.enyo.de","sentAt":"2006-01-21T19:06:19Z","receivedAt":"2006-01-21T19:06:19Z","isPatch":true,"sender":{"key":"fw@deneb.enyo.de","avatar":null},"body":"Change \"SVN:: Perl\" to \"SVN::Perl\", wrap a long line, and clean up the\ndescription of positional arguments.\n\nSigned-off-by: Florian Weimer \n\n---\n\n Documentation/git-svnimport.txt | 12 ++++--------\n 1 files changed, 4 insertions(+), 8 deletions(-)\n\n592699f759a61aee33060d91cfe13ff432c147cf\ndiff --git a/Documentation/git-svnimport.txt b/Documentation/git-svnimport.txt\nindex db1ce38..63e28b8 100644\n--- a/Documentation/git-svnimport.txt\n+++ b/Documentation/git-svnimport.txt\n@@ -21,7 +21,7 @@ DESCRIPTION\n Imports a SVN repository into git. It will either create a new\n repository, or incrementally import into an existing one.\n \n-SVN access is done by the SVN:: Perl module.\n+SVN access is done by the SVN::Perl module.\n \n git-svnimport assumes that SVN repositories are organized into one\n \"trunk\" directory where the main development happens, \"branch/FOO\"\n@@ -74,8 +74,8 @@ When importing incrementally, you might \n -l ::\n \tSpecify a maximum revision number to pull.\n \n-\tFormerly, this option controlled how many revisions to pull, due to\n-\tSVN memory leaks. (These have been worked around.)\n+\tFormerly, this option controlled how many revisions to pull,\n+\tdue to SVN memory leaks. (These have been worked around.)\n \n -v::\n \tVerbosity: let 'svnimport' report what it is doing.\n@@ -100,11 +100,7 @@ with a 40x error pretty quickly.\n If you're using the \"-d\" or \"-D\" option, this is the URL of the SVN\n repository itself; it usually ends in \"/svn\".\n \n-::\n-\tThe URL of the SVN module you want to import. For local\n-\trepositories, use \"file:///absolute/path\".\n-\n-\n+::\n \tThe path to the module you want to check out.\n \n -h::\n-- \n1.1.3\n"},{"id":"14991","messageId":"7vslrho038.fsf@assigned-by-dhcp.cox.net","threadId":"3119","inReplyTo":"E1F0O43-0002pn-7O@mid.deneb.enyo.de","subject":"Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-01-21T19:24:59Z","receivedAt":"2006-01-21T19:24:59Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Florian Weimer writes:\n\n> Change \"SVN:: Perl\" to \"SVN::Perl\", wrap a long line, and clean up the\n> description of positional arguments.\n\nThanks. Is this the part you mean by \"positional arguments\"?\n\n> -::\n> -\tThe URL of the SVN module you want to import. For local\n> -\trepositories, use \"file:///absolute/path\".\n> -\n> -\n> +::\n> \tThe path to the module you want to check out.\n\nThis looks to me as if we do not allow non file:/// URL like\nsvn:// anymore, or we never supported it but the documentation\npretended we did. Is that what you intended to fix?\n\nI am a bit confused...\n"},{"id":"14992","messageId":"8764od8jk2.fsf@mid.deneb.enyo.de","threadId":"3119","inReplyTo":"7vslrho038.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] AsciiDoc fixes for the git-svnimport manpage","fromName":"Florian Weimer","fromEmail":"fw@deneb.enyo.de","sentAt":"2006-01-21T19:31:09Z","receivedAt":"2006-01-21T19:31:09Z","isPatch":true,"sender":{"key":"fw@deneb.enyo.de","avatar":null},"body":"* Junio C. Hamano:\n\n> Florian Weimer writes:\n>\n>> Change \"SVN:: Perl\" to \"SVN::Perl\", wrap a long line, and clean up the\n>> description of positional arguments.\n>\n> Thanks. Is this the part you mean by \"positional arguments\"?\n\nUhm, \"positional arguments\" are non-option arguments.\n\n>> -::\n>> -\tThe URL of the SVN module you want to import. For local\n>> -\trepositories, use \"file:///absolute/path\".\n>> -\n>> -\n>> +::\n>> \tThe path to the module you want to check out.\n>\n> This looks to me as if we do not allow non file:/// URL like\n> svn:// anymore, or we never supported it but the documentation\n> pretended we did. Is that what you intended to fix?\n\nNo, there was a :: missing, and the SVN_repository_URL part was\nduplicated. The documentation tries to remind the reader that the\n\"/absolute/path\" syntax does not work and you have to use a real URL.\n\n(I haven't got git-svnimport to work so far, but the Subversion RA\nlayer is transport-agnostic, so if it works for file://, it will also\nwork for svn:// or http://).\n"}]}