Hi, On Wed, May 13, 2015 at 3:52 AM, Michael Haggerty <mhagger@xxxxxxxxxxxx> wrote: > I never use pull so I'm not really acquainted with its semantics. But it > seems to me that when you remove the special "--tags" error message, you > might also have to adjust the logic later in the function that looks at > "$#". Specifically, unless the presence of a "--tags" option can provide > candidates for merging, then in "[ $# -gt 1 ]", "$#" might need to be > changed to "the number of arguments *not including --tags arguments*". Yes, I'm aware of the problems of using "$@" and "$#" including git-fetch's options in git-pull[1]. It's not just enough to skip over arguments that look like options, though, as we do not know if the options take values or not, so the whole logic may become complicated. I'm planning to solve it in a later patch series by explicitly parsing git-fetch's options as well. [1] http://thread.gmane.org/gmane.comp.version-control.git/268510/focus=268565 Thanks, Paul -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html