{"thread":{"id":"19843","subject":"Improving git-svn documentation","startedAt":"2009-06-17T20:18:51Z","lastAt":"2009-06-25T08:47:27Z","messageCount":8,"participants":["Yann Dirson","Marc Branchaud","Junio C Hamano","Eric Wong"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"116497","messageId":"20090617201851.GA6123@nan92-1-81-57-214-146.fbx.proxad.net","threadId":"19843","inReplyTo":null,"subject":"Improving git-svn documentation","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2009-06-17T20:18:51Z","receivedAt":"2009-06-17T20:18:51Z","isPatch":false,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"While diving into the git-svn code, I realized that many things could\nbe done to make it more documented/understundable.  I need to get more\nunderstanding of it, so I'd like to improve this state of things.  But\nfirst, I'd like to be sure there is a consensus on what is a good idea\nto do, since that could easily turn up into a lot of textual change.\n\n- (on the user doc side of things) some options appear not to be\n  documented (I spotted --parent for 'clone' and --revision for\n  'dcommit').  But looking at where to document them, I found it not\n  always easy, since some options are documented together with the\n  command they modify, some others in the \"options\" section (even when\n  they are documented as applying to a single command, like --shared\n  or --stdin).  This IMHO leads to confusion for the user looking for\n  information, as well as to the reviewer trying to check that nothing\n  was forgotten.  I would rather make that only very commons are\n  described in a common \"options\" section, and that all commands using\n  them explicitely say so in their descriptions (with xref).\n\n- (on the code side of things) git-svn.perl weights more than 5500\n  lines, most classes functions and methods severely lack\n  documentation, and some extensively-used variable names are so short\n  they make the code harder to grasp\n\n  Eg. $gs to refer to an instance of the Git::SVN class, which I would\n  suggest to change to something like $gsrepo, while at the same time\n  renaming Git::SVN to eg. Git::SVN::Repository - which would make it\n  much easier for a newcomer to grasp what this is supposed to\n  represent - supposing, that is, that my understunding of this part\n  is accurate enough, which it is probably not after spending many\n  hours in there :)\n\n  As to the size of the file, it seems natural to me to split the\n  classes into their own files.  That would still let git-svn.perl and\n  the Git::SVN class to be 1500-lines tall, the largest others\n  achieving around 500 lines.  That should be much more manageable\n  pieces, and would require some refactoring wrt a couple of global\n  variables used throughout the script; which, incidentally, could\n  make it much easier to simultanously look at several git-svn\n  repositories (for my work on mapping externals to submodules), and\n  to allow reusing the existing code, eg. as a git-vcs backend.\n\nHow are you people feeling about this rough plans ?\n\n-- \nYann\n"},{"id":"116506","messageId":"4A396854.1090008@xiplink.com","threadId":"19843","inReplyTo":"20090617201851.GA6123@nan92-1-81-57-214-146.fbx.proxad.net","subject":"Re: Improving git-svn documentation","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2009-06-17T22:04:04Z","receivedAt":"2009-06-17T22:04:04Z","isPatch":false,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"I can't comment on the wisdom of your plan, but I do have a hankering to hack that git-svn.perl (to support multiple \"branches\" paths -- see http://marc.info/?l=git&m=124484360104600&w=2).\n\nAs I've quickly gotten lost in two attempts at figuring out how the script supports its --branches option, I applaud any effort to make this thing more approachable.  Thanks, and good luck!  (I'll need it!)\n\n\t\tM.\n\n\nYann Dirson wrote:\n> While diving into the git-svn code, I realized that many things could\n> be done to make it more documented/understundable.  I need to get more\n> understanding of it, so I'd like to improve this state of things.  But\n> first, I'd like to be sure there is a consensus on what is a good idea\n> to do, since that could easily turn up into a lot of textual change.\n> \n> - (on the user doc side of things) some options appear not to be\n>   documented (I spotted --parent for 'clone' and --revision for\n>   'dcommit').  But looking at where to document them, I found it not\n>   always easy, since some options are documented together with the\n>   command they modify, some others in the \"options\" section (even when\n>   they are documented as applying to a single command, like --shared\n>   or --stdin).  This IMHO leads to confusion for the user looking for\n>   information, as well as to the reviewer trying to check that nothing\n>   was forgotten.  I would rather make that only very commons are\n>   described in a common \"options\" section, and that all commands using\n>   them explicitely say so in their descriptions (with xref).\n> \n> - (on the code side of things) git-svn.perl weights more than 5500\n>   lines, most classes functions and methods severely lack\n>   documentation, and some extensively-used variable names are so short\n>   they make the code harder to grasp\n> \n>   Eg. $gs to refer to an instance of the Git::SVN class, which I would\n>   suggest to change to something like $gsrepo, while at the same time\n>   renaming Git::SVN to eg. Git::SVN::Repository - which would make it\n>   much easier for a newcomer to grasp what this is supposed to\n>   represent - supposing, that is, that my understunding of this part\n>   is accurate enough, which it is probably not after spending many\n>   hours in there :)\n> \n>   As to the size of the file, it seems natural to me to split the\n>   classes into their own files.  That would still let git-svn.perl and\n>   the Git::SVN class to be 1500-lines tall, the largest others\n>   achieving around 500 lines.  That should be much more manageable\n>   pieces, and would require some refactoring wrt a couple of global\n>   variables used throughout the script; which, incidentally, could\n>   make it much easier to simultanously look at several git-svn\n>   repositories (for my work on mapping externals to submodules), and\n>   to allow reusing the existing code, eg. as a git-vcs backend.\n> \n> How are you people feeling about this rough plans ?\n> \n"},{"id":"116725","messageId":"1245624521-30769-1-git-send-email-ydirson@altern.org","threadId":"19843","inReplyTo":"20090617201851.GA6123@nan92-1-81-57-214-146.fbx.proxad.net","subject":"[WIP PATCH 0/2] Some documentation improvements for git-svn","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2009-06-21T22:48:39Z","receivedAt":"2009-06-21T22:48:39Z","isPatch":true,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"This is a first step towards improving the git-svn documentation, as\npreviously announced.  It is by far not complete, but I'd prefer to here\nany objections to the way I change things before I have invested too\nmuch time in this :)\n\nYann Dirson (2):\n  git-svn: add some in-code documentation (options-related).\n  git-svn user documentation update.\n\n Documentation/git-svn.txt |  463 ++++++++++++++++++++++++---------------------\n git-svn.perl              |    8 +\n 2 files changed, 253 insertions(+), 218 deletions(-)\n"},{"id":"116726","messageId":"1245624521-30769-2-git-send-email-ydirson@altern.org","threadId":"19843","inReplyTo":"1245624521-30769-1-git-send-email-ydirson@altern.org","subject":"[PATCH 1/2] git-svn: add some in-code documentation (options-related).","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2009-06-21T22:48:40Z","receivedAt":"2009-06-21T22:48:40Z","isPatch":true,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"\nSigned-off-by: Yann Dirson <ydirson@altern.org>\n---\n git-svn.perl |    8 ++++++++\n 1 files changed, 8 insertions(+), 0 deletions(-)\n\ndiff --git a/git-svn.perl b/git-svn.perl\nindex 3301797..e2d5eff 100755\n--- a/git-svn.perl\n+++ b/git-svn.perl\n@@ -71,6 +71,13 @@ my ($_stdin, $_help, $_edit,\n \t$_git_format, $_commit_url, $_tag);\n $Git::SVN::_follow_parent = 1;\n $_q ||= 0;\n+\n+# Option structures here are used to define both command-line options\n+# and git-config keys.  They are handled by read-repo-config().\n+# Options in mixed case and with only a long form are only for the\n+# config file.  All options available from command-line are available\n+# as config-file options too, but without any dash in their names.\n+\n my %remote_opts = ( 'username=s' => \\$Git::SVN::Prompt::_username,\n                     'config-dir=s' => \\$Git::SVN::Ra::config_dir,\n                     'no-auth-cache' => \\$Git::SVN::Prompt::_no_auth_cache,\n@@ -184,6 +191,7 @@ my %cmd = (\n \t\t\t  'show-commit' => \\$Git::SVN::Log::show_commit,\n \t\t\t  'non-recursive' => \\$Git::SVN::Log::non_recursive,\n \t\t\t  'authors-file|A=s' => \\$_authors,\n+\t\t\t  # FIXME: lacks authors-prog ?\n \t\t\t  'color' => \\$Git::SVN::Log::color,\n \t\t\t  'pager=s' => \\$Git::SVN::Log::pager\n \t\t\t} ],\n-- \n1.6.3.1\n"},{"id":"116727","messageId":"1245624521-30769-3-git-send-email-ydirson@altern.org","threadId":"19843","inReplyTo":"1245624521-30769-2-git-send-email-ydirson@altern.org","subject":"[PATCH 2/2] git-svn user documentation update.","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2009-06-21T22:48:41Z","receivedAt":"2009-06-21T22:48:41Z","isPatch":true,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"- do not use dash-form for git commands\n- review option documentation for init and fetch:\n  - list undocumented options\n  - move some options to the right command\n  - move some options from the generic \"options\" section to where\n    they belong\n- document that all options are available through config file\n- make formatting more consistent\n- various phrasing improvements\n\nSigned-off-by: Yann Dirson <ydirson@altern.org>\n---\n Documentation/git-svn.txt |  463 ++++++++++++++++++++++++---------------------\n 1 files changed, 245 insertions(+), 218 deletions(-)\n\ndiff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt\nindex ca3fc3d..652c007 100644\n--- a/Documentation/git-svn.txt\n+++ b/Documentation/git-svn.txt\n@@ -11,11 +11,11 @@ SYNOPSIS\n \n DESCRIPTION\n -----------\n-'git-svn' is a simple conduit for changesets between Subversion and git.\n+'git svn' is a simple conduit for changesets between Subversion and git.\n It provides a bidirectional flow of changes between a Subversion and a git\n repository.\n \n-'git-svn' can track a single Subversion branch simply by using a\n+'git svn' can track a single Subversion branch simply by using a\n URL to the branch, follow branches laid out in the Subversion recommended\n method (trunk, branches, tags directories) with the --stdlayout option, or\n follow branches in any layout with the -T/-t/-b options (see options to\n@@ -27,11 +27,10 @@ Subversion updated from git by the 'dcommit' command.\n \n COMMANDS\n --------\n---\n \n 'init'::\n \tInitializes an empty git repository with additional\n-\tmetadata directories for 'git-svn'.  The Subversion URL\n+\tmetadata directories for 'git svn'.  The Subversion URL\n \tmay be specified as a command-line argument, or as full\n \tURL arguments to -T/-t/-b.  Optionally, the target\n \tdirectory to operate on can be specified as a second\n@@ -54,28 +53,17 @@ COMMANDS\n \twhich is the Subversion default. If any of the other options are given\n \tas well, they take precedence.\n --no-metadata;;\n-\tSet the 'noMetadata' option in the [svn-remote] config.\n+\tSet the 'noMetadata' option in the [svn-remote] config.  See\n+\t'CONFIG FILE OPTIONS' below.\n --use-svm-props;;\n-\tSet the 'useSvmProps' option in the [svn-remote] config.\n+\tSet the 'useSvmProps' option in the [svn-remote] config.  See\n+\t'CONFIG FILE OPTIONS' below.\n --use-svnsync-props;;\n-\tSet the 'useSvnsyncProps' option in the [svn-remote] config.\n+\tSet the 'useSvnsyncProps' option in the [svn-remote] config.  See\n+\t'CONFIG FILE OPTIONS' below.\n --rewrite-root=<URL>;;\n-\tSet the 'rewriteRoot' option in the [svn-remote] config.\n---use-log-author;;\n-\tWhen retrieving svn commits into git (as part of fetch, rebase, or\n-\tdcommit operations), look for the first From: or Signed-off-by: line\n-\tin the log message and use that as the author string.\n---add-author-from;;\n-\tWhen committing to svn from git (as part of commit or dcommit\n-\toperations), if the existing log message doesn't already have a\n-\tFrom: or Signed-off-by: line, append a From: line based on the\n-\tgit commit's author string.  If you use this, then --use-log-author\n-\twill retrieve a valid author string for all commits.\n---username=<USER>;;\n-\tFor transports that SVN handles authentication for (http,\n-\thttps, and plain svn), specify the username.  For other\n-\ttransports (eg svn+ssh://), you must include the username in\n-\tthe URL, eg svn+ssh://foo@svn.bar.com/project\n+\tSet the 'rewriteRoot' option in the [svn-remote] config.  See\n+\t'CONFIG FILE OPTIONS' below.\n --prefix=<prefix>;;\n \tThis allows one to specify a prefix which is prepended\n \tto the names of remotes if trunk/branches/tags are\n@@ -85,25 +73,83 @@ COMMANDS\n \tspecified, the prefix must include a trailing slash.\n \tSetting a prefix is useful if you wish to track multiple\n \tprojects that share a common repository.\n+--repack[=<n>];;\n+--repack-flags=<flags>;;\n+--repack-args=<flags>;;\n+--repack-opts=<flags>;;\n+\tThese should help keep disk usage sane for large fetches\n+\twith many revisions.\n++\n+'--repack' takes an optional argument for the number of revisions\n+to fetch before repacking.  This defaults to repacking every\n+1000 commits fetched if no argument is specified.\n++\n+'--repack-flags' are passed directly to 'git repack'.  '--repack-args'\n+and '--repack-opts' are aliases for '--repack-flags'.\n++\n+[verse]\n+config key: svn.repack\n+config key: svn.repackflags\n+\n+--shared[={false|true|umask|group|all|world|everybody}];;\n+--template=<template_directory>;;\n+\tThese are passed directly to 'git init'.\n+--minimize-url;;\n+-m;;\n+\tFIXME: to be documented.\n+\n++\n+The following options to 'init' are also available to the 'fetch'\n+command:\n++\n+--\n+--username=<USER>;;\n+\tFor transports that SVN handles authentication for (http,\n+\thttps, and plain svn), specify the username.  For other\n+\ttransports (eg svn+ssh://), you must include the username in\n+\tthe URL, eg svn+ssh://foo@svn.bar.com/project\n --ignore-paths=<regex>;;\n \tWhen passed to 'init' or 'clone' this regular expression will\n \tbe preserved as a config key.  See 'fetch' for a description\n \tof '--ignore-paths'.\n+--no-auth-cache;;\n+--config-dir=<path>;;\n+\tFIXME: to be documented.\n+--\n \n 'fetch'::\n \tFetch unfetched revisions from the Subversion remote we are\n \ttracking.  The name of the [svn-remote \"...\"] section in the\n \t.git/config file may be specified as an optional command-line\n-\targument.\n+\targument.  FIXME: not all configured branches are fetched\n+\tunless '--fetch-all' is specified ?\n+\n+-q::\n+--quiet::\n+\tMake 'git svn' less verbose. Specify a second time to make it\n+\teven less verbose.\n+\n+--fetch-all;;\n+--all;\n+\tFIXME: to be documented.\n+\n+-r <ARG>;;\n+--revision <ARG>;;\n+\t   This allows revision ranges for partial/cauterized history\n+\t   to be supported.  $NUMBER, $NUMBER1:$NUMBER2 (numeric ranges),\n+\t   $NUMBER:HEAD, and BASE:$NUMBER are all supported.\n++\n+This can allow you to make partial mirrors; but is generally not\n+recommended because history will be skipped and lost.\n \n --localtime;;\n \tStore Git commit times in the local timezone instead of UTC.  This\n-\tmakes 'git-log' (even without --date=local) show the same times\n+\tmakes 'git log' (even without --date=local) show the same times\n \tthat `svn log` would in the local timezone.\n \n --parent;;\n \tFetch only from the SVN parent of the current HEAD.\n-\n++\n This doesn't interfere with interoperating with the Subversion\n repository you cloned from, but if you wish for your local Git\n repository to be able to interoperate with someone else's local Git\n@@ -116,20 +162,78 @@ the same local timezone.\n \tThe '--ignore-paths' option should match for every 'fetch'\n \t(including automatic fetches due to 'clone', 'dcommit',\n \t'rebase', etc) on a given repository.\n-\n++\n+[verse]\n config key: svn-remote.<name>.ignore-paths\n++\n+If the ignore-paths config key is set and the command\n+line option is also given, both regular expressions\n+will be used.\n++\n+Examples:\n++\n+\t--ignore-paths=\"^doc\" - skip \"doc*\" directory for every fetch.\n+\t--ignore-paths=\"^[^/]+/(?:branches|tags)\" - skip \"branches\" and \"tags\"\n+of first level directories.\n \n-\tIf the ignore-paths config key is set and the command\n-\tline option is also given, both regular expressions\n-\twill be used.\n+--no-checkout;;\n+\tDo not checkout the tree after fetching.  Note that this does\n+\tnot make the converted repository a bare one.\n \n-Examples:\n+--use-log-author;;\n+\tWhen retrieving svn commits into git (as part of fetch, rebase, or\n+\tdcommit operations), look for the first From: or Signed-off-by: line\n+\tin the log message and use that as the author string.\n \n-\t--ignore-paths=\"^doc\" - skip \"doc*\" directory for every\n-\t    fetch.\n+--add-author-from;;\n+\tWhen committing to svn from git (as part of commit or dcommit\n+\toperations), if the existing log message doesn't already have a\n+\tFrom: or Signed-off-by: line, append a From: line based on the\n+\tgit commit's author string.  If you use this, then --use-log-author\n+\twill retrieve a valid author string for all commits.\n \n-\t--ignore-paths=\"^[^/]+/(?:branches|tags)\" - skip\n-\t    \"branches\" and \"tags\" of first level directories.\n+-A<filename>;;\n+--authors-file=<filename>;;\n+\tSyntax is compatible with the file used by 'git cvsimport':\n++\n+------------------------------------------------------------------------\n+\tloginname = Joe User <user@example.com>\n+------------------------------------------------------------------------\n++\n+If this option is specified and 'git svn' encounters an SVN\n+committer name that does not exist in the authors-file, 'git svn'\n+will abort operation. The user will then have to add the\n+appropriate entry.  Re-running the previous 'git svn' command\n+after the authors-file is modified should continue operation.\n++\n+[verse]\n+config key: svn.authorsfile\n+\n+--authors-prog=<filename>;;\n+\tIf this option is specified, for each SVN committer name that\n+\tdoes not exist in the authors file, the given file is executed\n+\twith the committer name as the first argument.  The program is\n+\texpected to return a single line of the form \"Name <email>\",\n+\twhich will be treated as if included in the authors file.\n+\n+--follow-parent;;\n+\tThis is especially helpful when we're tracking a directory\n+\tthat has been moved around within the repository, or if we\n+\tstarted tracking a branch and never tracked the trunk it was\n+\tdescended from. This feature is enabled by default, use\n+\t--no-follow-parent to disable it.\n++\n+[verse]\n+config key: svn.followparent\n+\n+--log-window-size=<size>;;\n+\tFIXME: to be documented.\n+\n+--username=<USER>;;\n+--ignore-paths=<regex>;;\n+--no-auth-cache;;\n+--config-dir=<path>;;\n+\tSee description above under 'init' command.\n \n 'clone'::\n \tRuns 'init' and 'fetch'.  It will automatically create a\n@@ -137,29 +241,29 @@ Examples:\n \tor if a second argument is passed; it will create a directory\n \tand work within that.  It accepts all arguments that the\n \t'init' and 'fetch' commands accept; with the exception of\n-\t'--fetch-all'.   After a repository is cloned, the 'fetch'\n-\tcommand will be able to update revisions without affecting\n-\tthe working tree; and the 'rebase' command will be able\n-\tto update the working tree with the latest changes.\n+\t'--fetch-all' and '--parent'.   After a repository is cloned,\n+\tthe 'fetch' command will be able to update revisions without\n+\taffecting the working tree; and the 'rebase' command will be\n+\table to update the working tree with the latest changes.\n \n 'rebase'::\n \tThis fetches revisions from the SVN parent of the current HEAD\n \tand rebases the current (uncommitted to SVN) work against it.\n-\n-This works similarly to `svn update` or 'git-pull' except that\n-it preserves linear history with 'git-rebase' instead of\n-'git-merge' for ease of dcommitting with 'git-svn'.\n-\n-This accepts all options that 'git-svn fetch' and 'git-rebase'\n++\n+This works similarly to `svn update` or 'git pull' except that\n+it preserves linear history with 'git rebase' instead of\n+'git merge' for ease of dcommitting with 'git svn'.\n++\n+This accepts all options that 'git svn fetch' and 'git rebase'\n accept.  However, '--fetch-all' only fetches from the current\n [svn-remote], and not all [svn-remote] definitions.\n-\n-Like 'git-rebase'; this requires that the working tree be clean\n++\n+Like 'git rebase'; this requires that the working tree be clean\n and have no uncommitted changes.\n \n -l;;\n --local;;\n-\tDo not fetch remotely; only run 'git-rebase' against the\n+\tDo not fetch remotely; only run 'git rebase' against the\n \tlast fetched commit from the upstream SVN.\n \n 'dcommit'::\n@@ -167,7 +271,7 @@ and have no uncommitted changes.\n \trepository, and then rebase or reset (depending on whether or\n \tnot there is a diff between SVN and head).  This will create\n \ta revision in SVN for each commit in git.\n-\tIt is recommended that you run 'git-svn' fetch and rebase (not\n+\tIt is recommended that you run 'git svn' fetch and rebase (not\n \tpull or merge) your commits against the latest changes in the\n \tSVN repository.\n \tAn optional command-line argument may be specified as an\n@@ -179,18 +283,17 @@ and have no uncommitted changes.\n \tAfter committing, do not rebase or reset.\n --commit-url <URL>;;\n \tCommit to this SVN URL (the full path).  This is intended to\n-\tallow existing git-svn repositories created with one transport\n+\tallow existing 'git svn' repositories created with one transport\n \tmethod (e.g. `svn://` or `http://` for anonymous read) to be\n \treused if a user is later given access to an alternate transport\n \tmethod (e.g. `svn+ssh://` or `https://`) for commit.\n-\n++\n+[verse]\n config key: svn-remote.<name>.commiturl\n-\n config key: svn.commiturl (overwrites all svn-remote.<name>.commiturl options)\n-\n-\tUsing this option for any other purpose (don't ask)\n-\tis very strongly discouraged.\n---\n++\n+Using this option for any other purpose (don't ask)\n+is very strongly discouraged.\n \n 'branch'::\n \tCreate a branch in the SVN repository.\n@@ -228,7 +331,7 @@ The following features from `svn log' are supported:\n \tsupported\n --\n +\n-New features:\n+Features not present in 'svn log':\n +\n --\n --show-commit;;\n@@ -241,7 +344,7 @@ NOTE: SVN itself only stores times in UTC and nothing else. The regular svn\n client converts the UTC time to the local time (or based on the TZ=\n environment). This command has the same behaviour.\n +\n-Any other arguments are passed directly to 'git-log'\n+Any other arguments are passed directly to 'git log'\n \n 'blame'::\n        Show what revision and author last modified each line of a file. The\n@@ -249,15 +352,14 @@ Any other arguments are passed directly to 'git-log'\n        `svn blame' by default. Like the SVN blame command,\n        local uncommitted changes in the working copy are ignored;\n        the version of the file in the HEAD revision is annotated. Unknown\n-       arguments are passed directly to 'git-blame'.\n+       arguments are passed directly to 'git blame'.\n +\n --git-format;;\n-\tProduce output in the same format as 'git-blame', but with\n+\tProduce output in the same format as 'git blame', but with\n \tSVN revision numbers instead of git commit hashes. In this mode,\n \tchanges that haven't been committed to SVN (including local\n \tworking-copy edits) are shown as revision 0.\n \n---\n 'find-rev'::\n \tWhen given an SVN revision number of the form 'rN', returns the\n \tcorresponding git commit hash (this can optionally be followed by a\n@@ -271,7 +373,13 @@ Any other arguments are passed directly to 'git-log'\n \tabsolutely no attempts to do patching when committing to SVN, it\n \tsimply overwrites files with those specified in the tree or\n \tcommit.  All merging is assumed to have taken place\n-\tindependently of 'git-svn' functions.\n+\tindependently of 'git svn' functions.\n+\n+-;;\n+--stdin;;\n+\tRead a list of commits from stdin and commit them in reverse\n+\torder.  Only the leading sha1 is read from each line, so 'git\n+\trev-list --pretty=oneline' output can be used.\n \n 'create-ignore'::\n \tRecursively finds the svn:ignore property on directories and\n@@ -286,12 +394,12 @@ Any other arguments are passed directly to 'git-log'\n \n 'commit-diff'::\n \tCommits the diff of two tree-ish arguments from the\n-\tcommand-line.  This command does not rely on being inside an `git-svn\n+\tcommand-line.  This command does not rely on being inside an `git svn\n \tinit`-ed repository.  This command takes three arguments, (a) the\n \toriginal tree to diff against, (b) the new tree result, (c) the\n \tURL of the target Subversion repository.  The final argument\n-\t(URL) may be omitted if you are working from a 'git-svn'-aware\n-\trepository (that has been `init`-ed with 'git-svn').\n+\t(URL) may be omitted if you are working from a 'git svn'-aware\n+\trepository (that has been `init`-ed with 'git svn').\n \tThe -r<revision> option is required for this.\n \n 'info'::\n@@ -313,159 +421,78 @@ Any other arguments are passed directly to 'git-log'\n \tShows the Subversion externals.  Use -r/--revision to specify a\n \tspecific revision.\n \n---\n \n OPTIONS\n -------\n---\n-\n---shared[={false|true|umask|group|all|world|everybody}]::\n---template=<template_directory>::\n-\tOnly used with the 'init' command.\n-\tThese are passed directly to 'git-init'.\n-\n--r <ARG>::\n---revision <ARG>::\n-\n-Used with the 'fetch' command.\n-\n-This allows revision ranges for partial/cauterized history\n-to be supported.  $NUMBER, $NUMBER1:$NUMBER2 (numeric ranges),\n-$NUMBER:HEAD, and BASE:$NUMBER are all supported.\n-\n-This can allow you to make partial mirrors when running fetch;\n-but is generally not recommended because history will be skipped\n-and lost.\n-\n--::\n---stdin::\n-\n-Only used with the 'set-tree' command.\n-\n-Read a list of commits from stdin and commit them in reverse\n-order.  Only the leading sha1 is read from each line, so\n-'git-rev-list --pretty=oneline' output can be used.\n \n --rmdir::\n-\n-Only used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n-\n+\tOnly used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n++\n Remove directories from the SVN tree if there are no files left\n behind.  SVN can version empty directories, and they are not\n removed by default if there are no files left in them.  git\n cannot version empty directories.  Enabling this flag will make\n the commit to SVN act like git.\n-\n++\n+[verse]\n config key: svn.rmdir\n \n -e::\n --edit::\n-\n-Only used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n-\n+\tOnly used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n++\n Edit the commit message before committing to SVN.  This is off by\n default for objects that are commits, and forced on when committing\n tree objects.\n-\n++\n+[verse]\n config key: svn.edit\n \n -l<num>::\n --find-copies-harder::\n-\n-Only used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n-\n-They are both passed directly to 'git-diff-tree'; see\n+\tOnly used with the 'dcommit', 'set-tree' and 'commit-diff' commands.\n++\n+They are both passed directly to 'git diff-tree'; see\n linkgit:git-diff-tree[1] for more information.\n-\n++\n [verse]\n config key: svn.l\n config key: svn.findcopiesharder\n \n--A<filename>::\n---authors-file=<filename>::\n-\n-Syntax is compatible with the file used by 'git-cvsimport':\n-\n-------------------------------------------------------------------------\n-\tloginname = Joe User <user@example.com>\n-------------------------------------------------------------------------\n-\n-If this option is specified and 'git-svn' encounters an SVN\n-committer name that does not exist in the authors-file, 'git-svn'\n-will abort operation. The user will then have to add the\n-appropriate entry.  Re-running the previous 'git-svn' command\n-after the authors-file is modified should continue operation.\n-\n-config key: svn.authorsfile\n-\n---authors-prog=<filename>::\n-\n-If this option is specified, for each SVN committer name that does not\n-exist in the authors file, the given file is executed with the committer\n-name as the first argument.  The program is expected to return a single\n-line of the form \"Name <email>\", which will be treated as if included in\n-the authors file.\n-\n--q::\n---quiet::\n-\tMake 'git-svn' less verbose. Specify a second time to make it\n-\teven less verbose.\n-\n---repack[=<n>]::\n---repack-flags=<flags>::\n-\n-These should help keep disk usage sane for large fetches\n-with many revisions.\n-\n---repack takes an optional argument for the number of revisions\n-to fetch before repacking.  This defaults to repacking every\n-1000 commits fetched if no argument is specified.\n-\n---repack-flags are passed directly to 'git-repack'.\n-\n-[verse]\n-config key: svn.repack\n-config key: svn.repackflags\n-\n -m::\n --merge::\n -s<strategy>::\n --strategy=<strategy>::\n-\n-These are only used with the 'dcommit' and 'rebase' commands.\n-\n-Passed directly to 'git-rebase' when using 'dcommit' if a\n-'git-reset' cannot be used (see 'dcommit').\n+\tThese are only used with the 'dcommit' and 'rebase' commands.\n++\n+Passed directly to 'git rebase' when using 'dcommit' if a\n+'git reset' cannot be used (see 'dcommit').\n \n -n::\n --dry-run::\n-\n-This can be used with the 'dcommit', 'rebase', 'branch' and 'tag'\n-commands.\n-\n+\tThis can be used with the 'dcommit', 'rebase', 'branch' and 'tag'\n+\tcommands.\n++\n For 'dcommit', print out the series of git arguments that would show\n which diffs would be committed to SVN.\n-\n++\n For 'rebase', display the local branch associated with the upstream svn\n repository associated with the current branch and the URL of svn\n repository that will be fetched from.\n-\n++\n For 'branch' and 'tag', display the urls that will be used for copying when\n creating the branch or tag.\n \n---\n \n ADVANCED OPTIONS\n ----------------\n---\n \n -i<GIT_SVN_ID>::\n --id <GIT_SVN_ID>::\n-\n-This sets GIT_SVN_ID (instead of using the environment).  This\n-allows the user to override the default refname to fetch from\n-when tracking a single URL.  The 'log' and 'dcommit' commands\n-no longer require this switch as an argument.\n+\tThis sets GIT_SVN_ID (instead of using the environment).  This\n+\tallows the user to override the default refname to fetch from\n+\twhen tracking a single URL.  The 'log' and 'dcommit' commands\n+\tno longer require this switch as an argument.\n \n -R<remote name>::\n --svn-remote <remote name>::\n@@ -473,39 +500,39 @@ no longer require this switch as an argument.\n \tthis allows SVN multiple repositories to be tracked.\n \tDefault: \"svn\"\n \n---follow-parent::\n-\tThis is especially helpful when we're tracking a directory\n-\tthat has been moved around within the repository, or if we\n-\tstarted tracking a branch and never tracked the trunk it was\n-\tdescended from. This feature is enabled by default, use\n-\t--no-follow-parent to disable it.\n-\n-config key: svn.followparent\n \n---\n CONFIG FILE-ONLY OPTIONS\n ------------------------\n---\n \n-svn.noMetadata::\n-svn-remote.<name>.noMetadata::\n+All command-line options are also available as config-file items, with\n+the same name prefixed by \"svn.\" and with any dash in their names\n+removed.  Be aware that for some options it is probably not a good\n+idea to use them in the config file, and you should probably only use\n+the config keys explicitely mentionned in this manual.  If you find\n+one useful, but don't see it mentionned together with the\n+corresponding option, please mention this on the git mailing list.\n \n-This gets rid of the 'git-svn-id:' lines at the end of every commit.\n+The extra options described below are not available directly from the\n+command-line, although some of them can be set by specific flags at\n+repository initialization.\n \n-If you lose your .git/svn/git-svn/.rev_db file, 'git-svn' will not\n+svn.noMetadata::\n+svn-remote.<name>.noMetadata::\n+\tThis gets rid of the 'git-svn-id:' lines at the end of every commit.\n++\n+If you lose your .git/svn/git-svn/.rev_db file, 'git svn' will not\n be able to rebuild it and you won't be able to fetch again,\n either.  This is fine for one-shot imports.\n-\n-The 'git-svn log' command will not work on repositories using\n++\n+The 'git svn log' command will not work on repositories using\n this, either.  Using this conflicts with the 'useSvmProps'\n option for (hopefully) obvious reasons.\n \n svn.useSvmProps::\n svn-remote.<name>.useSvmProps::\n-\n-This allows 'git-svn' to re-map repository URLs and UUIDs from\n-mirrors created using SVN::Mirror (or svk) for metadata.\n-\n+\tThis allows 'git svn' to re-map repository URLs and UUIDs from\n+\tmirrors created using SVN::Mirror (or svk) for metadata.\n++\n If an SVN revision has a property, \"svm:headrev\", it is likely\n that the revision was created by SVN::Mirror (also used by SVK).\n The property contains a repository UUID and a revision.  We want\n@@ -522,23 +549,23 @@ svn-remote.<name>.useSvnsyncprops::\n \n svn-remote.<name>.rewriteRoot::\n \tThis allows users to create repositories from alternate\n-\tURLs.  For example, an administrator could run 'git-svn' on the\n+\tURLs.  For example, an administrator could run 'git svn' on the\n \tserver locally (accessing via file://) but wish to distribute\n \tthe repository with a public http:// or svn:// URL in the\n \tmetadata so users of it will see the public URL.\n \n svn.brokenSymlinkWorkaround::\n-This disables potentially expensive checks to workaround broken symlinks\n-checked into SVN by broken clients.  Set this option to \"false\" if you\n-track a SVN repository with many empty blobs that are not symlinks.\n-This option may be changed while \"git-svn\" is running and take effect on\n-the next revision fetched.  If unset, git-svn assumes this option to be\n-\"true\".\n+\tThis disables potentially expensive checks to workaround\n+\tbroken symlinks checked into SVN by broken clients.  Set this\n+\toption to \"false\" if you track a SVN repository with many\n+\tempty blobs that are not symlinks.  This option may be changed\n+\twhile 'git svn' is running and take effect on the next\n+\trevision fetched.  If unset, 'git svn' assumes this option to be\n+\t\"true\".\n \n---\n \n Since the noMetadata, rewriteRoot, useSvnsyncProps and useSvmProps\n-options all affect the metadata generated and used by 'git-svn'; they\n+options all affect the metadata generated and used by 'git svn'; they\n *must* be set in the configuration file before any history is imported\n and these settings should never be changed once they are set.\n \n@@ -556,7 +583,7 @@ Tracking and contributing to the trunk of a Subversion-managed project:\n \tgit svn clone http://svn.example.com/project/trunk\n # Enter the newly cloned directory:\n \tcd trunk\n-# You should be on master branch, double-check with git-branch\n+# You should be on master branch, double-check with 'git branch'\n \tgit branch\n # Do some work and commit locally to git:\n \tgit commit ...\n@@ -587,12 +614,12 @@ Tracking and contributing to an entire Subversion-managed project\n # of dcommit/rebase/show-ignore should be the same as above.\n ------------------------------------------------------------------------\n \n-The initial 'git-svn clone' can be quite time-consuming\n+The initial 'git svn clone' can be quite time-consuming\n (especially for large Subversion repositories). If multiple\n people (or one person with multiple machines) want to use\n-'git-svn' to interact with the same Subversion repository, you can\n-do the initial 'git-svn clone' to a repository on a server and\n-have each person clone that repository with 'git-clone':\n+'git svn' to interact with the same Subversion repository, you can\n+do the initial 'git svn clone' to a repository on a server and\n+have each person clone that repository with 'git clone':\n \n ------------------------------------------------------------------------\n # Do the initial import on a server\n@@ -606,7 +633,7 @@ have each person clone that repository with 'git-clone':\n \tgit fetch\n # Create a local branch from one of the branches just fetched\n \tgit checkout -b master FETCH_HEAD\n-# Initialize git-svn locally (be sure to use the same URL and -T/-b/-t options as were used on server)\n+# Initialize 'git svn' locally (be sure to use the same URL and -T/-b/-t options as were used on server)\n \tgit svn init http://svn.example.com/project\n # Pull the latest changes from Subversion\n \tgit svn rebase\n@@ -615,7 +642,7 @@ have each person clone that repository with 'git-clone':\n REBASE VS. PULL/MERGE\n ---------------------\n \n-Originally, 'git-svn' recommended that the 'remotes/git-svn' branch be\n+Originally, 'git svn' recommended that the 'remotes/git-svn' branch be\n pulled or merged from.  This is because the author favored\n `git svn set-tree B` to commit a single head rather than the\n `git svn set-tree A..B` notation to commit multiple commits.\n@@ -630,7 +657,7 @@ previous commits in SVN.\n DESIGN PHILOSOPHY\n -----------------\n Merge tracking in Subversion is lacking and doing branched development\n-with Subversion can be cumbersome as a result.  While 'git-svn' can track\n+with Subversion can be cumbersome as a result.  While 'git svn' can track\n copy history (including branches and tags) for repositories adopting a\n standard layout, it cannot yet represent merge history that happened\n inside git back upstream to SVN users.  Therefore it is advised that\n@@ -641,25 +668,25 @@ CAVEATS\n -------\n \n For the sake of simplicity and interoperating with a less-capable system\n-(SVN), it is recommended that all 'git-svn' users clone, fetch and dcommit\n-directly from the SVN server, and avoid all 'git-clone'/'pull'/'merge'/'push'\n+(SVN), it is recommended that all 'git svn' users clone, fetch and dcommit\n+directly from the SVN server, and avoid all 'git clone'/'pull'/'merge'/'push'\n operations between git repositories and branches.  The recommended\n method of exchanging code between git branches and users is\n-'git-format-patch' and 'git-am', or just 'dcommit'ing to the SVN repository.\n+'git format-patch' and 'git am', or just 'dcommit'ing to the SVN repository.\n \n-Running 'git-merge' or 'git-pull' is NOT recommended on a branch you\n+Running 'git merge' or 'git pull' is NOT recommended on a branch you\n plan to 'dcommit' from.  Subversion does not represent merges in any\n reasonable or useful fashion; so users using Subversion cannot see any\n merges you've made.  Furthermore, if you merge or pull from a git branch\n that is a mirror of an SVN branch, 'dcommit' may commit to the wrong\n branch.\n \n-'git-clone' does not clone branches under the refs/remotes/ hierarchy or\n-any 'git-svn' metadata, or config.  So repositories created and managed with\n-using 'git-svn' should use 'rsync' for cloning, if cloning is to be done\n+'git clone' does not clone branches under the refs/remotes/ hierarchy or\n+any 'git svn' metadata, or config.  So repositories created and managed with\n+using 'git svn' should use 'rsync' for cloning, if cloning is to be done\n at all.\n \n-Since 'dcommit' uses rebase internally, any git branches you 'git-push' to\n+Since 'dcommit' uses rebase internally, any git branches you 'git push' to\n before 'dcommit' on will require forcing an overwrite of the existing ref\n on the remote repository.  This is generally considered bad practice,\n see the linkgit:git-push[1] documentation for details.\n@@ -685,7 +712,7 @@ for git to detect them.\n CONFIGURATION\n -------------\n \n-'git-svn' stores [svn-remote] configuration information in the\n+'git svn' stores [svn-remote] configuration information in the\n repository .git/config file.  It is similar the core git\n [remote] sections except 'fetch' keys do not accept glob\n arguments; but they are instead handled by the 'branches'\n@@ -706,7 +733,7 @@ Keep in mind that the '\\*' (asterisk) wildcard of the local ref\n however the remote wildcard may be anywhere as long as it's an\n independent path component (surrounded by '/' or EOL).   This\n type of configuration is not automatically created by 'init' and\n-should be manually entered with a text-editor or using 'git-config'.\n+should be manually entered with a text-editor or using 'git config'.\n \n SEE ALSO\n --------\n-- \n1.6.3.1\n"},{"id":"116730","messageId":"7vmy81p2r5.fsf@alter.siamese.dyndns.org","threadId":"19843","inReplyTo":"1245624521-30769-2-git-send-email-ydirson@altern.org","subject":"Re: [PATCH 1/2] git-svn: add some in-code documentation (options-related).","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-06-21T23:48:30Z","receivedAt":"2009-06-21T23:48:30Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Yann Dirson <ydirson@altern.org> writes:\n\n> +# Option structures here are used to define both command-line options\n> +# and git-config keys.  They are handled by read-repo-config().\n> +# Options in mixed case and with only a long form are only for the\n> +# config file.  All options available from command-line are available\n> +# as config-file options too, but without any dash in their names.\n\nHmm... I am a bit confused.  So for example:\n\n\t\t'authors-file|A=s' => \\$_authors,\n\t\t'authors-prog=s' => \\$_authors_prog,\n\t\t'repack:i' => \\$Git::SVN::_repack,\n\t\t'noMetadata' => \\$Git::SVN::_no_metadata,\n\t\t'useSvmProps' => \\$Git::SVN::_use_svm_props,\n\nauthors-file, authors-prog and repack are not MixdCase (and the first and\nthird are not even \"only a long form\"), so it is not \"only for the config\nfile\".  But noMetadata and useSvmProps are \"only for the config file\".\n\nIt took me a bit to realize that these two equivalence hold:\n\n  \"not only for the config file\" === \"also available as command line option\"\n  \"only for the config file\"     === \"cannot be given on the command line\"\n\nSince \"option structures here are used for both command and config\", and\n\"anything usable from command line can be used in config file\", logically\nit follows that \"everything in these option structures here are usable in\nthe config\".  It took me a bit longer than necessary for that to click,\ntoo.\n\nPerhaps I am slow.  But I suspect I am not the slowest around here.\n\nAfter scratching my head for a few minutes, here is a rephrased\ndescription I came up with (I am not writing this as an improved\nsuggestion, but to ask you to sanity check if I read what you wanted to\nsay correctly):\n\n    These are all configurable in the config file (remove dash in their\n    names to use these names as the config key).\n\n    Options that have MixedCase names without short format (e.g. noMetadata)\n    can only be used from the configuration (iow they cannot be given from\n    the command line as options).\n\nBut then this raises a couple of questions.\n\n * What are the examples of MixedCase names that do have short format?\n   Are they usable from the command line?\n\n * %fc_opts() has \"noMetadata\" and %init_opts() has \"no-metadata\"; how are\n   they related, and how do they affect the above \"what's available where\n   and how to spell their names\" rule you outlined?\n"},{"id":"116783","messageId":"20090622191207.GA5590@nan92-1-81-57-214-146.fbx.proxad.net","threadId":"19843","inReplyTo":"7vmy81p2r5.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 1/2] git-svn: add some in-code documentation (options-related).","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2009-06-22T19:12:07Z","receivedAt":"2009-06-22T19:12:07Z","isPatch":true,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"On Sun, Jun 21, 2009 at 04:48:30PM -0700, Junio C Hamano wrote:\n> Yann Dirson <ydirson@altern.org> writes:\n> \n> > +# Option structures here are used to define both command-line options\n> > +# and git-config keys.  They are handled by read-repo-config().\n> > +# Options in mixed case and with only a long form are only for the\n> > +# config file.  All options available from command-line are available\n> > +# as config-file options too, but without any dash in their names.\n> \n> Hmm... I am a bit confused.  So for example:\n> \n> \t\t'authors-file|A=s' => \\$_authors,\n> \t\t'authors-prog=s' => \\$_authors_prog,\n> \t\t'repack:i' => \\$Git::SVN::_repack,\n> \t\t'noMetadata' => \\$Git::SVN::_no_metadata,\n> \t\t'useSvmProps' => \\$Git::SVN::_use_svm_props,\n> \n> authors-file, authors-prog and repack are not MixdCase (and the first and\n> third are not even \"only a long form\"), so it is not \"only for the config\n> file\".  But noMetadata and useSvmProps are \"only for the config file\".\n> \n> It took me a bit to realize that these two equivalence hold:\n> \n>   \"not only for the config file\" === \"also available as command line option\"\n>   \"only for the config file\"     === \"cannot be given on the command line\"\n> \n> Since \"option structures here are used for both command and config\", and\n> \"anything usable from command line can be used in config file\", logically\n> it follows that \"everything in these option structures here are usable in\n> the config\".  It took me a bit longer than necessary for that to click,\n> too.\n\nRight, that should probably be emphasized more.  I originally did not\nrealize that those also drive config-file settings, right after I\ndiscovered how the config items are handled, and this clearly shows :)\n\n\n> After scratching my head for a few minutes, here is a rephrased\n> description I came up with (I am not writing this as an improved\n> suggestion, but to ask you to sanity check if I read what you wanted to\n> say correctly):\n> \n>     These are all configurable in the config file (remove dash in their\n>     names to use these names as the config key).\n> \n>     Options that have MixedCase names without short format (e.g. noMetadata)\n>     can only be used from the configuration (iow they cannot be given from\n>     the command line as options).\n\nThat's what I had got from comments in read_repo_config() and tried to\nconvey in this comment.\n\n> But then this raises a couple of questions.\n> \n>  * What are the examples of MixedCase names that do have short format?\n>    Are they usable from the command line?\n\nThere are no such example, and I suspect this restriction was only\nmentionned for completeness, as a consequence that config-only items\nare defined in the code by containing only letters and at least one\nuppercase (that is, the real condition also encompasses uppercase-only\nitems).  I'll try to bring a better formulation.\n\n\n>  * %fc_opts() has \"noMetadata\" and %init_opts() has \"no-metadata\"; how are\n>    they related, and how do they affect the above \"what's available where\n>    and how to spell their names\" rule you outlined?\n\nThe mixed-case versions are the config items, and the dashed versions\nare used by \"git svn init\" to seed these items at init/clone time, so\none does not have to use \"git svn init\", then set config, and only\nthen doing initial fetch.\n\nAs a consequence, one can set eg. svn.nometadata in his ~/.gitconfig,\nand subsequently have all git-svn-cloned repository initialized with\nsvn.noMetadata.  That's probably not an example of clarity, and things\nbeing error-prone, such an example in the doc would probably be\nwelcomed; I can add this too.\n\n\nSpeaking about the doc, I was wondering whether it would not be better\nto split my previous patch into 1/3 formatting issues (patch which\ncould probably be applied early), 2/3 shuffling existing options to\nthe right place, 3/3 adding missing options.  Afterwards more work can\nbe done for clarification, but that is not my primary goal, I'd rather\ntry to document the internals first.\n"},{"id":"116907","messageId":"20090625084727.GA2901@dcvr.yhbt.net","threadId":"19843","inReplyTo":"20090617201851.GA6123@nan92-1-81-57-214-146.fbx.proxad.net","subject":"Re: Improving git-svn documentation","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2009-06-25T08:47:27Z","receivedAt":"2009-06-25T08:47:27Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"Yann Dirson <ydirson@altern.org> wrote:\n> While diving into the git-svn code, I realized that many things could\n> be done to make it more documented/understundable.  I need to get more\n> understanding of it, so I'd like to improve this state of things.  But\n> first, I'd like to be sure there is a consensus on what is a good idea\n> to do, since that could easily turn up into a lot of textual change.\n> \n> - (on the user doc side of things) some options appear not to be\n>   documented (I spotted --parent for 'clone' and --revision for\n>   'dcommit').  But looking at where to document them, I found it not\n>   always easy, since some options are documented together with the\n>   command they modify, some others in the \"options\" section (even when\n>   they are documented as applying to a single command, like --shared\n>   or --stdin).  This IMHO leads to confusion for the user looking for\n>   information, as well as to the reviewer trying to check that nothing\n>   was forgotten.  I would rather make that only very commons are\n>   described in a common \"options\" section, and that all commands using\n>   them explicitely say so in their descriptions (with xref).\n\nSadly I haven't ever done much maintenance on the documentation myself.\nI've found building/formatting the docs too painful of an experience for\nme (this coming from a guy willing to deal with SVN) so I haven't made\nmany attempts in several years.\n\n> - (on the code side of things) git-svn.perl weights more than 5500\n>   lines, most classes functions and methods severely lack\n>   documentation, and some extensively-used variable names are so short\n>   they make the code harder to grasp\n> \n>   Eg. $gs to refer to an instance of the Git::SVN class, which I would\n>   suggest to change to something like $gsrepo, while at the same time\n>   renaming Git::SVN to eg. Git::SVN::Repository - which would make it\n>   much easier for a newcomer to grasp what this is supposed to\n>   represent - supposing, that is, that my understunding of this part\n>   is accurate enough, which it is probably not after spending many\n>   hours in there :)\n\nI think I was at least fairly consistent with naming most Git::SVN\nobjects $gs, at least (and vectors are \"$gsv\" (a la arg => argv).  But I\ndo admit it's not the easiest code to follow since the majority of it\nwas written in a short time span in early 2007 before I was to\n(knowingly) embark on other life-changing (or life-consuming :)\nprojects.  I seem to recall most of my time was spent dealing\nwith/working around various issues I had with SVN Perl bindings, but\nI've probably blocked out the most traumatizing experiences.\n\n>   As to the size of the file, it seems natural to me to split the\n>   classes into their own files.  That would still let git-svn.perl and\n>   the Git::SVN class to be 1500-lines tall, the largest others\n>   achieving around 500 lines.  That should be much more manageable\n>   pieces, and would require some refactoring wrt a couple of global\n>   variables used throughout the script; which, incidentally, could\n>   make it much easier to simultanously look at several git-svn\n>   repositories (for my work on mapping externals to submodules), and\n>   to allow reusing the existing code, eg. as a git-vcs backend.\n> \n> How are you people feeling about this rough plans ?\n\nI've been planning on splitting git-svn.perl out into separate modules\nfor years now.  Please take the lead on it if you wish to, since I've\nobviously been unable to dedicate much time to git-svn in over two years\nnow.\n\n-- \nEric Wong\n"}]}